-
Notifications
You must be signed in to change notification settings - Fork 193
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
add a bridge mapping rule so tf/tfMessage maps to tf2_msgs/TFMessage #48
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.
Seems good to me. Nice to know about the mapping rules :).
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.
thanks for fixing this! that was a known issue we didnt find time to address and test during beta3.
I was originally concerned that the custom mapping rule would override the mapping of tf2/TFMessage but looking at the code it's just adding custom mappings to the default ones 👍
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.
added a comment in the mapping file (not necessary, but can't hurt), hope you don't mind
lgtm, thanks @dhood. |
@wjwwood should this be merged ? |
Yeah, thanks for the reminder. |
I did this according to this doc:
https://github.com/ros2/ros1_bridge/blob/master/doc/index.rst#how-can-i-install-mapping-rule-files
And some help from @dirk-thomas.
I tested this with our HSR demo.