Skip to content

Fix gripper encoder wrap correction - #82

Draft
branyang02 wants to merge 2 commits into
i2rt-robotics:mainfrom
AfterQuery-Research:main
Draft

Fix gripper encoder wrap correction#82
branyang02 wants to merge 2 commits into
i2rt-robotics:mainfrom
AfterQuery-Research:main

Conversation

@branyang02

Copy link
Copy Markdown

What changed

  • Extract the startup encoder-wrap adjustment into a testable helper.
  • Restrict the adjustment to the six arm motors.
  • Add a regression test proving a multi-turn seventh gripper motor is left unchanged.

Why

Linear grippers can legitimately travel beyond 2π radians. The existing startup loop applies its ±2π arm-encoder correction to every motor state, including the calibrated gripper. That changes the gripper motor offset while retaining the original calibration endpoints, putting feedback and commands in different coordinate frames.

This preserves the existing correction for arm joints while keeping calibrated gripper coordinates intact.

Validation

Not run yet at the operator's request because the robot is currently being used for another task. The regression test is included but has not been executed. No hardware, CAN interface, or robot process was accessed while preparing this change.

Move the replacement linear_4310 grasp point to 220 mm along flange-local -Z, align its axes with the UMI convention, and add FK/IK regression coverage.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant