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

rospc_to_o3dpc does not handle intensity #4

Open
stevencolinmartin opened this issue Mar 19, 2021 · 1 comment
Open

rospc_to_o3dpc does not handle intensity #4

stevencolinmartin opened this issue Mar 19, 2021 · 1 comment
Labels
enhancement New feature or request

Comments

@stevencolinmartin
Copy link

When importing ROS point clouds with 'intensity' fields it would be nice to have some way to pass this to open3d. As intensity is usually not accompanied by rgb it could be passed into these fields.

I would suggest something like:
r = b = g = intensity/255 for each point

Will likely create fork for own purposes but thought I discuss how best to implement. Should this have explicit flag in definition e.g.
def rospc_to_o3dpc(rospc, remove_nans=False, has_intensity=False):? If reading back to ros should it use average of r,g,b?

@SeungBack SeungBack added the enhancement New feature or request label Mar 19, 2021
@SeungBack
Copy link
Owner

Thank you for suggesting a feature to support more various ROS point clouds.
Maybe we can add a external flag "alpha" to "o3dpc_to_rospc", where alpha = 1 / intensity and "o3dpc_to_rospc" returns (r, g, b) * alpha. But I am not sure it is correct, so I will test it later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants