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

Publishing full TSDF/ESDF maps #80

Closed
simiken1234 opened this issue Oct 17, 2023 · 3 comments
Closed

Publishing full TSDF/ESDF maps #80

simiken1234 opened this issue Oct 17, 2023 · 3 comments
Assignees
Labels
enhancement New feature or request verify to close Waiting on confirm issue is resolved

Comments

@simiken1234
Copy link

Hello, are there any plans to publish the full esdf/tsdf maps on ROS topics as was the case with Voxblox? I really liked that implementation, also with the option to only publish updated blocks. With the currently published map slices it is hard to implement any type of 3D navigation on top of nvblox.

@alexmillane
Copy link
Collaborator

alexmillane commented Oct 21, 2023

Hi there.

This is a really good question and something that I've thought a bit about. You're correct that at the moment there really is no good way for downstream nodes to consume the 3D map.

Our friends at NVIDIA research who use nvblox for path-planning for robot arms (and who use nvblox for 3D mapping), take the approach of building nvblox, the ROS independent library, into their path-planner. But ideally, we'd like people to be able to consume the map in ROS.

Unfortunately, I think that with the resolution and rates nvblox is typically run at, the approach we took in voxblox of transmitting map deltas is not ideal. In nvblox we'd have to copy 3D voxels back to CPU, potentially serialize them, and copy them back to the GPU on the receiver(s) end. I think that for typical usage this would be too inefficient.

The correct approach, I think, is to do messaging using shared GPU memory. This is possible in ROS2, see here for example for the isaac_ros flavor.

There is a chance we'll do this in the future, but we'll have to see where the project goes.

@hemalshahNV hemalshahNV added verify to close Waiting on confirm issue is resolved enhancement New feature or request labels Oct 23, 2023
@simiken1234
Copy link
Author

I see. Seems like the shared GPU memory messaging would be a good approach then. Thanks for the advice, I will try to integrate nvblox directly into my project then.

For anyone who still wants a quick way to share the 3D map inefficiently, what I did was modify the pointcloud publishing function to publish a pointcloud of the full 3D map every time. On my end, I then stored the pointcloud in a hashmap which took a couple seconds for a pointcloud of about 0.5-1M points (working on CPU, no GPU accel) - usable for prototyping.

@simiken1234
Copy link
Author

I have to add that it would be nice to at least have a ros interface for querying individual voxel values. I know it wouldnt perform too well but it would be at least something

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

No branches or pull requests

3 participants