SleipnirGroup / Choreo

A graphical tool for planning time-optimized trajectories for autonomous mobile robots in the FIRST Robotics Competition.
BSD 3-Clause "New" or "Revised" License
150 stars 46 forks source link

`AutoChooser::AddAutoRoutine` lambda parameter inconsistent between Java and C++ #906

Closed bhall-ctre closed 1 week ago

bhall-ctre commented 3 weeks ago

The Java version takes a function that accepts an AutoFactory and returns an AutoRoutine (which does not completely match documentation), whereas the C++ version takes a function that accepts nothing and returns a CommandPtr (which also does not completely match documentation).