-
Notifications
You must be signed in to change notification settings - Fork 0
R2 73 implement run animation #23
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
base: main
Are you sure you want to change the base?
Conversation
| public class LEDs extends SubsystemBase { | ||
| private final LightsIO _io; | ||
| private final LightsIO _io; | ||
| private final CANdle _candle; |
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.
Very close to what we want, but I might not have told you the right type. If you look under src\main\java\frc\lib\W8\devices, what class might work here to run our animations
| } | ||
| public LEDs(LightsIO io) { | ||
| _io = io; | ||
| _candle = new CANdle(1, "rio"); |
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 shouldn't have to instantiate a new CANdle, it should be taken care of when we actually create the subsystem in robot container
| public Command runAnimation() { | ||
| return this.startEnd( | ||
| () -> _candle.setControl(LEDConstants.rainbowAnim.withSlot(0).withColor(LEDConstants.colorPaleBlue)), | ||
| () -> _candle.setControl(new SolidColor(0, 3).withColor(colorWheezerBlue)) |
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.
After you change out the mechanism, these will have to be updated. Make sure you run a build to catch any errors
Please work ( or something like that ).