Skip to content

Cliff sensor foxy - #93

Closed
owen7900 wants to merge 4 commits into
AutonomyLab:foxyfrom
owen7900:cliff_sensor_foxy
Closed

owen7900 wants to merge 4 commits into
AutonomyLab:foxyfrom
owen7900:cliff_sensor_foxy

Conversation

@owen7900

Copy link
Copy Markdown
Contributor

I have implemented the cliff sensor here. I have tested it on a Create2 robot. Backporting to melodic would be annoying for me to do as I don't have an 18.04 install handy, but I could do it if required.

@owen7900

Copy link
Copy Markdown
Contributor Author

This fixes #22

@eborghi10

Copy link
Copy Markdown

@owen7900 you should modify the feature list of the main README file: https://github.com/AutonomyLab/create_robot/blob/melodic/README.md#features

@jacobperron

Copy link
Copy Markdown
Member

Cool, thanks! Changes look alright to me on the first pass. I'll try it out sometime this week on a robot 👍

jacobperron pushed a commit that referenced this pull request Apr 26, 2022
* Add custom Cliff message
* Publish cliff sensor data
* Update README
@jacobperron

Copy link
Copy Markdown
Member

Thanks for the contribution!

I squashed merged this manually on foxy due to a Git conflict (c8de6c8)

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