evo icon indicating copy to clipboard operation
evo copied to clipboard

Unwarp angles before plotting them.

Open ValerioMagnago opened this issue 2 years ago • 1 comments
trafficstars

Jumps in the angle plot make hard to compare different trajectory. In this commit we unwrap the angles before plotting them.

Before: hard_to_read

After: easier_to_read

ValerioMagnago avatar Nov 06 '23 15:11 ValerioMagnago

Please don't change the default and make this an option (for the function and in the CLI or settings). I would only see this as meaningful when dealing with pure 2d rotation, but also there someone might want to see the absolute values instead.

Also note that for the purpose of rotation comparison, the rotation error metrics in RPE/APE are generally more stable than anything that is based on Euler angles.

MichaelGrupp avatar Nov 14 '23 14:11 MichaelGrupp