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

Gazebo環境にカメラを追加 #83

Merged
merged 11 commits into from
Dec 3, 2024
Merged

Conversation

Kuwamai
Copy link
Contributor

@Kuwamai Kuwamai commented Nov 28, 2024

What does this implement/fix?

Gazebo環境にカメラを追加しています。
ArUcoマーカ検出、色検出サンプルが動作するためにArUcoマーカ付きの立方体を追加しています。

それぞれのサンプルが動作することと、ドキュメントの内容をご確認お願いします。

Does this close any currently open issues?

しません

How has this been tested?

Gazeboを起動し、ArUcoマーカ検出、色検出サンプルが動作することを確認しました。
シミュレータの起動方法はcrane_plus_gazebo、crane_plus_examplesのREADMEに記載しています。

Any other comments?

ありません

Checklists

@Kuwamai Kuwamai added the Type: Feature New Feature label Nov 28, 2024
@Kuwamai Kuwamai requested a review from YusukeKato November 28, 2024 03:50
@Kuwamai Kuwamai self-assigned this Nov 28, 2024
@YusukeKato
Copy link
Collaborator

YusukeKato commented Nov 28, 2024

Gazebo上でArUcoマーカ検出と色検出の動作確認できました!
その他のサンプルも動作確認済みです。

共有事項として、サンプルプログラム(ExamplesとCamera Examplesどちらも)実行時に下記のような警告が出ます。
(以前からあった警告なのかは不明)

[pick_and_place_tf-2] [WARN] [1732778763.558544317] [rcl.logging_rosout]: Publisher already registered for provided node name. If this is due to multiple nodes with the same name then all logs for that logger name will go out over the existing publisher. As soon as any node with that name is destructed it will unregister the publisher, preventing any further logs for that name from being published on the rosout topic.
[pick_and_place_tf-2] [WARN] [1732778763.576075154] [rcl.logging_rosout]: Publisher already registered for provided node name. If this is due to multiple nodes with the same name then all logs for that logger name will go out over the existing publisher. As soon as any node with that name is destructed it will unregister the publisher, preventing any further logs for that name from being published on the rosout topic.
[pick_and_place_tf-2] Warning: Link 'camera_stand_p1' is not known to URDF. Cannot disable/enable collisons.
[pick_and_place_tf-2]          at line 594 in ./src/model.cpp
[pick_and_place_tf-2] Warning: Link 'camera_stand_p4' is not known to URDF. Cannot disable/enable collisons.
[pick_and_place_tf-2]          at line 594 in ./src/model.cpp
[pick_and_place_tf-2] Warning: Link 'camera_stand_p1' is not known to URDF. Cannot disable/enable collisons.
[pick_and_place_tf-2]          at line 589 in ./src/model.cpp
[pick_and_place_tf-2] Warning: Link 'camera_stand_p1' is not known to URDF. Cannot disable/enable collisons.
[pick_and_place_tf-2]          at line 589 in ./src/model.cpp
[pick_and_place_tf-2] Warning: Link 'camera_stand_p1' is not known to URDF. Cannot disable/enable collisons.
[pick_and_place_tf-2]          at line 589 in ./src/model.cpp
[pick_and_place_tf-2] Warning: Link 'camera_stand_p2' is not known to URDF. Cannot disable/enable collisons.
[pick_and_place_tf-2]          at line 589 in ./src/model.cpp

@Kuwamai
Copy link
Contributor Author

Kuwamai commented Nov 28, 2024

@YusukeKato ありがとうございます。これで表示されなくなったかと思います。

@Kuwamai
Copy link
Contributor Author

Kuwamai commented Nov 28, 2024

example.launch.py、camera_example.lauch.py両方にuse_cameraオプションを追加してみました。
カメラ付きモデルで起動した状態でカメラを使わないサンプルを実行する際も、サンプル起動時にuse_cameraをtrueにすることで警告が表示されなくなります。
引数の指定は必須ではないのでREADMEには書かなくて大丈夫かと思っています。

@YusukeKato
Copy link
Collaborator

@Kuwamai 修正ありがとうございます。
サンプル実行時にuse_cameraをtrueにすることで警告が出なくなることを確認できました。

@Kuwamai
Copy link
Contributor Author

Kuwamai commented Nov 29, 2024

上記エラーはURDFにないリンクをSRDFが参照していることによるものでした。
SRDFには干渉の除外設定しか記述していないので、もともとのURDFを干渉しない形状にすると一番シンプルに解決できそうです。

@YusukeKato
Copy link
Collaborator

サンプル実行時にuse_cameraをtrueにしなくても下記コメントの警告が出ないことを確認しました。
#83 (comment)

@YusukeKato
Copy link
Collaborator

LGTMです!

@YusukeKato YusukeKato merged commit 2f0964e into master Dec 3, 2024
2 checks passed
@YusukeKato YusukeKato deleted the feature/add-gazebo-camera branch December 3, 2024 01:54
Kuwamai added a commit that referenced this pull request Dec 3, 2024
* カメラ追加

* use_sim_timeオプション追加

* カメラの画角調整

* 色検出サンプルの色しきい値調整

* 把持位置調整

* 現バージョンのデフォルト設定を反映

* ArUcoマーカ付きの立方体を追加

* ワールド、launchファイル追加

* use_cameraの有効化

* use_cameraを引数で切り替えられるように変更

* カメラスタンドのURDF修正
YusukeKato pushed a commit that referenced this pull request Dec 3, 2024
* カメラ追加

* use_sim_timeオプション追加

* カメラの画角調整

* 色検出サンプルの色しきい値調整

* 把持位置調整

* 現バージョンのデフォルト設定を反映

* ArUcoマーカ付きの立方体を追加

* ワールド、launchファイル追加

* use_cameraの有効化

* use_cameraを引数で切り替えられるように変更

* カメラスタンドのURDF修正
mizonon pushed a commit that referenced this pull request Dec 4, 2024
* カメラ追加

* use_sim_timeオプション追加

* カメラの画角調整

* 色検出サンプルの色しきい値調整

* 把持位置調整

* 現バージョンのデフォルト設定を反映

* ArUcoマーカ付きの立方体を追加

* ワールド、launchファイル追加

* use_cameraの有効化

* use_cameraを引数で切り替えられるように変更

* カメラスタンドのURDF修正
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