-
Notifications
You must be signed in to change notification settings - Fork 28
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
First version of the package in ROS Noetic #25
First version of the package in ROS Noetic #25
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did a first pass and it LGTM Flor. I left a couple of comments for nits only. Could you also look for other missing references to foxy-devel
and ROS2
on your branch before merging?
massrobotics_amr_sender/sample/massrobotics_amr_sender_rosbag_launch.launch
Show resolved
Hide resolved
Done! Thanks Lean for checking that, I've updated all the references to ROS1. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is awesome @FlorGrosso ! I would go ahead and merge. We can then test it and fix as we find problems.
If you want to be extra nice, you can create issues for the remaining items (tests and publishing the package)
Thanks. @adamantivm ! I've created #26 and #27. |
Adds a version of the package in ROS 1 Noetic, Python 3. It includes:
massrobotics_amr_sender
package which can be installed from source.TODOs: