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 shell script chapter2 #44

Merged
merged 6 commits into from
Jun 18, 2024

Conversation

KariControl
Copy link
Contributor

@KariControl KariControl commented Jun 17, 2024

・PRリクエスト内容
easy-menuとして、Autoware入門講座の第二章の内容を一括実行するシェルスクリプトを用意しました。issue #40 の提案を受けて、autoware_pracitice_cource のスクリプトとして installして実行するようにしました。

・実行方法(確認方法)
ビルド後に、下記のコマンド実行により第二章の各例題の内容を一括実行します。例題02-01はtopic確認だけだったので、シェルスクリプトは特に用意していません。

-02-02
ros2 run autoware_practice_course run_autoware_practice_cource_example02_02.sh
※比例ゲインはデフォルトで5に設定。

-02-03
ros2 run autoware_practice_course run_autoware_practice_cource_example02_03.sh

-02-04
ros2 run autoware_practice_course run_autoware_practice_cource_example02_04.sh

@taikitanaka3 taikitanaka3 changed the title Feature to add shell script chapter2 feat: add shell script chapter2 Jun 17, 2024
@taikitanaka3
Copy link

@KariControl
ありがとうございます!
PRタイトルはconventional commitに従って改変させていただきました。

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.

ありがとうございます!LGTM!

@masahiro-kubota
Copy link
Contributor

@KariControl
イシューとPRの関連がわかりやすいように、イシューのリンクを付けさせていただきました!

Copy link
Contributor

@isamu-takagi isamu-takagi left a comment

Choose a reason for hiding this comment

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

ありがとうございます。このままでも LGTM ですが、スクリプト名は run_02_02.sh などと短くしてしまっても良さそうに思いました。

@KariControl
Copy link
Contributor Author

@masahiro-kubota @isamu-takagi ご確認いただきありがとうございます。ご提案いただいた通り、スクリプト名を変更いたしました。ご確認ください

@isamu-takagi isamu-takagi merged commit ea7149b into main Jun 18, 2024
1 check passed
@isamu-takagi isamu-takagi deleted the feature_to_add_shell_script_chapter2 branch June 18, 2024 05:27
@taikitanaka3
Copy link

resolve #40

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.

4 participants