-
Notifications
You must be signed in to change notification settings - Fork 691
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(planning_validator): add diag to check planning component latency #10241
feat(planning_validator): add diag to check planning component latency #10241
Conversation
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
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
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #10241 +/- ##
=======================================
Coverage 26.25% 26.26%
=======================================
Files 1381 1381
Lines 107503 107531 +28
Branches 41411 41416 +5
=======================================
+ Hits 28229 28238 +9
- Misses 76459 76478 +19
Partials 2815 2815
*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:
|
Description
I added a feature to calculate control component latency and to publish error diagnostic if it is larger than threshold.
Related links
Parent Issue:
How was this PR tested?
nominal_latency_threshold
.Notes for reviewers
None.
Interface changes
Add new field to publish current latency.
Effects on system behavior
None.