-
Notifications
You must be signed in to change notification settings - Fork 155
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
Comments
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 ;-) |
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. :) |
@greymfm |
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.
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
The text was updated successfully, but these errors were encountered: