lgmath icon indicating copy to clipboard operation
lgmath copied to clipboard

Rotation matrices not required to be orthonormal

Open ben-congram opened this issue 3 years ago • 0 comments

  • The reproject function called in lgmath::se3::Transformation's constructor (and other spots) is conditioned on det(C) = 1 but this is not a sufficient check to determine if C is in SO(3)
  • Therefore, users could create a Transformation with C_ab_ = diag(2,0.5,1), for example
  • Even when constructed properly, round-off error can stack up in rare cases if matrix is never forcibly reprojected
  • May want to add check that C^T * C = 1, if computationally feasible

ben-congram avatar May 06 '21 21:05 ben-congram