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

RealSense D435搭載モデルに対応 #164

Merged
merged 8 commits into from
Nov 15, 2022
Merged

RealSense D435搭載モデルに対応 #164

merged 8 commits into from
Nov 15, 2022

Conversation

Kuwamai
Copy link
Contributor

@Kuwamai Kuwamai commented Nov 10, 2022

What does this implement/fix?

RealSense D435搭載モデルに対応するためにdemo.launch起動時の引数use_d435を追加しました。
use_d435:=true時にRealSenseが起動し、ロボットモデルにcamera_linkが追加されます。

Does this close any currently open issues?

しません

How has this been tested?

下記コマンドを実行してRealSenseの起動とロボットモデルにcamera_linkが追加、RVizで点群が可視化されることを確認しています。

ros2 launch crane_x7_examples demo.launch.py use_d435:=true

Gazeboでのカメラのシミュレーションは未対応のためモデルとフレームだけ追加されます。

ros2 launch crane_x7_gazebo crane_x7_with_table.launch.py use_d435:=true

Any other comments?

Checklists

@Kuwamai Kuwamai self-assigned this Nov 10, 2022
@Kuwamai Kuwamai added the Type: Feature New Feature label Nov 10, 2022
@Kuwamai Kuwamai requested a review from ShotaAk November 10, 2022 10:05
@Kuwamai Kuwamai marked this pull request as ready for review November 10, 2022 10:05
@ShotaAk
Copy link
Contributor

ShotaAk commented Nov 11, 2022

gazeboを起動したときrealsenseのdaeファイルが読み込まれませんでした。
確認お願いします。

もしかしたら、IGN_GAZEBO_RESOURCE_PATHにrealsense2_descriptionのパスを追加しなければならない可能性あります。

@Kuwamai
Copy link
Contributor Author

Kuwamai commented Nov 11, 2022

gazeboを起動したときrealsenseのdaeファイルが読み込まれませんでした。 確認お願いします。

もしかしたら、IGN_GAZEBO_RESOURCE_PATHにrealsense2_descriptionのパスを追加しなければならない可能性あります。

Gazebo Citadelではframe_idの設定ができないためD435搭載モデルの対応は実機のみにします。また、ためしにD435のモデルを表示したところCOLLADAが透明に表示される現象を確認しました。Humble対応の際改めて調査します。
image

@ShotaAk ShotaAk added the ROS 2 label Nov 15, 2022
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.

見ました。良いと思います。

1箇所だけ修正お願いします。

crane_x7_examples/README.md Outdated Show resolved Hide resolved
@@ -51,6 +58,7 @@ def generate_launch_description():
description_loader = RobotDescriptionLoader()
description_loader.port_name = LaunchConfiguration('port_name')
description_loader.baudrate = LaunchConfiguration('baudrate')
description_loader.use_d435 = LaunchConfiguration('use_d435')
Copy link
Contributor

Choose a reason for hiding this comment

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

今回は対応しませんが、demo.launch.pyで生成されるrobot_descriptionと
example.launch.pyで生成されるrobot_descriptionが別物になるので、注意が必要です。
サンプルのmove_groupノードが扱うrobot_descriptionはカメラなしのものになります。

ただ、実運用ではtf2を使ってrobot_descriptionトピックをもとに座標変換するので
カメラ画像から目標位置を参照するような実装においては特に問題発生しないです。

exampleノードがrobot_descriptionトピックをsubscribeできれば解決するかもしれませんが、
作業優先度は低いです。

Co-authored-by: Shota Aoki <s.aoki@rt-net.jp>
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 c809856 into ros2 Nov 15, 2022
@ShotaAk ShotaAk deleted the add_d435 branch November 15, 2022 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants