-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Vision Files #54
Comments
Notes: Working on adding logging to visionstates Vision still needs to be tested along with its config values on actual |
Citing: https://www.chiefdelphi.com/t/need-help-getting-pidcommand-working/452333/5 and https://www.chiefdelphi.com/t/command-based-help/446143/19 for reference on building PID controllers instead of PIDCommand wpilib (2025) documentation: https://github.wpilib.org/allwpilib/docs/release/java/edu/wpi/first/math/controller/PIDController.html |
Update: Plan to process the commands directly into vision using PID controller to reroute our position with vision instead |
Seperated the four main configs (ideas, smth cause there's no name for the separation yet) for Vision.java: Config (Command Config)- used to build each seperate command's PID command values and offsets VisionStates Commands- commands used directly into vision states Vision Commands- built to send commands between Vision commands, basically a transit between config methods and Vision Command |
Using the new Vision Files and updating its logging, commands built for this season, etc.
The text was updated successfully, but these errors were encountered: