Skip to content

chore(planning, bvp): remove unused lanelet2_extension header#902

Merged
soblin merged 4 commits intoautowarefoundation:mainfrom
sarun-hub:chore/planning-bvp/remove-unused-lanelet2-extension
Mar 18, 2026
Merged

chore(planning, bvp): remove unused lanelet2_extension header#902
soblin merged 4 commits intoautowarefoundation:mainfrom
sarun-hub:chore/planning-bvp/remove-unused-lanelet2-extension

Conversation

@sarun-hub
Copy link
Copy Markdown
Contributor

Description

Several PRs already ported functions from autoware_lanelet2_extension to autoware_lanelet2_utils.
During the porting process, some header and depends are not removed after replacing usage.
This PR cleans it.

Target components:

  • planning
  • behavior_velocity_planner

Related links

#894
Parent Issue:

  • Link

How was this PR tested?

Pass CI and Evaluator (https://evaluation.tier4.jp/evaluation/reports/bff44f2c-cf63-53f7-ba73-4fc0fbce4649?project_id=autoware_dev)

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>
Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 11, 2026

Thank you for contributing to the Autoware project!

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

Please ensure:

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.41%. Comparing base (1acc034) to head (0020bf8).
⚠️ Report is 17 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #902      +/-   ##
==========================================
- Coverage   50.44%   50.41%   -0.03%     
==========================================
  Files         367      369       +2     
  Lines       23002    23017      +15     
  Branches    10215    10215              
==========================================
+ Hits        11603    11604       +1     
- Misses      10280    10297      +17     
+ Partials     1119     1116       -3     
Flag Coverage Δ *Carryforward flag
daily-humble 50.67% <ø> (-0.04%) ⬇️ Carriedforward from 1a217df
daily-jazzy 50.17% <ø> (-0.02%) ⬇️ Carriedforward from 1a217df
differential-humble 28.59% <ø> (?)
differential-jazzy 28.73% <ø> (?)
total 50.37% <ø> (ø) Carriedforward from 1a217df
total-humble 50.71% <ø> (+0.01%) ⬆️ Carriedforward from 1a217df
total-jazzy 50.19% <ø> (+<0.01%) ⬆️ Carriedforward from 1a217df

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

@sarun-hub
Copy link
Copy Markdown
Contributor Author

sarun-hub commented Mar 13, 2026

The build error is due to change in autoware_path_optimizer to load acados in autoware_static_centerline_generator in autoware_tools as mentioned in autoware_tool PR #370.

I will wait for this PR to be merged and re-run CI.
image

@soblin soblin merged commit a36ae31 into autowarefoundation:main Mar 18, 2026
39 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants