search

SSL-Roots / consai2r2

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

Use rosparam on SimSender #23

Closed spiralray closed 4 years ago

spiralray commented 4 years ago

フォーマッタかけたので変更が多く見えますが、実際に変わったのは2個目のコミットの方だけです。

spiralray commented 4 years ago

ROS1実装ではパラメータは

したがって

spiralray commented 4 years ago

This pull request is related to #3 .

ShotaAk commented 4 years ago

レビュー始めました。

ShotaAk commented 4 years ago

動作確認しました。 launchファイルを編集してgrsim_addr, grsim_portが機能することも確認しました。

lintは気にせずマージします

ShotaAk commented 4 years ago

cpplint通っててナイスです

HansRobo commented 4 years ago

ROS1実装ではパラメータは

  • consai2_description/grsim_addr
  • consai2_description/grsim_port となっていましたが、ROS2ではroscoreの廃止に伴い、上記のようなにパラメータ名で実装することができなくなっています。

したがって

  • ~/grsim_addr
  • ~/grsim_port として実装しました。

consai2r2_descriptionでパラメータ実装したので #33 がマージされたらそちらに移行するようにissueを立てる必要がありますね