jsk-ros-pkg / jsk_robot

jsk-ros-pkg/jsk_robot
https://github.com/jsk-ros-pkg/jsk_robot
73 stars 97 forks source link

[jsk_naoqi_robot/README] update output of wstool info because naoqi_dashboard is updated & add a note about NAOqi version and ROS version #1768

Closed kochigami closed 1 year ago

kochigami commented 1 year ago
k-okada commented 1 year ago

naoqi_dashboard は https://github.com/jsk-ros-pkg/jsk_robot/blob/22bd62a84a4dd2df6b6ac99b74e69230d207f24f/jsk_naoqi_robot/naoqi.rosinstall#L7-L9

https://github.com/jsk-ros-pkg/jsk_robot/blob/22bd62a84a4dd2df6b6ac99b74e69230d207f24f/jsk_naoqi_robot/naoqi.rosinstall#L23-L26

でインストールされるという認識なんだけど,これは https://github.com/kochigami/naoqi_dashboard/commits/kochigami-develop https://github.com/kochigami/naoqi_driver/commits/kochigami-develop の何処かのコミットで naoqi 2.9 対応されたという理解で良いでしょうか?

kochigami commented 1 year ago

ご指摘ありがとうございます。

naoqi_dashboard は以下でインストールされます。

https://github.com/jsk-ros-pkg/jsk_robot/blob/22bd62a84a4dd2df6b6ac99b74e69230d207f24f/jsk_naoqi_robot/naoqi.rosinstall#L7-L9

以下の変更 https://github.com/jsk-ros-pkg/jsk_robot/pull/1768/files#diff-375a422594be02bff5f6011ade16d89082a28a0fa467e818956252c15d831a5aR140 は、 https://github.com/kochigami/naoqi_dashboard/commits/kochigami-develop の https://github.com/kochigami/naoqi_dashboard/commit/7f32005e08e0e76475af30263e77e6fb84f35060 の変更を反映したものになっています。

これは、NAOqi 2.9の対応ではなく、 NAOqi 2.5・ROS melodicでnaoqi_dashboardを動かすための対応だったと思います。 関連issue: https://github.com/jsk-ros-pkg/jsk_robot/pull/1542#issuecomment-1192529540 https://github.com/k-okada/jsk_robot/pull/73#issuecomment-1214331696

ただ、以下で同様のプルリクエストが出されていて、閉じられているというのも見つけました:https://github.com/kochigami/naoqi_dashboard/pull/7 https://github.com/kochigami/naoqi_dashboard/commit/7f32005e08e0e76475af30263e77e6fb84f35060 は、このプルリクエストが閉じられた後に、自分の方で追加しているようでした。 その点少し謎なのですが、必要な変更なのではないかと思います。

k-okada commented 1 year ago

@kochigami なるほど,そういうことか.了解です.ちなみにNaoqi2.9のロボットっているのかな?PepeprのEducation版も2.5なんだよね?

kochigami commented 1 year ago

ちなみにNaoqi2.9のロボットっているのかな?PepeprのEducation版も2.5なんだよね?

NAOqi 2.9のロボット(2.9になりうるのはPepperだけだと思います)は研究室にはいなくて、 研究室にいる・これまでお借りしたPepperのバージョンは全て2.5です。

そのため、2.9でnaoqi_dashboardのプログラムが動くかは未確認です。

よろしくお願いいたします。