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

README gives the false impression that conversion from Pointcloud2 to grid_map layers are available. #457

Open
cemonem opened this issue May 17, 2024 · 2 comments

Comments

@cemonem
Copy link

cemonem commented May 17, 2024

Hello,

The Readme gives the false impression that there are seamless methods of conversion available for using pointcloud messages. This issue #158 suggests that is not the case, there are some workarounds with converting to octomap then grid_map or heavy lifting needs to be done by porting elevation mapping code from ROS1 to ROS2. Please update the README.

Thanks.

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Jul 11, 2024

Can you supply an example point cloud you'd like to convert?

There exists functions to convert from PCL types here: https://github.com/ANYbotics/grid_map/blob/master/grid_map_pcl/include/grid_map_pcl/GridMapPclConverter.hpp#L51

@ysozkaya
Copy link

I think the question is clear enough as it is stated in the issue. The pointcloud is from the official sensor_msgs called PointCloud2. The grid map should be usable in real-time with data from a PointCloud2 topic.

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

No branches or pull requests

3 participants