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

Remove black points in the generated colored point cloud #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

hany606
Copy link

@hany606 hany606 commented Sep 24, 2021

This code will generate black points if their RGB values are zero as they are only registered in the "cloud" PointCloud, but isn't the intention of the following snippet: (lines 221-223)

                if (RGB[0] == 0 && RGB[1] == 0 && RGB[2] == 0) {  
                    continue;
                }

to exclude the points in the point cloud that has no colour correspondence in order not to visualize it? If my assumption is correct about the real intention, then just setting the xyz coordinates after the condition will fix that small bug.

Thank you very much for your great package, it is really cool and works pretty good, but had some hard time with detecting the corners in the point cloud.

This code will generate black points if their RGB values are zero as they are only registered in the "cloud" PointCloud, but isn't the intention of the following snippet: ([lines 221-223](https://github.com/Livox-SDK/livox_camera_lidar_calibration/blob/27f9e6c8a1eda3326b7673da418e6d7112559c2f/src/color_lidar_display.cpp#L221-L223))
```
                if (RGB[0] == 0 && RGB[1] == 0 && RGB[2] == 0) {  
                    continue;
                }
```
to exclude the points in the point cloud that has no colour correspondence in order not to visualize it? If my assumption is correct about the real intention, then just setting the xyz coordinates after the condition will fix that small bug.

Thank you very much for your great package, it is really cool and works pretty good, but had some hard time with detecting the corners in the point cloud.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant