Skip to content

Conversation

YusukeKato
Copy link
Collaborator

@YusukeKato YusukeKato commented Nov 8, 2023

What does this implement/fix?

Raspberry Pi Mouseの項目にROS 2版シミュレータの導入手順と動作手順を追加します。

Does this close any currently open issues?

しません。

How has this been tested?

ローカル環境でpreview.shを実行して表示を確認しました。

Any other comments?

Checklists

@YusukeKato YusukeKato added the Type: Feature New Feature label Nov 8, 2023
@YusukeKato YusukeKato requested a review from ShotaAk November 8, 2023 07:47
@YusukeKato YusukeKato self-assigned this Nov 8, 2023
Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

確認しました。全体的な文章OKです。
一部指摘事項と、コマンドについてコメントしてます。

Raspberry Pi Mouse SimulatorはメインボードにRaspberry Piを使った左右独立二輪方式の小型移動プラットフォームロボット、Raspberry Pi Mouseのシミュレータです。

Gazebo上で動きます。ROSとGazeboがインストールされた環境で使用することができます
Gazebo上で動きます。ROS/ROS 2とGazeboがインストールされた環境で使用することができます
Copy link
Contributor

Choose a reason for hiding this comment

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

この一文は削除しましょう。
ROS/ROS 2が必要なことは使用機材のところで説明している & gazeboはraspimouse_simと同時にインストールされるので。

Suggested change
Gazebo上で動きます。ROS/ROS 2とGazeboがインストールされた環境で使用することができます。


# シミュレータのインストール

Raspberry Pi Mouse SimulatorはメインボードにRaspberry Piを使った左右独立二輪方式の小型移動プラットフォームロボット、Raspberry Pi Mouseのシミュレータです。
Copy link
Contributor

Choose a reason for hiding this comment

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

Raspberry Pi Mouse Simulatorとraspimouse_simが同じものであることを示したいです。

Suggested change
Raspberry Pi Mouse SimulatorはメインボードにRaspberry Piを使った左右独立二輪方式の小型移動プラットフォームロボット、Raspberry Pi Mouseのシミュレータです。
Raspberry Pi Mouse Simulator(rt-net/raspimouse_sim)はメインボードにRaspberry Piを使った左右独立二輪方式の小型移動プラットフォームロボット、Raspberry Pi Mouseのシミュレータです。

$ git clone -b humble-devel https://github.com/rt-net/raspimouse_sim.git

# 依存パッケージをインストール
$ git clone -b humble-devel https://github.com/rt-net/raspimouse_description.git
Copy link
Contributor

Choose a reason for hiding this comment

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

raspimouse_descriptionはraspimouse_simの依存関係に入っているのでコマンド不要です。

Suggested change
$ git clone -b humble-devel https://github.com/rt-net/raspimouse_description.git

```

=== "ROS 2"
次のコマンドを実行します。
Copy link
Contributor

Choose a reason for hiding this comment

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

リリースが完了したらapt installコマンドに書き換えましょう

Copy link
Contributor

Choose a reason for hiding this comment

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

コマンドは右上のボタンでコピーできるので、コピー&実行しやすいレイアウトに変更したいです。
image

ROS 2 Docのようなドキュメントをイメージしてます。https://docs.ros.org/en/foxy/Installation/Ubuntu-Install-Debians.html

具体的には

  • 文頭のドルマーク($)を書かない
  • コマンドの塊を分割する(ダウンロード、インストール、ビルド単位の塊にする)
  • #で始まるコメントを本文に記載する (コメント文は実行されないので、問題は無いですが)

Copy link
Contributor

Choose a reason for hiding this comment

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

最終的には他のページのコマンド文も書き換えたいです。

YusukeKato and others added 4 commits November 9, 2023 11:49
Copy link
Contributor

@ShotaAk ShotaAk left a comment

Choose a reason for hiding this comment

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

LGTMです!

@ShotaAk ShotaAk merged commit 59985c0 into main Nov 9, 2023
@ShotaAk ShotaAk deleted the add-raspimouse-sim branch November 9, 2023 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants