-
Notifications
You must be signed in to change notification settings - Fork 348
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
Odom Topic & Frame Namespaces #461
Odom Topic & Frame Namespaces #461
Conversation
378fd80
to
820f38b
Compare
I'd really prefer to merge this into To fix most of the formatting issues, you can run |
This pull request is in conflict. Could you fix it @sp-sophia-labs? |
…s#308) (ros-controls#359) Co-authored-by: John Morris <john@zultron.com> Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
"~/odom" -> "odom" final namespacing will look like: /tf /robot_0/odom /robot_0/diff_drive_controller/cmd_vel
b0d147b
to
86a08cc
Compare
@bmagyar No problem for me, I've rebased the changes on master and fixed the linting with |
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.
Assign namespace prefix directly to odom_params to simplify code
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.
Assign namespace prefix directly to odom_params to simplify code
Codecov Report
@@ Coverage Diff @@
## master #461 +/- ##
==========================================
- Coverage 35.78% 29.74% -6.04%
==========================================
Files 189 7 -182
Lines 17570 743 -16827
Branches 11592 428 -11164
==========================================
- Hits 6287 221 -6066
+ Misses 994 162 -832
+ Partials 10289 360 -9929
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Description
The goal of this PR is to enable namespaces on diff_drive_controller for the odom topic and the odom frame
How has this been tested?
We have tested this code on create3 and turtlebot4 robots. You can find our project here: https://github.com/sp-sophia-labs/turtlebot4_multi_sim. Reviewers may clone the project and replicate exactly what we use for testing (you can find the details in the project's README)
Checklist