-
Notifications
You must be signed in to change notification settings - Fork 2
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
Steering controller time related test fixes #13
Steering controller time related test fixes #13
Conversation
Changed Times to be in RCL_ROS_TIME
…ntrollers into ackermann-odometry-library
Although changing the time objects to be in RCL_ROS_TIME doesn't fix the problem for the |
These changes are inspired by the diff_drive_controller tests on the main branch so I don't think these are band-aid fixes, I think they are resolving the actual problem. |
this is the output I'm getting now:
as opposed to all the time related issues in this ci (I did get those errors before in my build and I don't anymore) |
This is great, thank you!! |
b9bdf63
into
StoglRobotics-forks:ackermann-odometry-library
It seems there was a time comparison happening in the background and the two time objects were not on the same clock:
changed the clocks to be
RCL_ROS_TIME
and it resolved most of the problems except for thereceive_message_and_publish_updated_status
tests in each of the following 3 packages: