RobotWebTools / rclnodejs

Node.js version of ROS 2.0 client
https://docs.ros.org/en/humble/Concepts/Basic/About-Client-Libraries.html?highlight=rclnodejs#community-maintained
Apache License 2.0
311 stars 70 forks source link

Wrong topic type for /clock #927

Closed AntoninRousset closed 10 months ago

AntoninRousset commented 10 months ago

Description According to the ROS documentation, clock messages have type rosgraph_msgs/Clock, but rclnodejs uses builtin_interfaces/msg/Time.

Steps To Reproduce

node <your_rclnodejs_node>.js --ros-args -p use_sim_time:=true

Expected Behavior Subscibe to /clock with type rosgraph_msgs/msg/Clock.

Actual Behavior Subscribe to /clock with type builtin_interfaces/msg/Time.

minggangw commented 10 months ago

@AntoninRousset v0.22.3 rolls out with your patch, thanks for fixing this issue, close it now.