Skip to content

feat(autoware_carla_interface): allow explicit carla_map override in launch - #13193

Merged
hakuturu583 merged 3 commits into
autowarefoundation:mainfrom
hakuturu583:feat/carla-map-override
Aug 18, 2026
Merged

hakuturu583 merged 3 commits into
autowarefoundation:mainfrom
hakuturu583:feat/carla-map-override

Conversation

@hakuturu583

@hakuturu583 hakuturu583 commented Aug 14, 2026 •

Copy link
Copy Markdown
Contributor

Description

Add an optional carla_map launch argument that, when non-empty, overrides the CARLA map/level name that is otherwise derived from the basename of map_path. This is useful when the CARLA level name does not match the Autoware map directory name.

Map-name resolution is factored into a carla_map_resolved <let>. When the carla_map argument is empty (the default), the name is derived from map_path exactly as before, so behavior is unchanged unless the user opts in.

This is one of a series of small enabler PRs toward CARLA 0.10.0 support, and is a launch-only change.

How was this PR tested?

  • Verified the launch XML is well-formed.
  • Verified that, when carla_map is empty (the default), the value derived from map_path is identical to the previous behavior.

Backward compatibility

  • carla_map defaults to "". Unless the user opts in, the resolved result is the same as today and the carla_map value passed to the node is unchanged.

Related PR

…launch

Add an optional `carla_map` launch argument that, when non-empty,
overrides the CARLA map/level name that is otherwise derived from the
basename of `map_path`. This is useful for CARLA levels whose name does
not match the Autoware map directory name.

The resolution is moved into a `carla_map_resolved` let: when the
`carla_map` argument is empty (the default), the name is derived from
`map_path` exactly as before, so behavior is unchanged unless the user
opts in.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Masaya Kataoka <ms.kataoka@gmail.com>
@hakuturu583 hakuturu583 added component:simulation Virtual environment setups and simulations. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) labels Aug 14, 2026
@github-actions

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 Aug 14, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 20.26%. Comparing base (f1bb0a6) to head (57b78d9).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #13193      +/-   ##
==========================================
- Coverage   20.33%   20.26%   -0.07%     
==========================================
  Files        1942     1948       +6     
  Lines      136595   136371     -224     
  Branches    48724    48633      -91     
==========================================
- Hits        27781    27641     -140     
- Misses      86291    86345      +54     
+ Partials    22523    22385     -138     
Flag Coverage Δ *Carryforward flag
full-suite 20.26% <ø> (-0.07%) ⬇️ Carriedforward from bf30f33

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

☔ View full report in Codecov by Harness.
📢 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.

@hakuturu583

Copy link
Copy Markdown
Contributor Author

Downstream consumer: the autoware_launch side that forwards this argument from e2e_simulator.launch.xml is prepared in autowarefoundation/autoware_launch#1948 (Draft; to be merged after this PR).

@hakuturu583
hakuturu583 requested a review from Max-Bin August 17, 2026 05:30
@hakuturu583
hakuturu583 marked this pull request as ready for review August 17, 2026 05:30

@Max-Bin Max-Bin 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.

Checked and LGTM

@hakuturu583
hakuturu583 enabled auto-merge (squash) August 17, 2026 11:46
@hakuturu583
hakuturu583 merged commit cbf3fc0 into autowarefoundation:main Aug 18, 2026
49 of 52 checks passed
@github-project-automation github-project-automation Bot moved this from To Triage to Done in Software Working Group Aug 18, 2026
hakuturu583 added a commit to hakuturu583/autoware.universe that referenced this pull request Aug 20, 2026
… with upstream main

The branch was cut before autowarefoundation#13195, autowarefoundation#13196, autowarefoundation#13198, autowarefoundation#13193 and autowarefoundation#13210
merged into main, so its diff against main had started to show those
already-landed fixes (steering-curve sort, has_attribute guards,
map_origin_x/y, carla_map override, no_rendering_mode) as reverted.
Merge main back in so the branch only carries its own README changes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Masaya Kataoka <ms.kataoka@gmail.com>
TetsuKawa pushed a commit to TetsuKawa/autoware_universe that referenced this pull request Sep 7, 2026
…launch (autowarefoundation#13193)

Signed-off-by: TetsuKawa <kawaguchitnon@icloud.com>
TetsuKawa pushed a commit to TetsuKawa/autoware_universe that referenced this pull request Sep 7, 2026
shmpwk pushed a commit to tier4/pilot_auto_simulation that referenced this pull request Sep 14, 2026
TetsuKawa pushed a commit to TetsuKawa/autoware_universe that referenced this pull request Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:simulation Virtual environment setups and simulations. (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