pathplanner

pathplanner copied to clipboard

changed "getStaringPoseFromAutoFile" to non-misspelled version

2

our programmers tore their hair out over this

ironon

Altered Naming Convention for Duplicates

6

In my experience the current naming convention of "Copy of {AutoName}" is very inconvenient, primarily when sorting the Path/Auto names alphabetically as duplicating an auto will send it to a...

Ammelll

From code inspection, the C++ implementation does not honour the rotation target override. https://github.com/mjansen4857/pathplanner/blob/05a123d86f47976c564564674fdb997e25b03773/pathplannerlib/src/main/native/cpp/pathplanner/lib/controllers/PPHolonomicDriveController.cpp#L59-L68 Compare with he Java implementation: https://github.com/mjansen4857/pathplanner/blob/05a123d86f47976c564564674fdb997e25b03773/pathplannerlib/src/main/java/com/pathplanner/lib/controllers/PPHolonomicDriveController.java#L147-L156 I suggest that this should should instead say: ```c++ units::radians_per_second_t...

bovlb

Add ability to inject parameters into commands from the path planner GUI

7

I've had this idea on the wish list before and I release its hard to do because the commands are declared in code before they are added to the command...

WarrenReynolds

Changing name of Auto in GUI does not remove old name from RIO or Chooser on Driver Station

3

**Describe the bug** Editing the name of an Auto or Path within the Pathplanner GUI does not remove the old filename from the RoboRIO on the next deploy. This results...

OhHey-ItsDave

This fixes the static initialization order in NamedCommands by creating a function that initializes the variable on first use. This is the recommended fix by Isocpp https://isocpp.org/wiki/faq/ctors#static-init-order-on-first-use-members

r4stered

Added filtering to buildAutoChooser for C++

1

Added a new input for buildAutoChooser for C++ which allows filtering out autos based on their internal data like name or start position. Default Option is guaranteed to be unfiltered...

ThePhoenixFox

**Is your feature request related to a problem? Please describe.** Currently if a team tries to build a path where the file doesn't exist, their entire code crashes, which if...

adamsong

Use std::string_view instead of string when applicable to avoid heap allocations

2

For example, in NamedCommands, we pass an std::string, instead of a std::string_view. Passing a string_view would save some allocations.

r4stered

**Describe the bug** When calling NamedCommands::registerCommand(), the static vector namedCommands might not be initialized based on the order of compilation. https://en.cppreference.com/w/cpp/language/siof **To Reproduce** Steps to reproduce the behavior: 1. Create...

r4stered