-
Notifications
You must be signed in to change notification settings - Fork 298
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
Update ros2_control_node.cpp #961
Conversation
Had similar issues to what's described in #859. There could be a better way to solve this, but this works on both simulated and real machines. |
Dear @ObeidHaidar could you elaborate a bit on the test you did in simulation ? |
It looks like the ROS way to do what you want is to follow https://github.com/ros2/rclcpp/blob/399f4df7396d67b42ccaea651bbd87d66b0d62b3/rclcpp/test/rclcpp/test_time_source.cpp#L284 |
@ObeidHaidar which kind of simulation are you using? This not shouldn't be used in simulation at all. The simulation is setting time there not the node. |
ping @ObeidHaidar can you refer to the qustions? |
Will keep applying the fix in my own branch. |
No description provided.