RobustFieldAutonomyLab / LeGO-LOAM

LeGO-LOAM: Lightweight and Ground-Optimized Lidar Odometry and Mapping on Variable Terrain
BSD 3-Clause "New" or "Revised" License
2.35k stars 1.11k forks source link

question about KDTree #155

Closed SakuraMemoryKnight closed 4 years ago

SakuraMemoryKnight commented 4 years ago

When I run the official dataset downloaded from the home page url(https://drive.google.com/drive/folders/1dIsonWdZNBGiUiGh2H6dWJKoJ1k8_A2j?usp=sharing), I get this information: /////////////// [pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud! ////////////// What should I do?

TixiaoShan commented 4 years ago

It means the input point cloud to kd-tree has zero points. Every scan in the sample dataset has enough features. So it's probably there is a bug in your modified code.

SakuraMemoryKnight commented 4 years ago

Thanks for your @TixiaoShan reply! As you said, I modified the parameters of the lidar(utlity.h), and when I change back to the default,the official dataset can work normally.

In addition, I find that there are almost eight times as many imu frames as there are radar frames,this means that in the official data set the imu data frequency is very high, but in kitti dataset the imu has the same frequency as the radar. I think this is the main reason for the poor performance of the kitti dataset. what do you think?

TixiaoShan commented 4 years ago

@zcg3648806 The higher the IMU frequency the better. In fact, the IMU used in my dataset is very low-end and gives only 75Hz. It's recommended to use an IMU that is at least 200Hz, like the Xsens IMU used in Ji Zhang's paper.

SakuraMemoryKnight commented 4 years ago

Thanks for your @TixiaoShan reply! I got it.