-
Notifications
You must be signed in to change notification settings - Fork 650
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(route_selector): add processing time publisher #9343
base: main
Are you sure you want to change the base?
feat(route_selector): add processing time publisher #9343
Conversation
Signed-off-by: Kasunori-Nakajima <[email protected]>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Signed-off-by: Kasunori-Nakajima <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #9343 +/- ##
==========================================
- Coverage 29.21% 29.19% -0.03%
==========================================
Files 1335 1342 +7
Lines 102960 103042 +82
Branches 39950 39953 +3
==========================================
Hits 30080 30080
- Misses 70010 70092 +82
Partials 2870 2870
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
The service callback functions of this node may abort due to exceptions. In that case, the processing time will not be published, will this be OK?
Description
The processing time of
route_selector
was measured and made to Pub.Since A is not a module that is processed cyclically, Pub was added to the following four functions that may become heavy processing.
on_set_waypoint_route_main
on_set_lanelet_route_main
on_set_waypoint_route_mrm
on_set_lanelet_route_mrm
(TIER IV internal usecase)
This output is then read by the Basic scenario and tested daily for cycle failures.
Related links
How was this PR tested?
ros2 topic echo /planning/mission_planning/route autoware_planning_msgs/msg/LaneletRoute > routefile
start_pose
and---
on the last line of the routefileros2 service call /planning/mission_planning/route_selector/mrm/set_lanelet_route tier4_planning_msgs/srv/SetLaneletRoute "$(cat routefile)"
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.