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

Performance issues with smallgicp, while the old fast-gicp version does very well with the same data. PCD is too small warning. #89

Open
VTrencsenyi opened this issue Sep 2, 2024 · 4 comments
Labels
bug Something isn't working

Comments

@VTrencsenyi
Copy link

The fast-gicp version of GICP works pretty well with my python pipeline and data. However, with small-gicp I keep getting the warning "target/source pcd is too small" and the resulting accuracy is very poor. Do you have any intuition where the difference may be coming from? Note, this is the case regardless of downsampling.

@VTrencsenyi VTrencsenyi added the bug Something isn't working label Sep 2, 2024
@koide3
Copy link
Owner

koide3 commented Sep 3, 2024

The warning on the point cloud size is shown when the number of points is less or equals to 10, that indicates something strange is happening on your input point clouds. Can you provide minimum example code and data to reproduce the problem?

if (traits::size(target) <= 10) {

@junwei1999
Copy link

Hi, Koide3. When I tried to run the code in the 01_basic_registration.cpp from the example, this problem occurred. However, I didn't modify any code.
Before the program runs to the align function, the sizes of the target and source point clouds are normal.
But during the execution of the align function, the following warnings are generated:
warning: Empty point cloud
warning: Empty point cloud
warning: target point cloud is too small. |target|=0
warning: source point cloud is too small. |source|=0

Setting the downsampling_resolution in the setting to 0.0 also generates the same warnings.

@koide3
Copy link
Owner

koide3 commented Feb 10, 2025

@junwei1999
Can you print out the number of points just before running example1?

@junwei1999
Copy link

junwei1999 commented Feb 10, 2025 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants