Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Laser SLAM crashing #28

Open
vkee opened this issue Jul 28, 2017 · 3 comments
Open

Laser SLAM crashing #28

vkee opened this issue Jul 28, 2017 · 3 comments

Comments

@vkee
Copy link

vkee commented Jul 28, 2017

When running on the KITTI datasets, the laser_mapper node crashes and respawns every so often, restarting mapping and state estimation. It seems to be linked to exceptions or errors being related to ICP convergence issues.

I think performance could be greatly increased by catching the exceptions and then relying on odometry rather than letting the node die and then respawning it.

@vkee vkee changed the title Instability with poor ICP convergence System Instability with Poor ICP Convergence Jul 28, 2017
@vkee vkee changed the title System Instability with Poor ICP Convergence Laser SLAM crashing Jul 28, 2017
@rdube
Copy link
Contributor

rdube commented Jul 28, 2017

Thanks for reporting this! Could you please copy and paste the error log?

@vkee
Copy link
Author

vkee commented Jul 28, 2017

I've gotten a variety of them - here's the last one that I got.

I0728 14:00:16.570504   506 incremental_estimator.cpp:104] Took 2.078 ms to create loop closures sub maps.
I0728 14:00:16.570518   506 incremental_estimator.cpp:106] Creating loop closure ICP.
terminate called after throwing an instance of 'PointMatcher<float>::ConvergenceError'
  what():  no outlier to filter

@rdube
Copy link
Contributor

rdube commented Jul 28, 2017

Ok that occurs when doing an ICP step after a loop closure is detected. In our latest development we turned this option off by setting the following parameter to false:

https://github.com/ethz-asl/segmatch/blob/master/laser_mapper/launch/kitti/kitti_loop_closure.yaml#L108

In general, relying on the segment centroids already provide a satisfying transformation. Could you maybe try turning this parameter to false?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants