-
Notifications
You must be signed in to change notification settings - Fork 664
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
feat(lidar_centerpoint, pointpainting): add diag publisher for max voxel size #9720
base: main
Are you sure you want to change the base?
feat(lidar_centerpoint, pointpainting): add diag publisher for max voxel size #9720
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
7cd4111
to
d13ae2c
Compare
…xel size Signed-off-by: kminoda <[email protected]>
d13ae2c
to
190bcb6
Compare
6f0b47b
to
ede9a66
Compare
Signed-off-by: kminoda <[email protected]>
Signed-off-by: kminoda <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9720 +/- ##
==========================================
+ Coverage 28.79% 29.59% +0.80%
==========================================
Files 1455 1451 -4
Lines 109092 108611 -481
Branches 42493 42633 +140
==========================================
+ Hits 31412 32147 +735
+ Misses 74602 73301 -1301
- Partials 3078 3163 +85
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Description
Add diagnostics for pointpainting and centerpoint, in order to handle a case where number of voxels gets higher than the threshold.
TIER IV INTERNAL LINK
Related links
Parent Issue:
How was this PR tested?
Tested by running logging simulator
Centerpoint
PointPainting
Notes for reviewers
None.
Interface changes
Topic changes
Additions and removals
/diagnostics
diagnostic_msgs/msg/DiagnosticArray
Effects on system behavior
None.