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

Speed angle pair to main #34

Merged
merged 2 commits into from
Jan 21, 2024
Merged

Speed angle pair to main #34

merged 2 commits into from
Jan 21, 2024

Conversation

PatribotsProgramming
Copy link
Member

No description provided.

GalexY727 and others added 2 commits January 21, 2024 02:33
which is a way to store a speed and an angle in one type.
This kind of code might be controversial,
so discussions will need to be made about it before it gets merged,
if it gets merged.

I think its a good idea though :)
rename speaker map constant
add measurement distance constant
@PatribotsProgramming PatribotsProgramming added enhancement New feature or request autonomous Something that needs to get done for the Autonomous period of the match logic A logic problem or solution ready to test Ready to test on the robot :> teleop Something that needs to get done for the Teleop period of the match labels Jan 21, 2024
@PatribotsProgramming PatribotsProgramming added this to the Piece Control milestone Jan 21, 2024
@PatribotsProgramming PatribotsProgramming merged commit 3977d14 into main Jan 21, 2024
1 check passed
@PatribotsProgramming PatribotsProgramming deleted the SpeedAnglePair branch January 21, 2024 22:26
@Jacob1010-h Jacob1010-h mentioned this pull request Jan 31, 2024
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 enhancement New feature or request logic A logic problem or solution ready to test Ready to test on the robot :> teleop Something that needs to get done for the Teleop period of the match
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants