Skip to content

feat: add MaxonActuator brushed motor support - #532

Closed
ninankh wants to merge 8 commits into
neurobionics:mainfrom
ninankh:vso_actuator_update
Closed

ninankh wants to merge 8 commits into
neurobionics:mainfrom
ninankh:vso_actuator_update

Conversation

@ninankh

@ninankh ninankh commented Aug 14, 2026

Copy link
Copy Markdown

**Depends on #531 ** — this PR is built on top of that branch and includes its commits. Please review/merge #531 first; this diff will shrink to just the new changes once that's merged and this PR is retargeted to main.

Summary

Adds brushed motor support via a new MaxonActuator class, with the base
Actuator class updated to accommodate the brushed motor control interface.
Includes unit tests and a tutorial with sample usage code.

Changes

  • MaxonActuator: new brushed motor actuator implementation (H-bridge PWM control)
  • Actuator base class: updated to support brushed motor configuration
  • Unit tests added for MaxonActuator and updated base class behavior
  • Tutorial added under tutorials/ with sample code demonstrating homing and commanding motor position.

Testing

  • Added pytest coverage for MaxonActuator and affected base class paths
  • Ran make test locally, all passing
  • Ran make check for lint/format compliance
  • Manually verified tutorial sample code runs end-to-end

Checklist

  • Tests included for new functionality
  • Tutorial/docs added for new functionality

@ninankh ninankh closed this Aug 14, 2026
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