jsk-ros-pkg / jsk_pr2eus

PR2 euslisp packages
https://github.com/jsk-ros-pkg/jsk_pr2eus
4 stars 41 forks source link

add screenpoint example #436

Closed k-okada closed 4 years ago

k-okada commented 4 years ago

topicを使ってPointcloudScreenpointを呼ぶためのサンプルプログラムについてもプルリクエストを出したので、参考になれば嬉しいです。 https://github.com/MiyabiTane/rtmros_tutorials/pull/2

これ,めちゃめちゃ便利&必要だと思うので,https://github.com/jsk-ros-pkg/jsk_pr2eus/blob/master/pr2eus_tutorials/launch/pr2_tabletop_grasp.launch でimage_view2をクリックしたら,その位置をgraspするモードも作ってくれると嬉しいです.

708yamaguchi commented 4 years ago

image_view2を使ったインタラクティブなgraspモードを実装してみました。 https://github.com/jsk-ros-pkg/jsk_pr2eus/pull/437

knorth55 commented 4 years ago

コメントが遅くなりましたがこのサンプルでは元々Rvizのバウンディングボックスを選択するとPR2が掴むデモになっています そちらを壊さないように実装してほしいです

knorth55 commented 4 years ago

勘違いしていました。 reviewでもかきましたがargumentで分けるのではなく、常にrvizでもimage view2でもできるようにどちらも起動すればいいとおもいます。 全て一つでできるlaunchのほうpr2_tabletop_grasp.launchのほうを使うようにオススメするのであれば、今のreadmeのeus, rvizを順番で自分の手で立ち上げる項目は消した方がわかりやすいと思います。 ただ個人的な趣味で、動作実行、可視化を同じlaunchで立ち上げるのが不便(gazeboや実機が突然動き出す)ので別にしていました。 あと普通に実機のほうでは画像処理ノードをPR2体内で、eus, rvizはローカルPCで立ち上げることが想定されているので、pr2_tabletop_grasp.launchの使いどころはないと思います。

708yamaguchi commented 4 years ago

ありがとうございます。 以下の方針でプルリクエストを修正しました。 https://github.com/jsk-ros-pkg/jsk_pr2eus/pull/437