-
Notifications
You must be signed in to change notification settings - Fork 287
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
Bridge appears incapable of bridging ROS1 /clock to ROS2 #104
Comments
Could this perhaps be solved by specifying a custom mapping rule? |
Where could I put the remapping aside from having to modify a core ros package? It appears it would need to be added to the builtin_interfaces package for the bridge to accept it otherwise I get an error similar to this one: EDIT: |
As there is no equivalent type for ROS1 |
The problem is not related to the differences in the The bridge simply can't bridge a message definition which is not available on both side. And |
I believe this issue still persists. I'm getting the same error than @matt-attack:
I've also tried the manual mapping but it seems to be ignored. Printing the mapping shows no instance of the required mapping. Also the mapping needs to be done in the 'builtin_interfaces' package. |
This is not a bug on the @tfoote Can you please follow up on this since you added the clock topic to |
I created ros2/rclcpp#473 to track this. |
Thank you @dirk-thomas . While this is being worked, is there a temporary solution maybe creating a mapping for this topic? |
Yes, you can temporarily work around it by defining a remapping rule. |
In ROS1 the rosgraph_msgs/Clock message consists of a time type internally, but in ROS2 the /clock topic is a builtin_interfaces/Time. It does not appear that the bridge can handle bridging this topic from ROS1 to ROS2.
Is there a way to force the bridge to handle this kind of conversion or should a special case be added to the bridge to allow for sim time and bag file playback?
The text was updated successfully, but these errors were encountered: