Skip to content

feat(autoware_lanelet2_map_visualizer): add new maker#828

Merged
yuki-takagi-66 merged 2 commits intoautowarefoundation:mainfrom
tier4:takagi/add-map_area_filter-maker
Feb 19, 2026
Merged

feat(autoware_lanelet2_map_visualizer): add new maker#828
yuki-takagi-66 merged 2 commits intoautowarefoundation:mainfrom
tier4:takagi/add-map_area_filter-maker

Conversation

@yuki-takagi-66
Copy link
Copy Markdown
Contributor

@yuki-takagi-66 yuki-takagi-66 commented Feb 3, 2026

Description

add new maker type for obstacle_removal_area

Related links

lanelet2extension PR:
autowarefoundation/autoware_lanelet2_extension#96
autowarefoundation/autoware_lanelet2_extension#100

How was this PR tested?

visualized as the following:
Screenshot from 2026-02-03 15-12-51

Notes for reviewers

New released version of lanelet2extension is required

Interface changes

None.

Effects on system behavior

None.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 3, 2026

Thank you for contributing to the Autoware project!

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

Please ensure:

@yuki-takagi-66 yuki-takagi-66 marked this pull request as ready for review February 3, 2026 06:27
@yuki-takagi-66 yuki-takagi-66 added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Feb 3, 2026
@sasakisasaki sasakisasaki self-assigned this Feb 4, 2026
Copy link
Copy Markdown
Contributor

@sasakisasaki sasakisasaki left a comment

Choose a reason for hiding this comment

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

Reviewed code and the test result looks good 👍

@yuki-takagi-66 yuki-takagi-66 force-pushed the takagi/add-map_area_filter-maker branch from 318ca48 to 1208ff7 Compare February 16, 2026 09:47
@yuki-takagi-66 yuki-takagi-66 added run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) and removed run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) labels Feb 16, 2026
Signed-off-by: yuki-takagi-66 <yuki.takagi@tier4.jp>
Signed-off-by: yuki-takagi-66 <yuki.takagi@tier4.jp>
@yuki-takagi-66 yuki-takagi-66 added run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) and removed run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) labels Feb 18, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.75%. Comparing base (a013124) to head (656f401).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #828   +/-   ##
=======================================
  Coverage   50.74%   50.75%           
=======================================
  Files         348      348           
  Lines       22529    22533    +4     
  Branches    10067    10071    +4     
=======================================
+ Hits        11433    11437    +4     
  Misses       9999     9999           
  Partials     1097     1097           
Flag Coverage Δ *Carryforward flag
daily-humble 50.55% <ø> (-0.02%) ⬇️ Carriedforward from a013124
daily-jazzy 50.00% <ø> (ø) Carriedforward from a013124
differential-humble 6.89% <100.00%> (?)
total 50.18% <ø> (ø) Carriedforward from a013124
total-humble 50.54% <ø> (ø) Carriedforward from a013124
total-jazzy 50.00% <ø> (ø) Carriedforward from a013124

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

@yuki-takagi-66 yuki-takagi-66 merged commit e157e16 into autowarefoundation:main Feb 19, 2026
76 of 88 checks passed
@yuki-takagi-66 yuki-takagi-66 deleted the takagi/add-map_area_filter-maker branch February 19, 2026 06:42
@xmfcx
Copy link
Copy Markdown
Contributor

xmfcx commented Feb 20, 2026

Why was this merged when https://github.com/autowarefoundation/autoware_core/actions/runs/22088376814 has failed?

image

https://github.com/autowarefoundation/autoware_core/actions/runs/22088376814/job/63827899733#step:14:1561

/__w/autoware_core/autoware_core/map/autoware_lanelet2_map_visualizer/src/lanelet2_map_visualization_node.cpp: In member function ‘void autoware::lanelet2_map_visualizer::Lanelet2MapVisualizationNode::on_map_bin(autoware_map_msgs::msg::LaneletMapBin_<std::allocator<void> >::ConstSharedPtr)’:
/__w/autoware_core/autoware_core/map/autoware_lanelet2_map_visualizer/src/lanelet2_map_visualization_node.cpp:331:48: error: ‘obstacleRemovalAreaAsMarkerArray’ is not a member of ‘lanelet::visualization’
  331 |     &map_marker_array, lanelet::visualization::obstacleRemovalAreaAsMarkerArray(
      |                                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

I get the same error when building nightly.

I know that these following prs are necessary:

But they should be merged, the new docker image should be built, then this CI would pass.

Right now autoware nightly is not building because this was merged with bypassing the CI requirements.

@xmfcx
Copy link
Copy Markdown
Contributor

xmfcx commented Feb 20, 2026

It was my fault for not pulling the autoware.repos first.

But still, please don't override the CI unless there is an emergency or something.

@mitsudome-r
Copy link
Copy Markdown
Member

@xmfcx
I don't think it was merged with by-pass merge. The final results were passing:
https://github.com/autowarefoundation/autoware_core/actions/runs/22135634335

image

@xmfcx
Copy link
Copy Markdown
Contributor

xmfcx commented Feb 20, 2026

So weird, when I clicked Checks -> build-test-tidy-pr link, it redirected me to the failing CI page which was https://github.com/autowarefoundation/autoware_core/actions/runs/22088376814 but now it redirects to the correct page as yours. I'm a bit confused on how it happened.

image

Oh now I see, there are 2 entries in this page: https://github.com/autowarefoundation/autoware_core/pull/828/checks

image

I'm not sure why the duplicates are there.

image

There were 4 run retries but 2 are visible there. One for the first run, one for the last one.

Sorry for all the commotion lol.

sarun-hub pushed a commit to sarun-hub/autoware_core that referenced this pull request Mar 5, 2026
…tion#828)

* add maker
---------
Signed-off-by: yuki-takagi-66 <yuki.takagi@tier4.jp>
Signed-off-by: Sarun Mukdapitak <sarun.mukda@gmail.com>
yuki-takagi-66 added a commit to tier4/autoware_core that referenced this pull request Mar 11, 2026
…tion#828)

* add maker
---------
Signed-off-by: yuki-takagi-66 <yuki.takagi@tier4.jp>
kotaro-hihara added a commit to tier4/autoware_core that referenced this pull request Mar 16, 2026
feat(autoware_lanelet2_map_visualizer): add new maker (autowarefoundation#828)
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