rt-net / crane_plus

CRANE+ V2 ROS 2 Packages
Apache License 2.0
44 stars 8 forks source link

MoveItの手先をjoint4からグリッパの中心へ変更 #79

Closed y-masutani closed 2 months ago

y-masutani commented 3 months ago

What does this implement/fix?

MoveItのIKの対象(アームの先端)をjoint4からグリッパの中心へ変更します.姿勢も変更しています.

Does this close any currently open issues?

いいえ

How has this been tested?

Any other comments?

Checklists

Kuwamai commented 2 months ago

@y-masutani 追加いただいたリンク位置はlink4から9 cm飛び出した位置にありますが、この位置に設定した理由はありますでしょうか。

y-masutani commented 2 months ago

9㎝の理由ですが,500mLのペットボトル把持すると,その中心がlink4から約9㎝になるので,その値に決めました.それだけのことです.Issuesでご指摘があったように,対象や作業が変わればオフセット長は変わります.xacroの引数として,オフセットを設定できるようにするのがいいのではないかと考えています.

Kuwamai commented 2 months ago

承知いたしました。サンプルコードが5 cm角のキューブを掴む内容のため、適切なデフォルト値については社内でも確認させていただきます。

y-masutani commented 2 months ago

レビューを反映して変更しました.確認をお願いいたします.

y-masutani commented 2 months ago

RViz内で「arm_tcp」を選択するとインタラクティブマーカが2つ表示されてしまいます.この問題が解決できていません.

y-masutani commented 2 months ago

link4からTCPへオフセット(位置と姿勢)をローンチファイル内で指定できるようにしました.

y-masutani commented 2 months ago

crane_plus.urdf.xacro の引数でオフセットを指定しなければ,tcpはlink4と一致しますので,arm_tcpグループを作らなくても,過去との互換性を保つことができそうです.検討をお願いいたします.

Kuwamai commented 2 months ago

link4からTCPへオフセット(位置と姿勢)をローンチファイル内で指定できるようにしました.

現状TCPリンクを用途に応じて変更するサンプル等がない点と、今後のメンテナンスを考え、リンク位置姿勢を変更するxacro引数の追加は今回のPull requestでは行わない方がよいと考えています。 その他の課題も含めて来週返答させていただきます。

y-masutani commented 2 months ago

何度もレビューしていただいた後ですが(勉強になりました),このリポジトリをフォークした先で独自にやることにします. https://github.com/AI-Robot-Book-Humble/crane_plus/tree/airobot したがいまして,このプルリクエストはクローズします.

Kuwamai commented 2 months ago

@y-masutani 承知いたしました。TCPリンクやグループの追加に関しては改めてこちらでも対応させていただきます。 ご提案、ご対応ありがとうございました。