KBKN-Autonomous-Robotics-Lab / orange_ros2

This project is to use orange robot with ROS2
Apache License 2.0
12 stars 2 forks source link

add_waypoint_app_in_rosinstall #62

Closed shunki1006 closed 1 year ago

shunki1006 commented 1 year ago

概要

なぜこのタスクを行うのか

やったこと

できるようになること

その他

shunki1006 commented 1 year ago

orange_navigationでhumble-develというブランチを切って対応して頂いていますが、実際はROS2用というよりかはmainの内容からROS1用のパッケージを削除しただけだと思うので、ブランチ名が適当でない気がします。 また、humble-develのREADMEで以下のように記述されていますが、ROS2用のパッケージに変更したわけではないのでこれは誤解を生むと思います。

This repository provides ROS2 humble packages to support waypoint navigation and a Python project for editing waypoints, mainly for Tsukuba Challenge.

以下のSetupも、ROS1/ROS2用のパッケージは無いので不要です。

$ rosdep install -r -y -i --from-paths . $ colcon build

レビューお願いいたします。

shunki1006 commented 1 year ago

言葉足らずですみません。

ros2_humbleにはROS1もしくはROS2用のパッケージがあるわけではないので、ブランチ名をros2_humbleのようにあたかもROS2でしか使えないものとして限定すべきではないということです。極論、ros2_humbleの内容はwindowsやmac環境でも動くと思います。 なので、waypoint_app-develapp-develなどROSという単語を使う必要はありません。以下も同様です。

This branch provides the setup for using the waypoint editing application in ROS2.

承知しました. 再度レビューお願いいたします.