ethzasl_msf icon indicating copy to clipboard operation
ethzasl_msf copied to clipboard

Allow initializing with full quaternion.

Open rikba opened this issue 4 years ago • 2 comments

This PR exposes initialization using the full orientation. This becomes useful when starting MSF with an IMU that is not perfectly aligned with the inertial frame. May solve https://github.com/ethz-asl/ethzasl_msf/issues/169 and others.

Remarks:

  • This PR does not solve the issue that MSF is initialized from parameter server instead of the first message arriving which would be more convenient in my opinion. In our case we wrote an extra node that reads the first IMU orientation and GPS position from ROS messages and then writes it to dynamic reconfigure. I can share this node if desired.
  • This PR removes initialization from yaw only. If there is a neat or preferred way to preserve backwards compatibility we can implement that. Let me know.

rikba avatar May 11 '20 07:05 rikba

Can one of the admins verify this patch?

ethzasl-jenkins avatar May 11 '20 07:05 ethzasl-jenkins

Looks good. Would maybe make sense to change it for the other sensors as well (PositionPoseSensor and SphericalPositionSensor), to stay consistent

clanegge avatar May 12 '20 07:05 clanegge