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

Add smart downsampling to better support sparse LiDARs #240

Open
nachovizzo opened this issue Oct 12, 2023 · 0 comments
Open

Add smart downsampling to better support sparse LiDARs #240

nachovizzo opened this issue Oct 12, 2023 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@nachovizzo
Copy link
Collaborator

Problem description

Some LiDARs, mainly 16-beam, 32-beam, and Livox-like sensors, produce relatively sparse point clouds compared to Velodyne-64/128, Ouster-64/128. Initially, KISS-ICP was mainly tested in 64/32 beam LiDARs, and doing a "double-downsample" made sense to speed up the registration loop.

But, as more and more people use the system, more LiDARs are being tested, which has become more of a problem. I've seen the pipeline fail because of this many times myself. In my case, it's easier to identify because I always inspect the "keypoint" we are using for registration, and when tested with such sparse LiDARs, the expected results are clear to me: it will fail. So I typically hack the pipeline and remove the downsample (as specified in #128 (comment) and #239 (comment) )

Hotfixes

Possible solutions (please comment if you have ideas!)

  • Add a boolean flag in the configuration to specify that KISS is leading with a "sparse" lidar. This flag should also be exposed through the CLI so people can try kiss_icp_pipline <dataloader> --sparse-lidar
  • Add some more intelligent way of downsampling, trying to reason from the registration point of view. This would be the most "scientific" approach, but It might be harder to generalize this idea
  • Ideas from the community?

Related issues

@nachovizzo nachovizzo added enhancement New feature or request help wanted Extra attention is needed labels Oct 12, 2023
@nachovizzo nachovizzo pinned this issue Oct 12, 2023
@nachovizzo nachovizzo mentioned this issue Nov 9, 2023
@nachovizzo nachovizzo unpinned this issue Apr 2, 2024
@nachovizzo nachovizzo pinned this issue Apr 2, 2024
@tizianoGuadagnino tizianoGuadagnino unpinned this issue Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant