autoware.universe icon indicating copy to clipboard operation
autoware.universe copied to clipboard

fix(euclidean_cluster): add intensity field

Open badai-nguyen opened this issue 10 months ago • 2 comments

Description

  • To process and output the fields of clusters' pointcloud as the input topic including intensity field for voxel_grid_based_euclidean_cluster to solve the issue https://github.com/autowarefoundation/autoware.universe/issues/6785

Related links

Tests performed

  • Confirmed by logging_simulator that voxel_grid_based_euclidean_cluster
    • Can process and output the topic with intensity field:

image

  • Processing time increase a little bit for a quite complicated scene image
Before After
image image

Notes for reviewers

Interface changes

Effects on system behavior

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

  • [ ] The PR follows the pull request guidelines.
  • [ ] The PR has been properly tested.
  • [ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • [ ] There are no open discussions or they are tracked via tickets.
  • [ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

badai-nguyen avatar Apr 15 '24 15:04 badai-nguyen

@YoshiRi could I ask your review for this PR also?

badai-nguyen avatar May 14 '24 06:05 badai-nguyen

@yukkysaito Do you have any comment for this PR? Otherwise, could you approve this PR as codeowner?

badai-nguyen avatar May 16 '24 04:05 badai-nguyen