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

[WIP] Fast go to kitchen #126

Merged

Conversation

708yamaguchi
Copy link
Collaborator

@708yamaguchi 708yamaguchi commented Jul 15, 2021

最近go to kitchenが遅くfetchの動作もガクガクしていたので、高速化できないかなぁと昨日から色々考えていたのですが、go to kitchenの高速化だけを考えるとfetch roboticsのmove_baseのデフォルトパラメータを使うのが良さげでした。

2回チャレンジしたのですが、その時のメールタイトルとhtopのキャプチャを送ります。
Fetch kitchen patrol demo: 2021/07/15 (18:00:15)
https://drive.google.com/file/d/1UrO21NxgYfb-LU2xef75yNq0xwVxkxuH/view?usp=sharing
Fetch kitchen patrol demo: 2021/07/15 (18:05:00)
https://drive.google.com/file/d/1vCUvqIiKvYos69NRd4pNOrvgrA_ymG3V/view?usp=sharing

2回ともfetchはスムーズに(高速に)動き、move_baseのCPU使用率も40%以下くらいに収まっています。「Control loop rateが低い」みたいなエラーも出ていませんでした。
(1回目はspotに近づきすぎて動けなくなったため、僕がspotをずらすと良い感じに動き始めました。失敗の原因は、fetchが高速に動いたからかamclによる自己位置推定がズレてspotに近づきすぎたからだと思います。)

go to kitchenの他に、73B2前の廊下から7階のエレベータ前まで行って返ってくるのを2D Nav Goalで行ってみましたが、こちらもスムーズに(高速に)動いていました。

もし時間があれば708yamaguchi/yamaguchi-fast-go-to-kitchenブランチを使って試してほしいです。

@knorth55 @sktometometo @mqcmd196
相談なのですが、
みんなが色々パラメータチューニングをしてきたと思うので、変更が必須なパラメータとその理由を教えてほしいです。「どこどこに行くためにはこのパラメータが必要」みたいな事情を知りたいです。
そこは必ずパラメータを変更して、yamlファイルに理由を書いておきます。

@708yamaguchi
Copy link
Collaborator Author

708yamaguchi commented Jul 15, 2021

山口TODO(追記アリ)

  • 必須なyamlの変更を追加する -> fetch15とfetch1075で分けた
  • <arg name="custom_move_base_config" default="true" />にする -> fetch15とfetch1075で分けた
  • Rvizのpathのトピック名を変更する -> 各プランナが出力するトピックをrelayして、同じ名前でアクセスできるようにした。

@708yamaguchi 708yamaguchi changed the title Fast go to kitchen [WIP] Fast go to kitchen Jul 15, 2021
@sktometometo
Copy link
Collaborator

@708yamaguchi いまのパラメータは15で動かす際のチューニング結果なので、15でも動くかどうか確認するか、15と1075でパラメータ分けるかしてほしいかもです。

@708yamaguchi
Copy link
Collaborator Author

そうですね、パラメータを分けるのが一番良い気がします。

@708yamaguchi 708yamaguchi merged commit 559492e into knorth55:fetch15 Jul 15, 2021
@708yamaguchi
Copy link
Collaborator Author

パラメータを分けて、fetch15もmove_baseが動作することを確認しました。

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

Successfully merging this pull request may close these issues.

2 participants