search

PRBonn / kiss-icp

A LiDAR odometry pipeline that just works
https://www.ipb.uni-bonn.de/wp-content/papercite-data/pdf/vizzo2023ral.pdf
MIT License
1.6k stars 321 forks source link

Fix ATE #324

Closed benemer closed 7 months ago

benemer commented 7 months ago

The ATE computation was wrong, now fixed

benemer commented 7 months ago

@nachovizzo can this be merged?

We double checked the ATE values with evo_ape and they match now.

nachovizzo commented 7 months ago

I don't really care about this particular case but we might try to be more verbose than :

The ATE computation was wrong, now fixed

If you come back in 5 months and try to understand what you changed and why you will have troubles. But don't just change it now