Add LED's to display robot state #46
Conversation
spacey-sooty
force-pushed
the
peter/LEDintegration
branch
from
d776edb
to
f1a8f1b
Compare
|
/format |
There was a problem hiding this comment.
IMO we should display info about intaking and robot position as well.
Needs coordination with driveteam and @totallysomeoneyoudontknow
cc @Hazzer890 I'm sure you have an opinion on what the LEDs should do
| lEDPwm.setSpeed(0.67); | ||
| break; | ||
|
|
||
| case kIntake: | ||
| case kSpeaker: | ||
| position = 3.7; | ||
| lEDPwm.setSpeed(0.73); | ||
| break; | ||
|
|
||
| case kSpeaker: | ||
| case kIntake: | ||
| position = 3.7; | ||
| lEDPwm.setSpeed(0.57); | ||
| break; | ||
|
|
||
| case kStowed: | ||
| position = 3.7; | ||
| lEDPwm.setSpeed(0.99); |
There was a problem hiding this comment.
Could we have comments to say what that actually displays, its not obvious from the code
| @@ -42,6 +43,8 @@ public enum Setpoint { | |||
| private DoubleLogEntry log_ff_output = new DoubleLogEntry(m_log, "/arm/ff/output"); | |||
| private StringLogEntry log_setpoint = new StringLogEntry(m_log, "/arm/setpoint"); | |||
|
|
|||
| PWM lEDPwm = new PWM(0); | |||
There was a problem hiding this comment.
this port needs to be determined still no?
| PWM lEDPwm = new PWM(0); | |
| private PWM m_led = new PWM(0); |
There was a problem hiding this comment.
this port needs to be determined still no?
maybe, was going off memory, could also be port 1
currently working off LED patterns decided with Layla during season. |
| lEDPwm.setSpeed(0.67); // gold // | ||
| break; | ||
|
|
||
| case kIntake: | ||
| case kSpeaker: | ||
| position = 3.7; | ||
| lEDPwm.setSpeed(0.73); // lime // | ||
| break; | ||
|
|
||
| case kSpeaker: | ||
| case kIntake: | ||
| position = 3.7; | ||
| lEDPwm.setSpeed(0.57); // hot pink // | ||
| break; | ||
|
|
||
| case kStowed: | ||
| position = 3.7; | ||
| lEDPwm.setSpeed(0.99); // blank // |
SturrockPeter
force-pushed
the
peter/LEDintegration
branch
from
0c58f49
to
f6c5cc0
Compare
Finished LED integration within arm states
Resolves #40