-
Notifications
You must be signed in to change notification settings - Fork 716
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(autoware_tracker_object_merger): add diagnostics #10257
feat(autoware_tracker_object_merger): add diagnostics #10257
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
perception/autoware_tracking_object_merger/src/decorative_tracker_merger_node.cpp
Outdated
Show resolved
Hide resolved
perception/autoware_tracking_object_merger/src/decorative_tracker_merger_node.cpp
Show resolved
Hide resolved
perception/autoware_tracking_object_merger/src/decorative_tracker_merger_node.cpp
Outdated
Show resolved
Hide resolved
perception/autoware_tracking_object_merger/src/decorative_tracker_merger_node.cpp
Outdated
Show resolved
Hide resolved
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Co-authored-by: Taekjin LEE <technolojin@gmail.com> Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
a1eb957
to
b63f4f6
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10257 +/- ##
==========================================
+ Coverage 25.64% 26.17% +0.52%
==========================================
Files 1362 1378 +16
Lines 104703 107627 +2924
Branches 39820 41480 +1660
==========================================
+ Hits 26856 28174 +1318
- Misses 75239 76627 +1388
- Partials 2608 2826 +218
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
perception/autoware_tracking_object_merger/config/decorative_tracker_merger.param.yaml
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
Add diagnostics to judge following case.
Related links
autowarefoundation/autoware_launch#1357
Parent Issue:
How was this PR tested?
https://evaluation.ci.tier4.jp/evaluation/reports/e019a2e3-0b9e-50f6-a238-abb79d8148a0?project_id=prd_jt
I checked certain rosbag w/o radar msg.
Screencast.from.03-12-2025.04.59.34.PM.webm
Notes for reviewers
None.
Interface changes
/diagnostics
diagnostic_msgs/msg/DiagnosticArray
Effects on system behavior
None.