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

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

Merged
merged 3 commits into from
Jan 27, 2023

Conversation

kochigami
Copy link
Contributor

@kochigami kochigami commented Dec 20, 2022

  • NAOqi OS 2.5のPepperとROS melodicはかみ合わせが悪いことを追記しました。
    (ROS melodic以上が使えるのはNAOqi OS 2.9以上とのこと)

  • wstoolを使ってインストールされるパッケージの情報を更新しました(主にnaoqi_dashboard)。

@kochigami kochigami changed the title [jsk_naoqi_robot/README] update output of wstool info because naoqi_dashboard is updated [jsk_naoqi_robot/README] update output of wstool info because naoqi_dashboard is updated & add a note about NAOqi version and ROS version Dec 21, 2022
@kochigami kochigami marked this pull request as ready for review December 22, 2022 07:03
@k-okada
Copy link
Member

k-okada commented Jan 24, 2023

naoqi_dashboard は

local-name: naoqi_dashboard
uri: https://github.com/kochigami/naoqi_dashboard
version: kochigami-develop

- git:
local-name: naoqi_driver
uri: https://github.com/kochigami/naoqi_driver
version: kochigami-develop

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

@kochigami
Copy link
Contributor Author

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

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

local-name: naoqi_dashboard
uri: https://github.com/kochigami/naoqi_dashboard
version: kochigami-develop

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

これは、NAOqi 2.9の対応ではなく、
NAOqi 2.5・ROS melodicでnaoqi_dashboardを動かすための対応だったと思います。
関連issue:
#1542 (comment)
k-okada#73 (comment)

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

@k-okada
Copy link
Member

k-okada commented Jan 25, 2023

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

@kochigami
Copy link
Contributor Author

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

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

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

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

@k-okada k-okada merged commit 6743071 into jsk-ros-pkg:master Jan 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants