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

feat(obstacle_stop): upd obstacle hunting #733

Closed
wants to merge 13 commits into from

Conversation

sfukuta
Copy link

@sfukuta sfukuta commented Aug 16, 2023

Description

経路上の障害物検知した後に、障害物が移動た場合、停止線が移動した位置に対して再設定される。
その結果、停車・発車を繰り返すチャタリングが発生する問題がある。
本PRは、この問題を対策する。
※今回の対策は、OSSとして適応されている以下、PRを流用する形で適応する。

対策内容

  • 対策1
    • 障害物検知による停車時に、一定時間その停止状態を維持するようにする事で、検知した障害物が移動した場合に、最初に停止した状態を維持する。
      これにより、障害物の移動によるチャタリングを回避する。
    • 停止状態を維持する時間は、chattering_thresholdで設定する。(defalut : 0.5s)
  • 対策2
    • 経路上に、複数の障害物を検知した場合、各停止線に対するチャタリングが発生しないようにする。

Related links

https://tier4.atlassian.net/browse/AEAP-465
https://github.com/tier4/autoware_launch.x1.eve/pull/419

Tests performed

機能検証

chattering_thresholdの設定値を、default値では機能検証が難しいため、5sに変更して確認を実施。

  • 障害物を経路上に設定、障害物検知し停車後に、障害物がなくなっても、即時に再発進しない事を確認済み(Psim)
  • 一定時間(chattering_thresholで設定した時間)停止後に、再発車する事を確認済み(Psim)
  • 経路上に障害物を、2つ設置し、2点間でチャタリングをしない事を確認済み(Psim)

リグレッション評価

  • 経路上に障害物がない場合に、停車しないで走行が完了する事を確認済み。(Psim)
  • 経路上に障害物がある場合に、障害物検知時に、停止線で停止する事。障害物除去後に、再発車できる事を確認済み(Psim)

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.
  • The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Shigekazu Fukuta and others added 5 commits July 31, 2023 09:48
Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>
Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>
Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>
Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>
@h-ohta
Copy link

h-ohta commented Aug 29, 2023

動き的には問題なさそうでした
あとはコードの修正と、CIの修正でいいと思います

@sfukuta sfukuta changed the title feat(obstacle_stop) : upd obstacle hunting feat(obstacle_stop): upd obstacle hunting Aug 29, 2023
@sfukuta
Copy link
Author

sfukuta commented Sep 4, 2023

@h-ohta PRのコメントについて、対応の確認をお願いします。

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

Successfully merging this pull request may close these issues.

2 participants