Andover-Robotics / ARC-Core

This is the combined common code base for Andover Robotics Club, packaged in a library.
0 stars 9 forks source link

Power domain inconsistencies #20

Closed broad-well closed 5 years ago

broad-well commented 5 years ago

This issue also hit us hard during diagnoses. For example, startMoveToPosition's implementation should not assume that the given power is absolute.

Furthermore, I have found that the power setting for encoder-based movement should not be negative. The motor will automatically move in the direction that makes the encoder value approach the target position.

broad-well commented 5 years ago

The changed tests now pass; this problem appears to be resolved. If it occurs again with the most current version of the codebase, reopen this issue.