Skip to content

Conversation

@Craftzman7
Copy link
Member

@Craftzman7 Craftzman7 commented Oct 28, 2025

This PR exists so main always has working code that corresponds to the currently built robot. As of right now that's just a Mk5n drivetrain.

@Craftzman7 Craftzman7 marked this pull request as draft October 28, 2025 22:05
@Craftzman7 Craftzman7 marked this pull request as ready for review November 14, 2025 05:57
@Craftzman7
Copy link
Member Author

It's a lot of code, I know. Skip over the Choreo library source code for obvious reasons. Only review actual robot code please :)

Copy link
Member

@Gavin-Niederman Gavin-Niederman left a comment

Choose a reason for hiding this comment

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

will do second pass in the future, overall except for the one issue and nit LGTM

Indexer.index(),
blinkLimelight()
).alongWith(
Commands.sequence(
Copy link
Member

Choose a reason for hiding this comment

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

make sure that the right commands are repeated

beforeFirstEnable = true
// autoCommand = Dashboard.autoChooser.selected
autoCommand?.schedule()
if (!RobotState.beforeFirstEnable)
Copy link
Member

Choose a reason for hiding this comment

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

falsy as fuck

@Craftzman7 Craftzman7 mentioned this pull request Nov 19, 2025
@Craftzman7 Craftzman7 merged commit acab603 into main Nov 19, 2025
1 check 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.

6 participants