add rwt_voice_message package

[furushchev]

#21 に従いrwt_voice_messageの追加を申請します。

[130s]

@garaemon review 御願いします．

package 名をもうちょっと specific にできると思いました．rwt_speech_topic とか．

[furushchev]

パッケージ名とデフォルトのトピック名は、同じ機能を有するandroidアプリの"android-voice-message"に合わせる形にしてありました。
https://play.google.com/store/apps/details?id=org.ros.android.androidVoiceMessage&hl=ja

[garaemon]

use ' for string

[garaemon]

コメントしましたが

• jsの文字列は'を使う
• できればローカルのjsを読むように
• ハードタブではなくソフトタブ
  でおねがいします

[k-okada]

android_voice_messageへの依存はVoiceMessage.msg かな？いずれにせよStringの配列を入れてもらえないかダメ元でpull reqしてみました．
ros/std_msgs#5

[furushchev]

プルリクエストの可否はcommon_msgsのissueが終わってからということになるのでしょうか？

[k-okada]

わすれていた．
https://github.com/ros/common_msgs/pulls
にStringArray.msg を投げてくれると助かります．どうなるか見てみましょう．

[130s]

パッケージ名とデフォルトのトピック名は、同じ機能を有するandroidアプリの"android-voice-message"に合わせる形にしてありました。
https://play.google.com/store/apps/details?id=org.ros.android.androidVoiceMessage&hl=ja

なるほど．では私は異論無しです．

ROS なので rwt_speech_topic とか rwt_voice_topicとか，ROS topic を吐き出すんだよ，というのが明確な方がより specific と思いましたが，オリジナルが存在するならそれに準ずるのはアリですね．

[k-okada]

にStringArray.msg を投げてくれると助かります．どうなるか見てみましょう．

pull requestしてみましょう．という意味でしたが，そもそも何処にどうやっていれたらいいか，いい案は無いということかな．
そうなると，そうかもしれない．やっぱstd_msgsにStringArray.msg があるほうがいいか．

もうひとつはcommon_msgsにgui_msgsをつくる，というものだけど，いい案を考えられないかな？ jsk_gui_msgs にあるものを整理して，それぞれ背景があればいいので（論文でいえばcitationできればよい），例えばAndroidAPIに準拠している，とか，そういう説明ができるようなgui_msgs をつくってpull requestしてみるというのはどうでしょうか？

[furushchev]

返信が遅れました。common_msgsにpull reqを送ってみました。 ros/common_msgs#32
このパッケージ自体のpull reqのマージは上記のpull reqが承認されたあとになるのでしょうか？

[k-okada]

そうですね．ros/common_msgs#32 の進捗をこのスレッドで更新してもらえると嬉しいです．

2014-04-07 12:05 GMT+09:00 Furushchev notifications@github.com:

返信が遅れました。common_msgsにpull reqを送ってみました。 ros/common_msgs#32ros/common_msgs#32
このパッケージ自体のpull reqのマージは上記のpull reqが承認されたあとになるのでしょうか？

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/23#issuecomment-39692786
.

[garaemon]

common_msgsから弾かれそうな予感もしますが... 進捗次第ではmsgの置きどころを考えましょう

[furushchev]

ros/common_msgs#32 の返信によると、jskのレポジトリにvoice_recognition_msgsなどを作り、しばらく様子を見て、common_msgsなどにプルリクエストを送るなどを検討することを考えることになりそうです。
jskのメタパッケージだとどこにおくのが正解でしょうか。

[k-okada]

jsk_commonにいれてください．

[garaemon]

jsk_commonにメッセージ追加のPRをください

[furushchev]

jsk_common にmsgを追加しました。jsk-ros-pkg/jsk_common#470
これに伴い、もともと怪しい名前だったので、msgをAPIに沿うように名前を変更しました。 #30