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

Update crane_plus_gazebo #30

Merged
merged 8 commits into from
Aug 27, 2021
Merged

Update crane_plus_gazebo #30

merged 8 commits into from
Aug 27, 2021

Conversation

ShotaAk
Copy link

@ShotaAk ShotaAk commented Aug 5, 2021

What does this implement/fix?

gazebo_ros2_controlパッケージを使い、Gazebo上でCRANE+V2を動かします。

このPRは、#29 がマージされる前提で作成しています。

#29 を取り入れた後に動作確認お願いします。

Does this close any currently open issues?

いいえ

How has this been tested?

Gazebo上で全てのサンプルが動作することを確認しています。

Any other comments?

箱(wood_cube_5cm)を掴むことを試みましたが、箱が弾けたりスリップしたり、と成功していません。

最適なモデルパラメータを見つけられたら、別PRで対応します。

Checklists

  • I have read the CONTRIBUTING guidelines.
  • I have checked to ensure there aren't other open Pull Requests for the same change.

@ShotaAk ShotaAk added the Type: Feature New Feature label Aug 5, 2021
@ShotaAk ShotaAk requested a review from nomumu August 5, 2021 08:11
@ShotaAk
Copy link
Author

ShotaAk commented Aug 26, 2021

このPRは、#29 がマージされる前提で作成しています。

#29 を取り入れた後に動作確認お願いします。

@nomumu
Copy link

nomumu commented Aug 26, 2021

手元でcrane_plus_gazeboをlaunchして動作することは確認できました。

@@ -2,18 +2,11 @@

CRANE+V2 のGazeboシミュレーションパッケージです。

**現在、Gazebo上のCRANE+V2を動かすことは出来ません**
Copy link

Choose a reason for hiding this comment

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

同様の文言がルートのReadme.mdにも残っているので編集をお願いします。

## ノードの起動

次のコマンドを実行するとGazeboが起動し、CRANE+V2モデルと、Table、Cubeが表示がされます
次のコマンドを実行するとGazeboが起動し、CRANE+V2モデルとTableが表示がされます
Copy link

Choose a reason for hiding this comment

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

Tableが表示がされます => Tableが表示されますに修正お願いします。


初回起動時はTableとCubeのモデルをダウンロードするため、モデルの表示に時間がかかることがあります。
初回起動時はTableのモデルをダウンロードするため、モデルの表示に時間がかかることがあります。

Copy link

Choose a reason for hiding this comment

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

CRANE-X7のReadmeに記載しているのと同様に、次のような一文を追加したいです。
実機との接続やdemo.launchの実行は必要ありません。

@nomumu
Copy link

nomumu commented Aug 27, 2021

修正ありがとうございました。
よいと思います。マージよろしくお願いします。

@ShotaAk ShotaAk merged commit 57c8e1d into master Aug 27, 2021
@ShotaAk ShotaAk deleted the crane_plus_gazebo_control branch August 27, 2021 05:02
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