FRO5401 / PowerUp2018

2 stars 0 forks source link

Moved threshold check and done flag #24

Closed kjm16216 closed 6 years ago

kjm16216 commented 6 years ago

Added the logic we discussed at the end of the meeting last night.

STATUS

REVIEW

DESCRIPTION

Incorporates the logic we discussed at the end of the meeting Mon night. We were finding the turn was frequently overshooting, thinking that is a latency issue from when IsFinished returns true and end method actually executes. Revision would check that angle is within threshold during execute, and if so, disable the PID and stop the motors. Creates a separate flag to trigger IsFinished that's raised in the same code block.

FILES THAT CONTAINTED MERGE CONFLICTS

-

TO DO

ptoil commented 6 years ago

Auto-Turn PID tested and is reliable to a margin of error up to ( +/- ) 1 degrees.