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(avoidance): add paramenters for dynamic detection area #634

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

satoshi-ota
Copy link
Contributor

@satoshi-ota satoshi-ota commented Oct 13, 2023

Description

This pull request refactors and improves the avoidance target filtering logic for the behavior_path_planner node. It introduces new parameters to configure the detection area for avoidance and lane change, and uses a dynamic detection range based on the vehicle speed and jerk limit. It also simplifies and optimizes the object separation code in utils.cpp and avoidance_module.cpp.

Tests performed

Evaluator

Effects on system behavior

Nothing.

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.

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.

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

@satoshi-ota satoshi-ota force-pushed the feat/change-detection-area-dynimic branch 2 times, most recently from 2dd23e5 to 12c076b Compare October 16, 2023 08:31
@satoshi-ota satoshi-ota marked this pull request as ready for review October 16, 2023 08:49
@satoshi-ota satoshi-ota force-pushed the feat/change-detection-area-dynimic branch from 12c076b to 063395c Compare October 17, 2023 05:36
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
@satoshi-ota satoshi-ota force-pushed the feat/change-detection-area-dynimic branch from 063395c to 3c0cdae Compare October 17, 2023 06:30
@satoshi-ota satoshi-ota merged commit 5d8a2f0 into main Oct 17, 2023
10 of 11 checks passed
@satoshi-ota satoshi-ota deleted the feat/change-detection-area-dynimic branch October 17, 2023 10:08
soblin pushed a commit to soblin/autoware_launch that referenced this pull request Nov 18, 2023
…-improvement

hotfix(crosswalk): cherry pick crosswalk module improvement
maxime-clem pushed a commit to maxime-clem/autoware_launch that referenced this pull request May 21, 2024
* chore: update fault injection params

* Update fault_injection.param.yaml

diag側にtypoがあるため、そちらに合わせてtypo

* Update fault_injection.param.yaml

元にしているdiagのtypoが修正されていたので、こちらも修正
yuki-takagi-66 referenced this pull request in tier4/autoware_launch Jul 8, 2024
* chore: update fault injection params

* Update fault_injection.param.yaml

diag側にtypoがあるため、そちらに合わせてtypo

* Update fault_injection.param.yaml

元にしているdiagのtypoが修正されていたので、こちらも修正
kyoichi-sugahara pushed a commit to kyoichi-sugahara/autoware_launch that referenced this pull request Nov 6, 2024
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