Skip to content

fix(diffusion_planner): removed result.predictions.clear();#12171

Merged
SakodaShintaro merged 1 commit intoautowarefoundation:mainfrom
tier4:fix/diffusion_planner/remove_predictions_clear
Feb 25, 2026
Merged

fix(diffusion_planner): removed result.predictions.clear();#12171
SakodaShintaro merged 1 commit intoautowarefoundation:mainfrom
tier4:fix/diffusion_planner/remove_predictions_clear

Conversation

@SakodaShintaro
Copy link
Contributor

@SakodaShintaro SakodaShintaro commented Feb 25, 2026

Description

The autoware_perception_msgs/TrafficLightGroup message type was updated to include a predictions array in the following PRs (v1.8.0 and later):

To maintain backward compatibility, this PR removes the line result.predictions.clear();. Since this line currently has no functional impact, removing it will not affect the system's behavior.

How was this PR tested?

  • planning simulator
  • logging simulator

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

Signed-off-by: Shintaro Sakoda <shintaro.sakoda@tier4.jp>
@SakodaShintaro SakodaShintaro self-assigned this Feb 25, 2026
@SakodaShintaro SakodaShintaro added 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) labels Feb 25, 2026
@github-actions
Copy link

Thank you for contributing to the Autoware project!

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

Please ensure:

@SakodaShintaro SakodaShintaro marked this pull request as ready for review February 25, 2026 01:42
@SakodaShintaro SakodaShintaro enabled auto-merge (squash) February 25, 2026 01:45
@SakodaShintaro SakodaShintaro merged commit ff44c63 into autowarefoundation:main Feb 25, 2026
37 of 43 checks passed
@SakodaShintaro SakodaShintaro deleted the fix/diffusion_planner/remove_predictions_clear branch February 25, 2026 01:48
@github-project-automation github-project-automation bot moved this from To Triage to Done in Software Working Group Feb 25, 2026
@codecov
Copy link

codecov bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 18.16%. Comparing base (ae88de5) to head (eecb350).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12171      +/-   ##
==========================================
+ Coverage   18.11%   18.16%   +0.05%     
==========================================
  Files        1841     1844       +3     
  Lines      126643   126900     +257     
  Branches    44419    44460      +41     
==========================================
+ Hits        22941    23051     +110     
- Misses      84592    84841     +249     
+ Partials    19110    19008     -102     
Flag Coverage Δ *Carryforward flag
daily 20.60% <ø> (ø) Carriedforward from ae88de5
daily-cuda 18.11% <ø> (+<0.01%) ⬆️ Carriedforward from ae88de5
differential-cuda 13.67% <ø> (?)
total-cuda 18.11% <ø> (+<0.01%) ⬆️ Carriedforward from ae88de5

*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.

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

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants