-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Optimal configuration for VLP16 #259
Comments
I have collected another dataset which, I believe, is more closer to what cartographer ideally expects. However, couldn't achieve loop closure yet. New bag file contains two loops and each lidar message contains only one velodyne UDP package. So, message frame rate increased to ~750 hz from 10 hz. Configuration file is still same, except that scans_per_accumulation parameter is changed from 1 to 75. |
@ozgudum Sorry for the late reply. I am quite back logged on issues these days. I only looked at Some assorted notes:
My parameters give good results, but loop closure is not real-time with these aggressive settings. This is not surprising: I arbitrarily picked really big values for the search space for loop closure. I did not spend time making the parameters faster again - I think you can tune them now to your needs. |
hi, @ozgudum Do you have any idea about the error? I really need your help. Thanks a lot! |
hi @zerox3802567, unfortunately it is been quite a long time since last time I worked on Cartographer and I don't have an idea about the error. Maybe @SirVer may give you some hints? |
posting screenshots of text is not a great way of sharing error messages. Please copy & past and put them into a gist. @zerox3802567 Your problem is that your configuration is not up to modern requirements of cartographer. It says in the first few lines: |
这是我运行检查数据的语句cartographer_rosbag_validate -bag_filename /media/autodrive/LidarDisk/testdata/data/loop.bag 这是我运行程序的语句roslaunch cartographer_ros demo_vlp16-imu.launch bag_filename:=/media/autodrive/LidarDisk/testdata/data/loop.bag started roslaunch server http://navinfo-drive:36831/ SUMMARYPARAMETERS
NODES auto-starting new master setting /run_id to e7601f96-4c48-11e9-b5f1-9c5c8e75471b
|
I have spent considerable amount of time on tuning the configuration parameters to get a good registration and loop closure performance without any luck. I was able to get a reasonable result in one set but when I tried to use the same config for another, it didn't work well. Having too many parameters makes it hard to see the individual effects of each of parameter.
I have a similar hardware configuration (VLP16) with the one mentioned in this post. Changes and comments in this (SirVer@72c4519) commit helped a lot to understand the effects but I couldn't achieve loop closure in this particular bag file. Here is my current configuration. Reverted them back to original because previous set of tuned parameters messed up in this bag file.
This is what I currently get:
Obviously, the overlap is small and we can only get the loop closure at the very end. I am also wondering which parameters would possibly reduce the drift in the trajectory. Otherwise we would need to set a large search_window to get loop closure.
Explanations for the config parameters here and here were helpful but I wish they would be complete.
The text was updated successfully, but these errors were encountered: