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

Extend PclConverter for loading a color layer from a textured mesh #374

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

Conversation

Jaeyoung-Lim
Copy link

@Jaeyoung-Lim Jaeyoung-Lim commented Sep 12, 2022

Problem Description
When loading a gridmap from a polygon mesh, only the geometry was being read on the grid map.

This commit extends the gird map pcl coverter for loading a color layer from a textured mesh

This is useful when loading colored pointclouds or reconstructions on the grid map for visualization purposes

Testing
A mesh can be read and added with a color layer with the two sequences

  grid_map::GridMapPclConverter::addLayerFromPolygonMesh(mesh, "elevation", grid_map_);
  grid_map::GridMapPclConverter::addColorLayerFromPolygonMesh(mesh, "color", grid_map_);

gridmap-mesh

This commit extends the gird map pcl coverter for loading a color layer from a textured mesh
@Jaeyoung-Lim
Copy link
Author

image

@maximilianwulf The CI Build failure doesn't seem to be related to this PR?

@Jaeyoung-Lim Jaeyoung-Lim changed the title Add method for loading a color layer from a textured mesh Extend PclConverter for loading a color layer from a textured mesh Sep 12, 2022
@Ryanf55 Ryanf55 added the ros1 Affects ROS 1 label Feb 12, 2024
@Jaeyoung-Lim
Copy link
Author

@Ryanf55 Could we get this in?

@Ryanf55
Copy link
Collaborator

Ryanf55 commented Mar 24, 2024

@Ryanf55 Could we get this in?

Sorry, I only have merge authority for ROS2 branches. Let's kindly ask @maximilianwulf to take a look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ros1 Affects ROS 1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants