We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
According to ROS docs, the IMU messages should use m/s² units (and not g's) for acceleration etc.:
http://docs.ros.org/en/noetic/api/sensor_msgs/html/msg/Imu.html
In the 'fake_imu' code, I don't see any earth gravity constant (1g = 9.8 m/s²) in the acceleration computation:
mcl_3dl/src/mcl_3dl.cpp
Line 241 in 1ebf897
Maybe 'mcl_3dl' is using wrong units in the expected IMU messages (e.g. IMU callbacks)? :-)
Line 942 in 1ebf897
Simulating ROS IMU messages should probably look something like this (according to ROS specifications)? :-) imu->linear_acceleration.x = accel.x_ * 9.8; imu->linear_acceleration.y = accel.y_ * 9.8; imu->linear_acceleration.z = accel.z_ * 9.8;
The text was updated successfully, but these errors were encountered:
mcl_3dl node only uses vector direction of the linear acceleration, so absolute length of the vector doesn't matter.
Sorry, something went wrong.
Thanks for clarification!
No branches or pull requests
According to ROS docs, the IMU messages should use m/s² units (and not g's) for acceleration etc.:
http://docs.ros.org/en/noetic/api/sensor_msgs/html/msg/Imu.html
In the 'fake_imu' code, I don't see any earth gravity constant (1g = 9.8 m/s²) in the acceleration computation:
mcl_3dl/src/mcl_3dl.cpp
Line 241 in 1ebf897
Maybe 'mcl_3dl' is using wrong units in the expected IMU messages (e.g. IMU callbacks)? :-)
mcl_3dl/src/mcl_3dl.cpp
Line 942 in 1ebf897
Simulating ROS IMU messages should probably look something like this (according to ROS specifications)? :-)
imu->linear_acceleration.x = accel.x_ * 9.8;
imu->linear_acceleration.y = accel.y_ * 9.8;
imu->linear_acceleration.z = accel.z_ * 9.8;
The text was updated successfully, but these errors were encountered: