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

Fix/velocity increase #76

Merged
merged 7 commits into from
Nov 8, 2023
Merged

Fix/velocity increase #76

merged 7 commits into from
Nov 8, 2023

Conversation

shunki1006
Copy link
Contributor

PR Type

  • Feature
  • Bug fix
  • Refactor
  • Documentation
  • Other

Overview

  • 速度向上及び, DWB内のcriticsの調整.

Detail

  • ロボットの最高速度を1.0m/sから1.5m/sまで上げました. motorのmax_rpm及びmax_velocityのパラメータを変更しました.
  • DWB内のcriticsのパラメータがdefaultの値と大きくかけ離れていたため, defaultの値に変更しました.

Test

  • 中庭で走行させましたが, 速度上昇, criticsの変更に伴う問題が生じないことを確認しました.

@shunki1006 shunki1006 added the enhancement New feature or request label Nov 8, 2023
@shunki1006 shunki1006 self-assigned this Nov 8, 2023
Copy link
Contributor

@Alpaca-zip Alpaca-zip left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

つくばチャレンジが6[km/h]まで出して良いので、上限は6[km/h]で良いと思います。

ちなみにですが、このパラメータ調整でナビゲーション中に1.5[m/s]出ることはありましたか?

@shunki1006
Copy link
Contributor Author

つくばチャレンジが6[km/h]まで出して良いので、上限は6[km/h]で良いと思います。

ちなみにですが、このパラメータ調整でナビゲーション中に1.5[m/s]出ることはありましたか?

最高速度を6.0km/hにすると, max_rpmが167になりキリが悪いので150にしておりました.
1.11倍の差なら気にならないと考えたのですがいかがでしょうか?

ちなみに1.0m/sに設定していた時同様, 1.5m/sは確認できませんでした.

@Alpaca-zip
Copy link
Contributor

最高速度を6.0km/hにすると, max_rpmが167になりキリが悪いので150にしておりました.
1.11倍の差なら気にならないと考えたのですがいかがでしょうか?

キリが良い悪いというよりかは数字にどんな意味があるかベースでパラメータ調整はした方が良いと思います。

ちなみに1.0m/sに設定していた時同様, 1.5m/sは確認できませんでした.

👍

@shunki1006
Copy link
Contributor Author

最高速度を6.0km/hにすると, max_rpmが167になりキリが悪いので150にしておりました.
1.11倍の差なら気にならないと考えたのですがいかがでしょうか?

キリが良い悪いというよりかは数字にどんな意味があるかベースでパラメータ調整はした方が良いと思います。

ちなみに1.0m/sに設定していた時同様, 1.5m/sは確認できませんでした.

👍

承知しました, 変更したのでご確認お願いします.

@shunki1006 shunki1006 requested a review from Alpaca-zip November 8, 2023 14:16
@Alpaca-zip Alpaca-zip merged commit d5e7630 into main Nov 8, 2023
1 check passed
@Alpaca-zip Alpaca-zip deleted the fix/velocity_increase branch November 8, 2023 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants