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(behavior_velocity_run_out_module): fix slow_down jerk filter #1744

Open
wants to merge 1 commit into
base: beta/v0.10.1
Choose a base branch
from

Conversation

tkimura4
Copy link
Collaborator

@tkimura4 tkimura4 commented Dec 26, 2024

Description

run_outをslow_down_limit.enable = trueで動かすと、JerkFilterによりrun_out以外の停止線も上書きされて無効化されてしまう現象があったため、修正

Related links

None

How was this PR tested?

Before:

  • stop_lineによる停止線がrun_outのジャークフィルターによって上書きされてしまっている
    (走行中にAutoware ControlをDisableにすることで強制的に停止線前で速度が出ている状況を再現)
cap-.2024-12-26-10-48-29.mp4

After:

  • 既存の停止線がrun_outのジャークフィルターによって上書きされないように修正
    (走行中にAutoware ControlをDisableにすることで強制的に停止線前で速度が出ている状況を再現)
cap-.2024-12-26-10-52-19.mp4
  • 通常のJerk Filterは今までどおり正常に機能している状態。直近のrun_outに対しては強い減速かからず、遠方のrun_outに対しては停止指示がかかっている。
    (なお、下記の動画ではrun_outのテストのためにobstacle_cruiseを無効にしている)
cap-.2024-12-26-10-53-35.mp4

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

slow_down_limit.enable = trueでrun)outを起動した際に、他の回避モジュールや一時停止モジュールによる停止線が上書きされて無視されるバグが修正される

@tkimura4 tkimura4 requested a review from keita1523 December 26, 2024 01:36
@tkimura4 tkimura4 marked this pull request as ready for review December 26, 2024 01:47
Copy link
Collaborator

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

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

LGTM

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

Successfully merging this pull request may close these issues.

2 participants