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

Autonomous Task #322

Merged
merged 15 commits into from
May 20, 2024
Merged

Autonomous Task #322

merged 15 commits into from
May 20, 2024

Conversation

huttongrabiel
Copy link
Contributor

See old PR for more detailed info/commits/reviews.

@huttongrabiel huttongrabiel requested a review from abhaybd May 4, 2024 17:46
huttongrabiel and others added 12 commits May 6, 2024 18:07
Use DriveToWaypointCommand functionality to execute an autonomous task.

Add necessary constants to Constants.h

Start autonomous task from MissionControlProtocol.cpp
JavaScript truncates any value such as 0.0 or 1.0 to 0 and 1,
respectively. This fails validation because we looks for floats and
those come across as unsigned ints. To fix this, we check if the
incoming number is either a float or an unsigned number. Both are
accepted.
@abhaybd abhaybd requested a review from quinnmp May 20, 2024 05:46
Copy link
Member

@abhaybd abhaybd left a comment

Choose a reason for hiding this comment

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

Looks good! (after how many months of testing lmao)

@abhaybd abhaybd merged commit 8902619 into master May 20, 2024
3 checks passed
@abhaybd abhaybd deleted the autonomous-task branch May 20, 2024 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants