-
Notifications
You must be signed in to change notification settings - Fork 276
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
Odometry publisher: also publish Pose_V (TF) #1534
Conversation
Signed-off-by: Louise Poubel <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
Looks like the test may be timing out in CI: https://build.osrfoundation.org/job/ignition_gazebo-ci-pr_any-ubuntu_auto-amd64/8879/testReport/junit/(root)/ServerRepeat_OdometryPublisherTest/Movement3d_0/ |
Signed-off-by: Louise Poubel <[email protected]>
Ouch I changed the implementation halfway but forgot to update the test. It should be fixed now. |
Codecov Report
@@ Coverage Diff @@
## ign-gazebo6 #1534 +/- ##
===============================================
+ Coverage 61.52% 64.04% +2.51%
===============================================
Files 361 316 -45
Lines 27733 25478 -2255
===============================================
- Hits 17063 16317 -746
+ Misses 10670 9161 -1509 Continue to review full report at Codecov.
|
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.
Works for me!
🎉 New feature
Summary
The
OdometryPublisher
plugin already publishesmsgs::Odometry
, but it's also useful for it to publishmsgs::Pose_V
, which can be bridged to the/tf
topic in ROS.I used
/mode/<name>/pose
as the default so that it would match thePosePublisher
. But note that this is different from theDiffDrive
's default, which is/model/<name>/tf
.Test it
/mode/<name>/pose
topic to/tf
, and check the transforms with ROSChecklist
Updated migration guide (as needed)Consider updating Python bindings (if the library has them)codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸