search

frc1418 / 2015-robot

Team 1418's Robot code for 2015
Apache License 2.0
1 stars 1 forks source link

Hopefully this works. #71

Closed ArchdukeTim closed 9 years ago

virtuald commented 9 years ago

In the future, I would recommend placing each individual new feature in it's own pull request (eg, separate commits, separate branches).

I'm pretty sure the accelerometer code isn't going to work at all. And your switch code is going to raise the forklift multiple times while the switches are pressed.

It shouldn't raise it multiple times, because self.autolift is false until both limit switches aren't pressed

virtuald commented 9 years ago

Oh, I see. You added a debounce. Ok, but the accelerometer comment stands.