Skip to content

feat(docker): add localization/map launch files to autoware:universe-localization-mapping images#5586

Merged
mitsudome-r merged 11 commits intomainfrom
feat/lm-launch
Mar 21, 2025
Merged

feat(docker): add localization/map launch files to autoware:universe-localization-mapping images#5586
mitsudome-r merged 11 commits intomainfrom
feat/lm-launch

Conversation

@youtalk
Copy link
Copy Markdown
Member

@youtalk youtalk commented Dec 19, 2024

Description

This PR makes it possible to run the autoware:universe-localization-mapping containers using the launch files designed for the components.

Ref.

How was this PR tested?

Notes for reviewers

None.

Effects on system behavior

None.

Signed-off-by: Yutaka Kondo <[email protected]>
@youtalk youtalk self-assigned this Dec 19, 2024
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 19, 2024

Thank you for contributing to the Autoware project!

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

Please ensure:

@youtalk youtalk added type:containers Docker containers, containerization of components, or container orchestration. component:openadkit Issues or Features related to Open AD Kit run:health-check Run health-check labels Dec 19, 2024
@youtalk youtalk requested review from mitsudome-r and xmfcx December 19, 2024 13:23
Comment thread docker/Dockerfile Outdated
youtalk added 4 commits March 17, 2025 20:06
Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: Yutaka Kondo <[email protected]>
@youtalk youtalk marked this pull request as ready for review March 17, 2025 11:09
@youtalk youtalk requested a review from mitsudome-r March 17, 2025 11:09
@mitsudome-r
Copy link
Copy Markdown
Member

I will check locally if we can launch sensing/perception without an error with the built image.

@mitsudome-r
Copy link
Copy Markdown
Member

I have tested the locally built docker with the following steps and confirmed that they launch without an error. (I just checked that there is no ERROR output and did't test with the full stack autoware)

Localization Launch

same instruction in #5586 (comment)

Map Launch

  1. Download sample map configuration files map_config.zip and sample-map-planning from autoware tutorial

  2. run docker

docker run -it -v /home/user/autoware_map:/autoware_map -v /home/user/map_config:/map_config ghcr.io/autowarefoundation/autoware:universe-localization-mapping
  1. run launch file
ros2 launch tier4_map_launch map.launch.xml \
  pointcloud_map_path:=/autoware_map/sample-map-planning/pointcloud_map.pcd \
  pointcloud_map_metadata_path:=/autoware_map/sample-map-planning/pointcloud_map_metadata.yaml \
  lanelet2_map_path:=/autoware_map/sample-map-planning/lanelet2_map.osm \
  map_projector_info_path:=/autoware_map/sample-map-planning/map_projector_info.yaml \
  pointcloud_map_loader_param_path:=/map_config/pointcloud_map_loader.param.yaml \
  lanelet2_map_loader_param_path:=/map_config/lanelet2_map_loader.param.yaml \
  map_tf_generator_param_path:=/map_config/map_tf_generator.param.yaml \
  map_projection_loader_param_path:=/map_config/map_projection_loader.param.yaml

@youtalk youtalk enabled auto-merge (squash) March 19, 2025 21:52
@mitsudome-r mitsudome-r disabled auto-merge March 21, 2025 04:49
@mitsudome-r
Copy link
Copy Markdown
Member

Health check is failing due to disk space issue, but I believe this is due to the increase in cuda images and not from this PR.
I have locally tested the docker build locally and woks fine so I'm merging this PR for now. We will create another PR to fix the disk space issue.

@mitsudome-r mitsudome-r merged commit 63e7702 into main Mar 21, 2025
16 of 18 checks passed
@mitsudome-r mitsudome-r deleted the feat/lm-launch branch March 21, 2025 04:52
@github-project-automation github-project-automation Bot moved this from Execution to Done in Open AD Kit Working Group Mar 21, 2025
sykwer pushed a commit to sykwer/autoware that referenced this pull request Mar 22, 2025
…se-localization-mapping` images (autowarefoundation#5586)

Signed-off-by: Yutaka Kondo <[email protected]>
Signed-off-by: sykwer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:openadkit Issues or Features related to Open AD Kit run:health-check Run health-check type:containers Docker containers, containerization of components, or container orchestration.

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

2 participants