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

In which unit does the complementary/madgwick filter need the acc-data for proper working? #206

Open
ChrisWoll opened this issue Aug 27, 2024 · 1 comment

Comments

@ChrisWoll
Copy link

I think it's not 100 % clear in which unit the algorithm needs the acc date:

In the Complementary Filter paper, it says normalized, aswell in the comment in the header-file for the update() method.
But if I look in the implementation, for example the checkstate() or getAdaptiveGain() function, it compares the magnitude of acc with the gravity, which would be completly nonsense if the input is a normalized acc-vector. And then I ve seen in a YouTube Video that it wants in mg.

@mintar
Copy link
Collaborator

mintar commented Aug 28, 2024

The authoritative answer is in the comments of the sensor_msgs/Imu message:

Accelerations should be in m/s^2 (not in g's), and rotational velocity should be in rad/sec

In general, all messages in ROS should be in SI units.

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

No branches or pull requests

2 participants