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

Add an option that allows users to switch use REFLECTIVITY to fill the intensity values instead of INTENSITY #427

Open
Samahu opened this issue Jan 27, 2025 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@Samahu
Copy link
Contributor

Samahu commented Jan 27, 2025

Is your feature request related to a problem? Please describe.
Historically the driver used the INTENSITY values to fill the point cloud intensity, however, using the REFLECTIVITY would be more appropriate since the values are calibrated. It also has a meaningful max. Checkout #408

Describe the solution you'd like
To preserve the current behavior for old instance we need to add an option that allows the users to choose between the current mode or used the REFLECTIVITY values instead when populating pt.intensity.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Targeted Platform (please complete the following information only if applicable, otherwise dot N/A):

  • Ouster Sensor?ANY
  • Ouster Firmware Version? ANY
  • ROS version/distro? ANY, for ROS2 humble and later set REFLECTIVITY as the default
  • Operating System? ANY
  • Machine Architecture? ANY
@Samahu Samahu added the enhancement New feature or request label Jan 27, 2025
@Samahu Samahu self-assigned this Jan 27, 2025
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

1 participant