Skip to content

Conversation

@BrodyKarr
Copy link
Contributor

I made the "shell" of the statemachine based of the given list for the states, and our drive states from on season.

I made the "shell" of the statemachine based of the given list for the states, and our drive states from on season.
@BrodyKarr BrodyKarr requested a review from a team as a code owner July 20, 2025 22:55
Copy link
Member

@TaylerUva TaylerUva left a comment

Choose a reason for hiding this comment

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

I would probably have the commands be prints right now as this will make the code crash (since commands don't exist yet). With prints you'll also be able to test the StateMachine in sim. Aside from that this looks great!

@BrodyKarr
Copy link
Contributor Author

I made it so it can build

Copy link
Member

@TaylerUva TaylerUva left a comment

Choose a reason for hiding this comment

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

In initialize of the commands we need to set the robot state to the name of the command

@TaylerUva TaylerUva enabled auto-merge (squash) July 21, 2025 02:04
@TaylerUva TaylerUva disabled auto-merge July 21, 2025 02:04
@TaylerUva TaylerUva enabled auto-merge (squash) July 21, 2025 02:04
Copy link
Member

@TaylerUva TaylerUva left a comment

Choose a reason for hiding this comment

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

In initialize of the commands we need to set the robot state to the name of the command

@TaylerUva TaylerUva merged commit e07710e into main Jul 24, 2025
1 check passed
@TaylerUva TaylerUva deleted the create-statemachine branch July 24, 2025 02:18
@BrodyKarr BrodyKarr linked an issue Jul 24, 2025 that may be closed by this pull request
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.

Finish StateMachine Subsystem

3 participants