SSL-Roots / consai2r2

https://github.com/SSL-Roots/consai_ros2 へ開発を移行しています。

MIT License

8 stars 7 forks source link

Closed spiralray closed 4 years ago

spiralray commented 4 years ago

11

ros2 topic echoしている感じでは動いているとは思うのですが、Wrapperがまだないため動作確認が不十分です。

ShotaAk commented 4 years ago

遅くなりましたが、レビュー開始です。

ShotaAk commented 4 years ago

refereeの変更が差分として表示されてるので、最新のmasterをdev/vision-receiverブランチにpullして再pushお願いします。

ShotaAk commented 4 years ago

geometry_msgs/Pose2 非推奨問題ですが、別途issueを立てて処置しましょう。 (このPRにクォータニオンが入ると脳がパンクします）

ShotaAk commented 4 years ago

不要なmsgファイルが複数あるので、削除お願いします。（ブランチを間違えてコミットしたように見えます）

このPRはvision_receiver追加のみでお願いします。

spiralray commented 4 years ago

修正いたしました。ただし、rebaseした影響で他PRに影響が出る可能性が非常に大きいです。

の通り、可能であれば #47 にて一括して確認いただけますと幸いです。

ShotaAk commented 4 years ago

むむ

ShotaAk commented 4 years ago

この手順だとダメですかね・・・？（もう手遅れか・・・） https://qiita.com/puttyo_bubu/items/691681f3f7629cf2d3ea

spiralray commented 4 years ago

定時いただいた方法にて再度pushしました。ご確認よろしくお願い致します。