Skip to content

Improve tensioning: default motor movement, stall detection, gradual release#66

Merged
clemens-chr merged 1 commit into
mainfrom
feature/improve-tensioning
May 12, 2026
Merged

Improve tensioning: default motor movement, stall detection, gradual release#66
clemens-chr merged 1 commit into
mainfrom
feature/improve-tensioning

Conversation

@fabricenoelbourquin

@fabricenoelbourquin fabricenoelbourquin commented May 12, 2026

Copy link
Copy Markdown
Member

Summary

  • Move motors by default: tension.py now moves motors without needing --move-motors. Added --no-move-motors flag to skip. --move-motors kept for compatibility.
  • Stall detection: Replaced fixed-duration movement loops with position-based stall detection — motors move until stalled, hold 1s, then proceed. No more unnecessary waiting at the edges.
  • Gradual torque release: After preconditioning, torque ramps down over 1s before re-engaging at the relaxed position, preventing tendon snap-back.

Test plan

  • uv run pytest tests/test_tension.py — all 4 tests pass
  • Run tension.py on hardware and verify motors move by default
  • Verify --no-move-motors skips motor movement
  • Verify stall detection transitions smoothly between directions
  • Verify gradual torque release prevents tendon snap-back

🤖 Generated with Claude Code

…torque release

- Default to move_motors=True so tension.py moves motors without needing --move-motors
- Add --no-move-motors flag to skip motor movement
- Replace fixed-duration movement loops with stall detection (1s hold after stall)
- Gradually ramp down torque before hold phase to prevent tendon snap-back
@clemens-chr
clemens-chr force-pushed the feature/improve-tensioning branch from baef93b to f358b37 Compare May 12, 2026 14:36

@clemens-chr clemens-chr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good frabrice

@clemens-chr
clemens-chr merged commit 4ea7d96 into main May 12, 2026
1 check passed
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.

2 participants