Skip to content

(#236)Added warning for incompatible autos - #238

Merged
Tyrone337 merged 3 commits into
mainfrom
236-implement-auto-location-warning
Oct 24, 2026
Merged

Tyrone337 merged 3 commits into
mainfrom
236-implement-auto-location-warning

Conversation

@Tyrone337

Copy link
Copy Markdown
Contributor

-changed autos to use a Record so they could store their location. Then, created a warning for if the selected auto and the robot's location dont match up.

-changed autos to use a Record so they could store their location. Then, created a warning for if the selected auto and the robot's location dont match up.

@godmar godmar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

see comments

Comment thread src/main/java/frc/robot/auto/Autos.java Resolved
Comment thread src/main/java/frc/robot/auto/Autos.java Resolved
-removed the aggresive from all auto names because its uneccessary without conservative autos and takes up too much space.
@Tyrone337
Tyrone337 requested review from godmar and thetaback October 23, 2026 22:55
new Elastic.Notification(
Elastic.NotificationLevel.ERROR,
"Incorrect Auto",
"Selected auto is incompatable with position."));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

incompatible not incompatable

@Tyrone337
Tyrone337 merged commit d16630a into main Oct 24, 2026
3 checks passed
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.

3 participants