Skip to content

feat(artifacts): add artifact download process for SIMPL#6346

Merged
youtalk merged 1 commit into
autowarefoundation:mainfrom
ktro2828:feat/artifacts/simpl
Aug 6, 2025
Merged

feat(artifacts): add artifact download process for SIMPL#6346
youtalk merged 1 commit into
autowarefoundation:mainfrom
ktro2828:feat/artifacts/simpl

Conversation

@ktro2828
Copy link
Copy Markdown
Contributor

@ktro2828 ktro2828 commented Jul 30, 2025

Description

Parent Issue: autowarefoundation/autoware_universe#6348
Relevant PR: autowarefoundation/autoware_universe#10824

This PR adds process to download artifact for ML-based motion prediction model called SIMPL.

How was this PR tested?

  • Confirmed setup-dev-env.sh worked
  • Confirmed autoware_simpl_prediction worked without any problems using the latest autoware_universe and default parameters as follows:
$ ros2 launch autoware_simpl_prediction simpl.launch.xml

Notes for reviewers

None.

Effects on system behavior

None.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 30, 2025

Thank you for contributing to the Autoware project!

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

Please ensure:

@ktro2828 ktro2828 requested review from YoshiRi and technolojin July 30, 2025 09:34
@ktro2828 ktro2828 added the run:health-check Run health-check label Jul 30, 2025
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
@ktro2828 ktro2828 force-pushed the feat/artifacts/simpl branch from dd06da4 to d17281f Compare July 30, 2025 10:13
@ktro2828 ktro2828 changed the title feat: add artifact download process for SIMPL feat(artifacts): add artifact download process for SIMPL Jul 30, 2025
@technolojin technolojin self-assigned this Aug 5, 2025
Copy link
Copy Markdown
Contributor

@technolojin technolojin left a comment

Choose a reason for hiding this comment

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

Confirmed the downloaded onnx file

Copy link
Copy Markdown

@YoshiRi YoshiRi left a comment

Choose a reason for hiding this comment

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

LGTM

@ktro2828
Copy link
Copy Markdown
Contributor Author

ktro2828 commented Aug 6, 2025

@oguzkaganozt @youtalk Could anyone review this PR as a maintainer?

Copy link
Copy Markdown
Member

@youtalk youtalk left a comment

Choose a reason for hiding this comment

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

LGTM

@youtalk youtalk merged commit 3e18a8e into autowarefoundation:main Aug 6, 2025
16 of 18 checks passed
@ktro2828 ktro2828 deleted the feat/artifacts/simpl branch August 6, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run:health-check Run health-check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants