open-simulation-interface icon indicating copy to clipboard operation
open-simulation-interface copied to clipboard

Adding the missing field for echo pulse width to logical detections

Open PhRosenberger opened this issue 3 years ago • 1 comments

This is necessary to represent Ibeo LUX ECU data or similar that contain fused lidar point clouds with echo pulse width instead of intensity.

Take this checklist as orientation for yourself, if this PR is ready for the Change Control Board:

  • [x] My suggestion follows the style and contributors guidelines.
  • [x] I have taken care about the documentation.
  • [x] I have done the DCO signoff.
  • [ ] My changes generate no errors when passing CI tests.
  • [x] I have successfully implemented and tested my fix/feature locally.
  • [x] Appropriate reviewer(s) are assigned.

If you can’t check all of them, please explain why. If all boxes are checked or commented and you have achieved at least one positive review, you can assign the label ReadyForCCBReview!

PhRosenberger avatar Oct 18 '21 19:10 PhRosenberger

09.05.2022: @kmeids might this be a change suitable for v3.5?

clemenshabedank avatar May 09 '22 07:05 clemenshabedank