search

SSL-Roots / consai2r2

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

Imprement vision wrapper #55

Open spiralray opened 4 years ago

spiralray commented 4 years ago

conflict解消したverで再度PRします。

ShotaAk commented 4 years ago

対応ありがとうございます。 Projectの1st stepが終わるまで、レビューを保留させてください :bowing_man:

MiraiHattori commented 4 years ago

お疲れ様です & ありがとうございます． consai2からconsai2r2のportingの一部として ~description~ wrapperを移行する必要があると思っていたのですが， このPRとは別の方法で ~description~ wrapperを移行する感じになりそうでしょうか?

MiraiHattori commented 4 years ago

ごめんなさい，勘違いしていました． portingにかかれているexampleディレクトリをconsai2_examplesと混同していました

ShotaAk commented 4 years ago

@future731 勘違いしやすい文章だったので訂正しました。 https://github.com/SSL-Roots/consai2r2/projects

ShotaAk commented 3 years ago

どもども。確認にちょっと時間ください:eyes: マージする方向で、動かしてレビューします。

Pose2D廃止問題どうします？・・・ https://github.com/SSL-Roots/consai2r2/issues/67

直すならこのPRマージ前に、別PRで直してしまいたいな〜と思います。 （直さないとFoxyで動かないのかもしれませんが）

変更箇所はここかな？ https://github.com/SSL-Roots/consai2r2/blob/83f288e9c3f59d6870b807dac526fe906d9c1e1f/consai2r2_msgs/msg/DetectionRobot.msg#L2

https://github.com/SSL-Roots/consai2r2/blob/83f288e9c3f59d6870b807dac526fe906d9c1e1f/consai2r2_msgs/msg/DetectionRobot.msg#L2

https://github.com/SSL-Roots/consai2r2/blob/83f288e9c3f59d6870b807dac526fe906d9c1e1f/consai2r2_receiver/src/vision_receiver.cpp#L80-L100

spiralray commented 3 years ago

このプロジェクトはconsai2からの移行という前提を考えると、個人的にはまずはPose2Dで一通り実装してからPose3Dに変更したほうが良いかな、と思います。 (v.0.1.0としてPose2D型を用いたconsai2r2をリリースするべきだと思います)

FoxyでもPose2Dのメッセージ型自体はまだ残っています。 「基本的には使用禁止で、今後のバージョンでは削除される可能性はある」という扱いです。

ShotaAk commented 3 years ago

ありがとうございます。

v.0.1.0としてPose2D型を用いたconsai2r2をリリースするべき

この方針でいきましょう。