Skip to content
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

Allow for HPFC things to be tuned numerically through advantagescope #186

Merged
merged 12 commits into from
Feb 23, 2024

Conversation

GalexY727
Copy link
Member

Also speeds up autos. Good things! This could use some extraction if anyone is up for it, though I don't think its critical.

@GalexY727 GalexY727 added autonomous Something that needs to get done for the Autonomous period of the match command A command on the robot that controls a subsystem labels Feb 23, 2024
@GalexY727 GalexY727 self-assigned this Feb 23, 2024
@GalexY727
Copy link
Member Author

When reviewing please remember to use the file filter to ignore .paths and .jsons, thank me later :)

@GalexY727
Copy link
Member Author

i forgot to show off what changed via gif:
i
d

Copy link
Contributor

@Oliver-Cushman Oliver-Cushman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mmm.......

the fix for pid wrapping should make this reasonable
@GalexY727 GalexY727 merged commit 285b9e7 into main Feb 23, 2024
1 check passed
@GalexY727 GalexY727 deleted the auto-2 branch February 23, 2024 20:13
@Jacob1010-h
Copy link
Member

#187 implements network table manager

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autonomous Something that needs to get done for the Autonomous period of the match command A command on the robot that controls a subsystem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants