eskf-gps-imu-fusion icon indicating copy to clipboard operation
eskf-gps-imu-fusion copied to clipboard

fix some formula error

Open LeatherWang opened this issue 3 years ago • 1 comments

Modifications

  • adjust the coordinate system of ins/gps measurement
  • add gyro/accel bias compensation into prediction equation
  • fix error observation equation and jacobian

Results

fix_bug

LeatherWang avatar Jan 17 '22 15:01 LeatherWang

@LeatherWang 请问这个mr merge之后,对于输入数据的坐标系是否有要求。例如: accel-0.csv: accel_x (m/s^2),accel_y (m/s^2),accel_z (m/s^2), 是NED还是ENU? gps-0.csv: gps_lat (deg),gps_lon (deg),gps_alt (m),gps_vN (m/s),gps_vE (m/s),gps_vD (m/s),是否依然可以?注意,此处速度为NED gyro-0.csv:gyro_x (deg/s),gyro_y (deg/s),gyro_z (deg/s):相对于IMU body frame的坐标系,or ENU?

shockjiang avatar Oct 30 '22 14:10 shockjiang

@LeatherWang 在理解这个branch的时候遇到了一些困难,不知是否方便请教一下?

shockjiang avatar Nov 16 '22 09:11 shockjiang

@LeatherWang 请问这个mr merge之后,对于输入数据的坐标系是否有要求。例如: accel-0.csv: accel_x (m/s^2),accel_y (m/s^2),accel_z (m/s^2), 是NED还是ENU? gps-0.csv: gps_lat (deg),gps_lon (deg),gps_alt (m),gps_vN (m/s),gps_vE (m/s),gps_vD (m/s),是否依然可以?注意,此处速度为NED gyro-0.csv:gyro_x (deg/s),gyro_y (deg/s),gyro_z (deg/s):相对于IMU body frame的坐标系,or ENU?

根据改动,accel-0 应该是 ENU, 因为imu_flow.cpp 中的TransformCoordinate都被删了。 gyro-0与accel-0一致。 gps-0保持NED, 在tool.h 中对其进行了转换 (注意Eigen::AngleAxisd复合运算时,旋转轴始终是初始的xyz,因此去掉了.inverse()

NicolasHHH avatar May 03 '23 21:05 NicolasHHH

Thank you for your contribution to this project. I have further refactored the entire algorithm according to your ideas, welcome to review. @LeatherWang

zm0612 avatar Aug 16 '23 15:08 zm0612