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/nav2 param navigation #82

Merged
merged 3 commits into from
Nov 21, 2023
Merged

Fix/nav2 param navigation #82

merged 3 commits into from
Nov 21, 2023

Conversation

shunki1006
Copy link
Contributor

PR Type

  • Feature
  • Bug fix
  • Refactor
  • Documentation
  • Other

Overview

  • stopするはずのwaypointで停止しない問題対策のNav2param調整と, 狭い道を通るためのロボットfootprint調整.

Detail

  1. tandemが終わる位置にstopのパラメータがtrueになっているwaypointがあるとstopせずに通過してしまう問題があり, その原因として, navigation2のゴール到達判定と, waypoint_navigationのゴール到達判定の相違にあると考えたため, Nav2param内の到達判定を厳しく, waypoint_navigation内の到達判定を緩くすることで対応しました. これにより, 基本的waypoint_navigationの判定をもとに次の動作をするようになっています.
  2. 下記PRで急な障害物を確実に避けるために余分に大きくしたfootprintが大きすぎることが原因で狭い道を通ることができないことがつくばチャレンジ走行中に発覚したため, 下記PR変更前のfootprintに戻しました.
    参考;Fix/nav2 local costmap #73 (comment)

Test

  • つくばで動作確認済み.

Attention

  • レビューの際にはお手数ですがdevel/tsukuba2023_Seniorに対して出しているPRの内容とのずれがないかも確認していただきたいです. お願いいたします.

@shunki1006 shunki1006 added the bug Something isn't working label Nov 20, 2023
@shunki1006 shunki1006 self-assigned this Nov 20, 2023
@shunki1006 shunki1006 mentioned this pull request Nov 20, 2023
6 tasks
@koki0624
Copy link
Contributor

devel/tsukuba2023_Senior と同じ launch ファイルを編集しているので、競合するかもです。

@shunki1006
Copy link
Contributor Author

devel/tsukuba2023_Senior と同じ launch ファイルを編集しているので、競合するかもです。

devel/tsukuba2023_Seniorはmainにmergeしない予定ブランチなので競合は起こらないのでは?

@koki0624
Copy link
Contributor

devel/tsukuba2023_Senior と同じ launch ファイルを編集しているので、競合するかもです。

devel/tsukuba2023_Seniorはmainにmergeしない予定ブランチなので競合は起こらないのでは?

マージ先が同じものと勘違いしてました。すみません。

@shunki1006
Copy link
Contributor Author

devel/tsukuba2023_Senior と同じ launch ファイルを編集しているので、競合するかもです。

devel/tsukuba2023_Seniorはmainにmergeしない予定ブランチなので競合は起こらないのでは?

マージ先が同じものと勘違いしてました。すみません。

分かりにくくてすみません, mergeお願いします.

@Alpaca-zip Alpaca-zip merged commit 649c1ac into main Nov 21, 2023
1 check passed
@Alpaca-zip Alpaca-zip deleted the fix/nav2_param_navigation branch November 21, 2023 08:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants