search

SSL-Roots / consai2r2

https://github.com/SSL-Roots/consai_ros2 へ開発を移行しています。
MIT License
8 stars 7 forks source link

sender with yaml config #62

Closed MiraiHattori closed 4 years ago

MiraiHattori commented 4 years ago

https://github.com/SSL-Roots/consai2r2/issues/3

いつかsim_sender.cppをsim_sender_node.cppに，sim_senderをsim_sender_nodeに改名したい気持ちがあるのですが，diffが見づらいのでこれだけPRを出しておきます．

MiraiHattori commented 4 years ago

https://github.com/SSL-Roots/consai2r2/issues/8 についても，READMEを追加しました． consai2からの移植という意味で，sim_sender.cppにはコメントを追加していません．

ShotaAk commented 4 years ago

対応ありがとうございます。レビューします。

末尾に_nodeを付けないとNGなんでしたっけ？（素朴な疑問）

ShotaAk commented 4 years ago

動作チェックしました。 コメントの確認をお願いします。

MiraiHattori commented 4 years ago

対応ありがとうございます。レビューします。

末尾に_nodeを付けないとNGなんでしたっけ？（素朴な疑問）

これに関しては，規定はないと思っています． ただ，「rclcpp::Nodeを継承したクラスを定義した.hppに対応する，実装を書いたcppファイル」と「.hppに書かれたクラスを使用してmain関数でループを回すcppファイル」がファイル名レベルで分かれていたほうが可読性が高いのではないかと思っています． https://github.com/ros2/examples/tree/master/rclcpp/minimal_composition/src がそのような運用になっていそうです．

MiraiHattori commented 4 years ago

[追記]よくみたところ，実際のところは逆で，main関数がある方は_nodeではないようです．．．よって，_nodeはいらないというのが正しそうです

MiraiHattori commented 4 years ago

修正してpushしました．

ShotaAk commented 4 years ago

確認しました。マージします。 ノード名に関しては、また気づきがあったら適宜修正しましょう。