Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add aichallenge submit launch arg #39

Merged

Conversation

taikitanaka3
Copy link
Contributor

  • docker 内部にあるautoware launchのparamを外だししました。
  • source build branchの1歩先のPRとして出します。

AWSIMを起動して通常通り動くことの確認
image

@isamu-takagi
Copy link
Contributor

マージ先は source の方じゃないです?

@taikitanaka3
Copy link
Contributor Author

@isamu-takagi
この変更はpullせずに予選大会を進められる&変更による挙動の変化はないのでmainへのprとしました

@isamu-takagi
Copy link
Contributor

@taikitanaka3 コピーが必要なのはsource buildのときに無駄な依存解決が出てくるのを防ぐためなので、dockerで動かす想定のmainnoの場合はdocker内にコンフィグファイルを参照する形で問題ないと思います。submitの内容なので変更が必要になったときに参加者がコピーして編集すればOKです。

@taikitanaka3
Copy link
Contributor Author

@isamu-takagi
このあとのマイルストーンがdockerとsource buildの共存になりそうであればこのPRは不要ですし、まだもう少しsource buildでの検証が必要そうであればsource buildでもdocker環境でも動く形に持っていき、動作確認はsource build側で行うのも良いと思います。

PRの動作確認がdockerで完結しそうであれば不要そうですかね。

@isamu-takagi
Copy link
Contributor

source build は main では使わないので一時作業用のブランチだけで良さそうです。

@isamu-takagi isamu-takagi changed the base branch from main to test/source-build August 22, 2024 13:19
@taikitanaka3
Copy link
Contributor Author

@masahiro-kubota
こちら今日の実験で問題なければapproveお願いします。

@masahiro-kubota
Copy link
Contributor

masahiro-kubota commented Aug 23, 2024

@taikitanaka3
こちら今日の実験で使っていないので、今日ベンチで確かめます

@masahiro-kubota
Copy link
Contributor

@taikitanaka3
以下のPRでactuation_cmdまで出るので、以下のPRをマージしてこちらはcloseしたいと思います
#42

@taikitanaka3
Copy link
Contributor Author

@masahiro-kubota
PRありがとうございます。
#42の変更が多すぎてPRのタイトルから少し離れてしまっているので、このPRをmergeしたあと#42をmergeしstep by stepで進めませんか?

具体的には#42をレビューする際にこのPRからどの部分が変わっているのかだけを知りたいです。

Copy link
Contributor

@masahiro-kubota masahiro-kubota left a comment

Choose a reason for hiding this comment

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

了解です!こちらマージします!

@masahiro-kubota masahiro-kubota merged commit fb563af into test/source-build Aug 24, 2024
@masahiro-kubota masahiro-kubota deleted the feat/add_aichallenge_submit_launch_arg branch August 24, 2024 03:26
knorrrr pushed a commit that referenced this pull request Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants