-
Notifications
You must be signed in to change notification settings - Fork 771
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
[ros2] Add noise to imu test #801
Conversation
@chapulina I didn't understand what the CI was supposed to show. Is there maybe a way to get it to test gazebo_ros_pkgs somehow? (doesn't look like it did in those builds) |
Sorry, I don't know why I expected the links to
|
OK no worries. Confirming that the test used to output errors when not using this change. CI from when https://github.com/ros-simulation/gazebo_ros_pkgs/pull/793/files#diff-d966e67ca1b4db645730d08da8e16054 was merged: http://build.ros2.org/job/Bpr__gazebo_ros_pkgs__ubuntu_bionic_amd64/31/consoleFull
However, I'd imagine this will make the test flaky because it was written assuming no noise. I'll relax the tolerance |
I simplified the noise and relaxed the tolerance, so this passes when repeated now. (e.g. with |
Thanks for the tweaks, @dhood ! |
See #799
It turns out that the noise is being loaded correctly, it was a local issue
Let's double-check when CI is done: