-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
States ash prep #117
States ash prep #117
Conversation
Signed-off-by: KP <86213869+Kanishk-Pandey@users.noreply.github.com>
Signed-off-by: KP <86213869+Kanishk-Pandey@users.noreply.github.com>
Signed-off-by: KP <86213869+Kanishk-Pandey@users.noreply.github.com>
driverController | ||
.a() | ||
.whileTrue(FeederCommands.feedToShooter(feeder)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can put this on the operator controller but right now we are trying to reduce the number of controls on the drivers controller
.andThen( | ||
LEDCommands.setIntakeType(leds) | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The LEDs are making me confused, if we had the LEDs actually visually indicate up or down then I might approve
}, shooter) | ||
.withName("Shooter Idle"); | ||
} | ||
|
||
public static Command SHOOTLONGWAY (Shooter shooter) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Name? Maybe document this one idk what this is supposed to do
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm really confused by some of these changes
basic changes