Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

現在のorne_navigationの整理,不具合の改善をする #531

Open
haruyama8940 opened this issue Apr 11, 2022 · 11 comments
Open

現在のorne_navigationの整理,不具合の改善をする #531

haruyama8940 opened this issue Apr 11, 2022 · 11 comments

Comments

@haruyama8940
Copy link

現在のorne_navigationの不具合の洗い出しと必要なパッケージの選択を行う.
調査しながら追記.
対象はnoeticを想定.

@uhobeike
Copy link
Member

パラメータは機体ごとにファルダ分けしたほうが良さそう。
共通なやつは、分けないでおくか、commonフォルダにまとめる。

@haruyama8940
Copy link
Author

haruyama8940 commented Apr 12, 2022

ありがとうございます.
フォルダは機体ごとに分けるでいいと思います.
それではまず初めにparamとlaunchを整理しようと思います.
構造として
-param
--alpha
--gamma

-launch
--common
--alpha
--gamma
とするのはどうでしょうか.
paramに関しては機体ごとに完全に分離してしまった方がいいと思います.

@yasuohayashibara
Copy link
Member

ファイル構造に関しては前例に倣ったほうが良いと思います.
探したのですが,複数のロボットを一つのパッケージで扱うサンプルが見つかりませんでしたので,できればメジャーなパッケージがどのような構造を選択しているかが分かる例があればお願いします.

@uhobeike
Copy link
Member

uhobeike commented Apr 18, 2022

turtlebot3だとロボットごとにフォルダは分けておらず、環境変数として
burger, waffle, waffle_piのどれかをセットしておくことでファイルの呼び出しを行っています。
https://github.com/ROBOTIS-GIT/turtlebot3/tree/master/turtlebot3_navigation/param

https://github.com/ROBOTIS-GIT/turtlebot3/blob/66681b33749c44e7d9022253ac210ef2da7843a0/turtlebot3_navigation/launch/move_base.launch#L3

https://github.com/ROBOTIS-GIT/turtlebot3/blob/66681b33749c44e7d9022253ac210ef2da7843a0/turtlebot3_navigation/launch/move_base.launch#L11-L12

探したのですが,複数のロボットを一つのパッケージで扱うサンプルが見つかりませんでしたので,できればメジャーなパッケージがどのような構造を選択しているかが分かる例があればお願いします.

前例やメジャーなパッケージに従うことは良いと思うのですが、
手の入れやすさや管理のことを考えるとフォルダごとに分けるのはありなのかなと思っています。

私は普段orne_navigationを使用しているわけではないため、あんまり言える立場では無いのですが
使用している林原研の方々はフォルダごとに分けたほうが開発がしやすいのか気になります。

@masakifujiwara1
Copy link
Member

現状, param内の共通なものでもロボットごとに変更することがあります.
共通のものも含めて, ロボットごとにフォルダが分かれている方が扱いやすいです.

@haruyama8940
Copy link
Author

ブランチを変えて作業します

@uhobeike
Copy link
Member

ブランチを変えて作業します

https://github.com/open-rdc/orne_navigation/tree/refactor/melodic-devel

@haruyama8940
Copy link
Author

ファイルの整理中です
改善
・base_footprintを追加する
・デフォルトだとlocal_costmapが機能しない
・alphaのurgの設定がドキュメント化されてない
・urdfの見直し

@haruyama8940
Copy link
Author

haruyama8940 commented Sep 30, 2022

ある程度ファイルの整理を行いました
その過程でicartパッケージからboxと同じくypspur_rosに移行しました
https://github.com/open-rdc/orne_navigation/tree/refactor/noetic-devel
現在は動作確認およびデバッグ中です.

@yasuohayashibara
Copy link
Member

こちらPRを投げていただき,差分を見ながら議論したほうが良いと思います.
あと小分けにして優先度の高いものからマージしたほうが良いと思います.
あと,個別には話しましたが,iCartをできるだけ利用したほうが良いと思います.
問題があるようでしたら林原の方でも調査しますので,issueを立てていただければと思います.

@haruyama8940
Copy link
Author

こちらの作業は冬休みを利用して議論,作業したいと考えています.

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

No branches or pull requests

4 participants