Skip to content

feat: add parameter to rename intensity field - #5

Open
m-sanchez-rico wants to merge 2 commits into
ros-develfrom
feat/rename_intensity_field
Open

feat: add parameter to rename intensity field#5
m-sanchez-rico wants to merge 2 commits into
ros-develfrom
feat/rename_intensity_field

Conversation

@m-sanchez-rico

Copy link
Copy Markdown

This pull request adds support for configurable intensity field names in the PointCloudToLaserScanNodelet. This allows users to specify which field in the point cloud message should be used for intensity, increasing flexibility for different sensor outputs.

Configurability improvements:

  • Added a new parameter intensity_field_name (defaulting to "intensity") in the nodelet's parameters, allowing users to specify the field name for intensity values. [1] [2]
  • Updated the point cloud iteration logic to use the configurable intensity_field_name instead of hardcoding "intensity", enabling compatibility with point clouds that use different field names.

@RomanRobotnik RomanRobotnik left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Increase package version please ;D

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.

2 participants