Skip to content

fix(robots): add reachy2 fixes #2783

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
imstevenpmwork merged 3 commits into from
Jan 12, 2026
Merged

fix(robots): add reachy2 fixes #2783

imstevenpmwork merged 3 commits into from
Jan 12, 2026

Conversation

@imstevenpmwork
Copy link
Collaborator

@imstevenpmwork imstevenpmwork commented Jan 12, 2026 •
edited
Loading

Type / Scope

  • Type: Bug & Feature
  • Scope: robots

Summary / Motivation

  • Several improvements & bug fixes for reachy2 implementation

Related issues

What changed

  • All details mentioned in the PRs above

How was this tested

  • pytest -vv tests/ with both reachy2-sdk installed and without.

Checklist (required before merge)

  • Linting/formatting run (pre-commit run -a)
  • All tests pass locally (pytest)
  • Documentation updated
  • CI is green

Reviewer notes

  • Bump of reach2-sdk dependency is needed because the latest one broke the path to CameraView class

@imstevenpmwork imstevenpmwork marked this pull request as ready for review January 12, 2026 16:11
@imstevenpmwork imstevenpmwork self-assigned this Jan 12, 2026
@github-actions github-actions bot added documentation Improvements or fixes to the project’s docs tests Problems with test coverage, failures, or improvements to testing robots Issues concerning robots HW interfaces sensors Everything related to sensors labels Jan 12, 2026
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Jan 12, 2026

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

This was referenced Jan 12, 2026
Closed
Closed
Merged
CarolinePascal
CarolinePascal approved these changes Jan 12, 2026
View reviewed changes
Copy link
Collaborator

@CarolinePascal CarolinePascal left a comment

Choose a reason for hiding this comment

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

LGTM !

@imstevenpmwork imstevenpmwork merged commit b8ec115 into main Jan 12, 2026
13 checks passed
@imstevenpmwork imstevenpmwork deleted the fix/reachy2_cam_imports branch January 12, 2026 17:05
sandhya-cb pushed a commit to sandhya-cb/lerobot-clutterbot that referenced this pull request Jan 28, 2026
@imstevenpmwork @sandhya-cb
fix(robots): add reachy2 fixes (huggingface#2783)
1bf9b6f 
* fix(robots): add reachy2 fixes

* tests(robots): remove reachy sdk stub
mengxyokok pushed a commit to mengxyokok/lerobot that referenced this pull request Jan 30, 2026
@imstevenpmwork @mengxyokok
fix(robots): add reachy2 fixes (huggingface#2783)
6bed47a 
* fix(robots): add reachy2 fixes

* tests(robots): remove reachy sdk stub
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CarolinePascal CarolinePascal CarolinePascal approved these changes

Assignees

@imstevenpmwork imstevenpmwork

Labels

documentation Improvements or fixes to the project’s docs robots Issues concerning robots HW interfaces sensors Everything related to sensors tests Problems with test coverage, failures, or improvements to testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@imstevenpmwork @HuggingFaceDocBuilderDev @CarolinePascal