Skip to content
This repository has been archived by the owner on Jan 28, 2023. It is now read-only.

Imprement vision wrapper #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

spiralray
Copy link
Contributor

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

@ShotaAk
Copy link
Contributor

ShotaAk commented Dec 26, 2019

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

@MiraiHattori
Copy link
Contributor

MiraiHattori commented Dec 27, 2019

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

@MiraiHattori
Copy link
Contributor

MiraiHattori commented Dec 27, 2019

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

@ShotaAk
Copy link
Contributor

ShotaAk commented Dec 30, 2019

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

@ShotaAk ShotaAk mentioned this pull request Jul 23, 2020
@spiralray spiralray force-pushed the dev/vision-wrapper branch from 93ba48c to dc21b99 Compare August 9, 2020 01:25
@spiralray spiralray changed the title Dev/vision wrapper Imprement vision wrapper Aug 9, 2020
@ShotaAk
Copy link
Contributor

ShotaAk commented Aug 9, 2020

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

Pose2D廃止問題どうします?・・・
#67

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

変更箇所はここかな?

geometry_msgs/Pose2D pose

geometry_msgs/Pose2D pose

consai2r2_msgs::msg::DetectionBall convert_to_ball_topic(const SSL_DetectionBall & raw_ball)
{
auto detection_ball = consai2r2_msgs::msg::DetectionBall();
detection_ball.pose.x = raw_ball.x() * TO_METER;
detection_ball.pose.y = raw_ball.y() * TO_METER;
return detection_ball;
}
consai2r2_msgs::msg::DetectionRobot convert_to_robot_topic(const SSL_DetectionRobot & raw_robot)
{
auto detection_robot = consai2r2_msgs::msg::DetectionRobot();
detection_robot.robot_id = raw_robot.robot_id();
detection_robot.pose.x = raw_robot.x() * TO_METER;
detection_robot.pose.y = raw_robot.y() * TO_METER;
detection_robot.pose.theta = raw_robot.orientation();
return detection_robot;
}

@spiralray
Copy link
Contributor Author

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

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

@ShotaAk
Copy link
Contributor

ShotaAk commented Aug 9, 2020

ありがとうございます。

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

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants