Skip to content
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.

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

Mode report workaround #67

Merged
merged 3 commits into from
Apr 7, 2022

Conversation

process1183
Copy link
Contributor

Addition of a simple flag to correct the off-by-one error for the mode present in the sensor stream of some 6xx Roomba models.

Ref: github.com/AutonomyLab/create_robot/issues/64

Copy link
Member

@jacobperron jacobperron left a comment

Choose a reason for hiding this comment

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

Thanks, looks reasonable to me.

Could you add a note about this workaround/API to the README? The Known Issues section seems like a good place.

@jacobperron
Copy link
Member

Also, feel free to add yourself to the contributors list :)

* Add note about 600 series OI mode reporting bug to Known Issues
  section and include details of API workaround option.
* Add myself to contributors list
@process1183
Copy link
Contributor Author

No problem! I've added details regarding the OI mode issue and workaround to the README.

@jacobperron jacobperron merged commit e99939c into AutonomyLab:master Apr 7, 2022
@jacobperron
Copy link
Member

Thanks for the contribution!

process1183 added a commit to process1183/roomba-rpi that referenced this pull request Apr 28, 2022
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