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

fix: add manual for submitting to the GettingStarted page #33

Merged
merged 5 commits into from
Jun 20, 2024

Conversation

dazzhe
Copy link
Contributor

@dazzhe dazzhe commented Jun 19, 2024

概要

GettingStartedページにオンライン採点環境周りの情報追加

@dazzhe dazzhe force-pushed the fix/getting_started branch from 1825aa9 to a907312 Compare June 19, 2024 09:11

[推奨環境](./setup/requirements.ja.md)を満たすPCが必要ですが、推奨と書かれているものについては満たしていなくても動作することは可能です。ただし、推奨よりも低いスペックで動作させる場合ROS2側での実行速度が安定せずシミュレーションの実行の度に挙動が大きく変わってしまう可能性があります。

!!! warning

Windows環境しかお持ちでない方は、Ubuntu22.04のインストールをお願いいたします。Windows環境と同じディスクにUbuntu環境を入れることもできますが、不慣れな場合Windows環境を破壊してしまう可能性があるため、新しく外付けまたは内蔵SSDを購入したうえでそちらへのインストールをすることを強くお勧めします。
Ubuntu22.04の環境しかお持ちでない方は、Ubuntu22.04のインストールをお願いいたします。既にOSが入っているディスクにUbuntu環境を入れることもできますが、不慣れな場合既存の環境を破壊してしまう可能性があるため、新しく外付けまたは内蔵SSDを購入したうえでそちらへのインストールをすることを強くお勧めします。
Copy link
Contributor

Choose a reason for hiding this comment

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

@dazzhe
ここの変更は不要そうに見えます。変更の意図を教えていただけますか?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@taikitanaka3
すみませんどうかしてたので戻しました。。。

@@ -23,6 +32,7 @@
- [ワークスペースのクローン](./setup/workspace-setup.ja.md)
- [Dockerのインストール](./setup/docker.ja.md)
- [AWSIMのダウンロード](./setup/headless-simulation.ja.md)
- この手順では描画無しバージョンのAWSIMがダウンロードされます。シミュレーション画面を確認したい方は[こちら](./setup/visible-simulation.ja.md)の手順に従ってください。
Copy link
Contributor

Choose a reason for hiding this comment

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

@dazzhe
描画ありのAWSIMでの実行は手順をひととおり終えてからの誘導としたいです

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@taikitanaka3
参考に移動しました

ユーザー登録がまだ済んでいない方はSlackでの案内に従って登録お願いします。

アクセスが出来たら、一度ソースコードの提出をしてみましょう。
[GoogleDrive](https://drive.google.com/file/d/19LU70cgeg48R6stEXjvwDp1pTT25OjeN)からサンプルコードの圧縮ファイルをダウンロードして、オンライン採点環境の「UPLOAD」ボタンからそのままアップロードすることで提出ができます。
Copy link
Contributor

Choose a reason for hiding this comment

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

@dazzhe
sample のファイルのアップロード画面のスクショは載せてもいいかと思います。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@taikitanaka3
追加しました

Copy link
Contributor

@taikitanaka3 taikitanaka3 left a comment

Choose a reason for hiding this comment

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

上記の変更を反映していただければLGTMです

@dazzhe dazzhe force-pushed the fix/getting_started branch from 2f06461 to 6cd6d49 Compare June 20, 2024 08:02
@dazzhe dazzhe marked this pull request as ready for review June 20, 2024 08:03
@taikitanaka3
Copy link
Contributor

(prd環境はまだリリースしていないので、pre-commitのlinkはfailで想定どおり)
Markdown Link Check......................................................Failed

@taikitanaka3 taikitanaka3 merged commit 10028cf into main Jun 20, 2024
1 of 2 checks passed
@taikitanaka3 taikitanaka3 deleted the fix/getting_started branch June 20, 2024 08:20
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