Skip to content

perf(road_user_stop): reduce the number of published debug markers - #11865

Merged
maxime-clem merged 2 commits into
autowarefoundation:mainfrom
maxime-clem:perf/road_user_stop-debug-poly
Jan 13, 2026
Merged

maxime-clem merged 2 commits into
autowarefoundation:mainfrom
maxime-clem:perf/road_user_stop-debug-poly

Conversation

@maxime-clem

Copy link
Copy Markdown
Contributor

Description

This PR reduces the time taken by the road_user_stop module to publish its debug markers.
To visualize many polygons, the module currently creates one LINE_STRIP debug marker for each polygon, which can result in more than 400 markers in my tests, leading to most of the module's runtime coming from publishing these markers.
With this PR, a single LINE_LIST marker is used to represent all polygons of a same namespace, leading the module to publish only 4 markers, greatly reducing the runtime of the node.

Related links

How was this PR tested?

Ran a scenario that was failing due to the performance issue.

Before After
image image

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

Slightly reduced runtime of the road_user_stop module.

Signed-off-by: Maxime CLEMENT <maxime.clement@tier4.jp>
@github-actions github-actions Bot added the component:planning Route planning, decision-making, and navigation. (auto-assigned) label Jan 13, 2026
@maxime-clem maxime-clem added run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) and removed component:planning Route planning, decision-making, and navigation. (auto-assigned) labels Jan 13, 2026
@github-actions

github-actions Bot commented Jan 13, 2026 •

Copy link
Copy Markdown

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@codecov

codecov Bot commented Jan 13, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 17.96%. Comparing base (573c5bd) to head (0412e78).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #11865      +/-   ##
==========================================
- Coverage   18.00%   17.96%   -0.04%     
==========================================
  Files        1811     1812       +1     
  Lines      125557   125595      +38     
  Branches    43640    43687      +47     
==========================================
- Hits        22603    22567      -36     
- Misses      84118    84217      +99     
+ Partials    18836    18811      -25     
Flag Coverage Δ *Carryforward flag
daily 20.48% <ø> (ø) Carriedforward from 7f803aa
daily-cuda 17.99% <ø> (ø) Carriedforward from 7f803aa
differential 0.40% <ø> (?)
total-cuda 17.99% <ø> (+<0.01%) ⬆️ Carriedforward from 7f803aa

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Maxime CLEMENT <maxime.clement@tier4.jp>
@github-actions github-actions Bot added the component:planning Route planning, decision-making, and navigation. (auto-assigned) label Jan 13, 2026

@Kotakku Kotakku left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@maxime-clem
maxime-clem merged commit 16fccb3 into autowarefoundation:main Jan 13, 2026
44 of 45 checks passed
@github-project-automation github-project-automation Bot moved this from To Triage to Done in Software Working Group Jan 13, 2026
@maxime-clem
maxime-clem deleted the perf/road_user_stop-debug-poly branch January 13, 2026 09:37
simonranjith pushed a commit to simonranjith/autoware_universe that referenced this pull request Jan 16, 2026
simonranjith pushed a commit to simonranjith/autoware_universe that referenced this pull request Jan 17, 2026
k-uta pushed a commit to tier4/autoware_universe that referenced this pull request Jan 28, 2026
KeisukeShima pushed a commit to KeisukeShima/autoware.universe that referenced this pull request Mar 16, 2026
mkquda pushed a commit to tier4/autoware_universe that referenced this pull request Mar 16, 2026
mkquda pushed a commit to tier4/autoware_universe that referenced this pull request Apr 22, 2026
tier4-autoware-public-bot Bot pushed a commit to tier4/pilot_auto_planning that referenced this pull request Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:planning Route planning, decision-making, and navigation. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants