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

sometimes not trigger loop under the same params and dataset #38

Closed
narutojxl opened this issue Dec 13, 2019 · 3 comments
Closed

sometimes not trigger loop under the same params and dataset #38

narutojxl opened this issue Dec 13, 2019 · 3 comments

Comments

@narutojxl
Copy link

Hi @ziv-lin , i use rosbag_loop.launch and don't modify any param in performance_precision.yaml , play with loop_hku_zym.bag. I run the demo 5 times, and i found sometimes it doesn't trigger loop, but sometimes it can. Did you meet this ? Here is This is the result of a experiment not loop.
image

@ziv-lin
Copy link
Member

ziv-lin commented Dec 13, 2019

This is because I add some randomness during the alignment of the map. For realtime consideration, we randomly drop some of the matchings during ICP of map alignment, this might cause the performance not the alway the same of each launching.
Once the maps can not alignment well, we treat the loop detect by 2D-histogram as false positive.

@ziv-lin
Copy link
Member

ziv-lin commented Dec 13, 2019

For disable such randomness, you can modify the value in performance_precision.yaml
loop_closure/scene_alignment_maximum_residual_block: 3000
from 3000 to an extrmely high value (like 999999).

In addition, you can also fine-tune the value
loop_closure/map_alignment_inlier_threshold: 0.2
to a larger value to accept the loop.

@narutojxl
Copy link
Author

@ziv-lin, thanks a lot :)

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