search

jsk-ros-pkg / jsk_control

jsk control ros packages
http://github.com/jsk-ros-pkg/jsk_control
14 stars 51 forks source link

[stabilizer_watcher.py]fix for kinetic #724

Closed Naoki-Hiraoka closed 5 years ago

Naoki-Hiraoka commented 5 years ago

robotsoundが

indigoからkineticに変わるときにSoundRequest型が変わり、 新たにvolumeというパラメータができました。

デフォルト値では音量0となってロボットが喋らないため、 volumeに値をセットしました。

k-okada commented 5 years ago

thank you! can you add code to check if the soundrequest has the volume attribute like https://github.com/jsk-ros-pkg/jsk_pr2eus/blob/fe214ce00e05b4384ee4b6d20ba469f29a894a3c/pr2eus/test/speak-test.py#L40 ?

we may also need to update following codes too https://github.com/jsk-ros-pkg/jsk_3rdparty/search?q=SoundRequest&unscoped_q=SoundRequest https://github.com/jsk-ros-pkg/jsk_common/search?q=SoundRequest&unscoped_q=SoundRequest https://github.com/jsk-ros-pkg/jsk_pr2eus/search?q=SoundRequest&unscoped_q=SoundRequest

◉ Kei Okada

2019年7月17日(水) 21:56 Naoki Hiraoka notifications@github.com:

robotsoundが

indigoからkineticに変わるときにSoundRequest型が変わり、 新たにvolumeというパラメータができました。

デフォルト値では音量0となってロボットが喋らないため、 volumeに値をセットしました。

You can view, comment on, or merge this pull request online at:

https://github.com/jsk-ros-pkg/jsk_control/pull/724 Commit Summary

  • [stabilizer_watcher.py]fix for kinetic

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jsk-ros-pkg/jsk_control/pull/724?email_source=notifications&email_token=AADYNXA2B7QTPDSKBSVGUQLP74JJFA5CNFSM4IEQB27KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4G7XCSUA, or mute the thread https://github.com/notifications/unsubscribe-auth/AADYNXF6WU4O24OASIG4DA3P74JJFANCNFSM4IEQB27A .

mmurooka commented 5 years ago

あれ，もしかしてindigoでこれを実行すると，sound.volumeというメンバはないとエラーになるでしょうか．そうだとしたらマージしたらいけなかった．

itohdak commented 5 years ago

エラーになる気がします．．

mmurooka commented 5 years ago

can you add code to check if the soundrequest has the volume attribute like https://github.com/jsk-ros-pkg/jsk_pr2eus/blob/fe214ce00e05b4384ee4b6d20ba469f29a894a3c/pr2eus/test/speak-test.py#L40

については，責任とって対応しました： https://github.com/jsk-ros-pkg/jsk_control/pull/726

k-okada commented 5 years ago

自分の書いていないコードも自分の書いたコードのように大切にできるとよいですね．ということで，気がついたところは直しておいたので，確認しておいてください．