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

Missing covariances in imu topic #51

Open
Timple opened this issue Feb 24, 2022 · 2 comments
Open

Missing covariances in imu topic #51

Timple opened this issue Feb 24, 2022 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@Timple
Copy link

Timple commented Feb 24, 2022

Is your feature request related to a problem? Please describe.
Thank you for picking up issue #16. The imu message is now usable within the ROS system. For sensor fusion however, the covariance fields needs to be filled.

Describe the solution you'd like
Preferably fill the covarience fields with the actual uncertainties.
If these are not known, fill them with the squared value of the stdev from the specsheet. (0.0044 for angular and 0.04 for linear acceleration in the novatel case i think).

Describe alternatives you've considered
We now have an 'fix' node inbetween which inserts these values, but this should be fixed at the source.

Additional context
We can offer a pull-request if you like.

@novatel-applications-engineering
Copy link
Contributor

Hi @Timple, thank you for the feedback! I believe this has also been reported as issue #23. We do have this in our queue of items to address and will note there is increased demand for a fix.

@Timple
Copy link
Author

Timple commented Feb 25, 2022

Ah yes, I missed #23.

Would you mind keeping issues open as long as they are issues?

Otherwise one never knows when something is fixed and where to look for open issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants