Open cosmiccricket opened 11 years ago
I am not 100% sure, I think we need to ask @gint271 about that. I think it should be ok, because it will force the case-statement to fall through to the default case, meaning it resets back to normal, but what is the intent of that button normally? (comment more what a thing is meant to be doing dude...)
The intent of that button is to make the arm stop whatever it's doing and go back to its default position.
ok, now i see. here is what that should look like then:
lifterstep
to a "retract/reset everything step" which is step 9 to make sure that the suction and arm are retracted as welllifterstep
is changed we should be doing this (except in the case statement...)lifterstart
is True
to make sure the clock is runningThis should do it if I am understanding correctly, but double check it first:
if (joystick.GetRawButton(7)) {
lifterStep = 0;
timer.Reset();
timer.Start();
lifterStart = true;
}
Also, I think that step 9 and 10 might be combined, but that might require testing...
https://github.com/team2517/2013_code/blob/master/2013robot.cpp#L316
Is this legit?
https://github.com/team2517/2013_code/blob/master/2013robot.cpp#L392
Should this be 1 instead of 0?