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

loop closure performance #40

Open
greymfm opened this issue Jul 12, 2021 · 3 comments
Open

loop closure performance #40

greymfm opened this issue Jul 12, 2021 · 3 comments

Comments

@greymfm
Copy link

greymfm commented Jul 12, 2021

Hello,
I have evaluated different SLAM software with the Livox Horizon LiDAR (cartographer, hdl_graph_slam, livox_mapping, etc.) and it seems 'lili-om' is the most robust SLAM for such a LiDAR - congratulations!

I tried your code (using 'run_fr_iosb_internal_imu.launch') with some larger recording (Livox Horizon with internal IMU). It works except the loop closure doesn't work at the end of the recording.

Screenshot from 2021-07-12 21-23-16

Screenshot from 2021-07-12 21-25-21

I have uploaded the bag file here (recorded using 'livox_lidar_msg.launch' from 'livox_ros_driver'):
https://drive.google.com/file/d/1NheqSfHnh1E00umY0no6I3AqigL7WDFK/view?usp=sharing

Maybe there is something I can try to tune the loop closures? :-)

PS: I plan to add cm-precise RTK-GPS factors (already available as LiDAR time-synchronized NavSatFix messages in the above recording) to the global graph pose optimization to make 'lili-om' even more robust ;-)

Thanks,
Alexander

@swjtuyang
Copy link

any update?

PS: I plan to add cm-precise RTK-GPS factors (already available as LiDAR time-synchronized NavSatFix messages in the above recording) to the global graph pose optimization to make 'lili-om' even more robust ;-)

@kailaili
Copy link
Contributor

Hi, you may change the parameters related to loop closure in the yaml file, e.g., raise the lc_search_radius. But yeah, adding additional external factors would be very good to achieve guaranteed robustness as sometimes the drift could be too large for closing the loop. I'm looking forward to seeing your further development. :)

@schoeller
Copy link

PS: I plan to add cm-precise RTK-GPS factors (already available as LiDAR time-synchronized NavSatFix messages in the above recording) to the global graph pose optimization to make 'lili-om' even more robust ;-)

@greymfm
Is there any news on this?

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

4 participants