Closed yoshikai-qt closed 1 week ago
yoshikai-qt
commented
2 weeks ago PR作成後に、エラーコードのtitleがユニークでないことに気がついたので、diagnostics_msg/msg/DiagnosticStatusのnameにセットするものとしてcodeをstringにしたものをセットするように変更しました。それに伴い、messageにはtitle_enをセットするように変更しました。 よろしくお願い致します。
yoshikai-qt
commented
1 week ago @hidai-pfr ご確認ありがとうございました。
私が Diagnostic の仕様について詳しくないのですが、カチャカの GetError をそのまま ROS2 の Diagnostic に変換して、ROS 側の仕様的には問題無さそうでしょうか？
ここは、悩ましいところで、独自トピックによる出力も合わせて検討していましたが、社内でも議論してやはりエラー情報ということでDiagnosticにまとめたほうがよいだろうという結論に今の所なっております。どうぞ、よろしくお願い致します。
hidai-pfr
commented
1 week ago もし Diagnostic として使っていく中で ROS 側の仕様との齟齬のようなものがあれば、その都度修正していけばよいと思います。こういった Contribution は歓迎ですので、今後ともよろしくお願いします。
ROS2にてエラー情報をdiagnostics_msgs/msg/DiagnosticArrayで出力するようにしました。 ./tools/lint/local_run.shは通してあり、今回の変更部分についてclangのチェックは通っている状態です。