Skip to content
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

Add imu 2d rostest #267

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from

Conversation

efernandez
Copy link
Collaborator

@efernandez efernandez commented Apr 18, 2022

$ make run_tests_fuse_models_rostest_test_imu_2d.test
[ROSTEST]-----------------------------------------------------------------------

[fuse_models.rosunit-imu_2d_test/BaseFrameYawAngularVelocity][FAILURE]----------
/home/efernandez/dev/ws/cpr_ws/src/fuse/fuse_models/test/test_imu_2d.cpp:255
The difference between w and orientation_delta / dt is 0.96695215295635117, which exceeds 1.0e-2, where
w evaluates to 0.48358168580225275,
orientation_delta / dt evaluates to -0.48337046715409837, and
1.0e-2 evaluates to 0.01.
--------------------------------------------------------------------------------


[fuse_models.rosunit-imu_2d_test/BaseFrameYawAngularVelocity][FAILURE]----------
/home/efernandez/dev/ws/cpr_ws/src/fuse/fuse_models/test/test_imu_2d.cpp:306
The difference between w and delta[2] / dt is 0.96695215295635073, which exceeds 1.0e-2, where
w evaluates to 0.48358168580225275,
delta[2] / dt evaluates to -0.48337046715409793, and
1.0e-2 evaluates to 0.01.
--------------------------------------------------------------------------------


SUMMARY
 * RESULT: FAIL
 * TESTS: 1
 * ERRORS: 0
 * FAILURES: 1

Some tolerances in the test are a bit too large. I believe that's due to the fact I didn't synchronize the transactions generated by the Imu2D sensor model and the ground truth. However, I don't think that's really important, because the error is quite large with the current implementation.

I've fixed this in #268

This is on top of #266, which was required for the rostest to compile because a header was missing in the Imu2D sensor model header file.

@efernandez efernandez self-assigned this Apr 18, 2022
@efernandez efernandez added the bug Something isn't working label Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Development

Successfully merging this pull request may close these issues.

2 participants