jackal / jackal_robot

Robot packages for Jackal
63 stars 58 forks source link

Magnetometer data is missing the covariance #70

Open mllofriu opened 1 year ago

mllofriu commented 1 year ago

Please provide the following information:

To Reproduce Provide the steps to reproduce:

  1. run jackal bringup
  2. ros2 topic echo /imu/mag
tonybaltovski commented 1 year ago

@mllofriu The magnetometer didn't have covariance data from ROS 1 when it was only a Vector3 message. Do you have a specific use case for the data? We can look for the covariance for the sensor and add it to the next release of the firmware.

mllofriu commented 1 year ago

Some lidar SLAM systems require a 9-degree imu, which could be composed of the 6-degree imu and the magnetometer.

To be honest, I'm not sure how sensitive to the covariance any of these solutions (if at all) are:

All in all, I think this one is lower priority, unless you believe it affects the result of the /imu/data estimate (madgwick).