Skip to content

feat: add VSO robot class with unitTest - #534

Open
ninankh wants to merge 17 commits into
neurobionics:mainfrom
ninankh:vso_robot_update
Open

feat: add VSO robot class with unitTest#534
ninankh wants to merge 17 commits into
neurobionics:mainfrom
ninankh:vso_robot_update

Conversation

@ninankh

@ninankh ninankh commented Aug 14, 2026

Copy link
Copy Markdown

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

Summary

Adds a new VSO robot class that integrates the device's sensors for
reading. Includes unit tests for the new class and a tutorial with
sample code demonstrating sensor reads.

Changes

  • VSO: new robot class integrating sensor reading functionality
  • Unit tests added for VSO sensor read methods
  • Tutorial added under tutorials/ with sample code demonstrating how to
    read VSO's sensors

Testing

  • Added pytest coverage for VSO sensor reading logic
  • Ran make test locally, all passing
  • Ran make check for lint/format compliance

Checklist

  • Tests included for new functionality

@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.60718% with 24 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
opensourceleg/robots/vso.py 87.9% 10 Missing and 1 partial ⚠️
opensourceleg/sensors/adc.py 98.3% 8 Missing and 1 partial ⚠️
opensourceleg/actuators/brushed.py 98.4% 2 Missing and 1 partial ⚠️
opensourceleg/sensors/hall.py 98.4% 0 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

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.

3 participants