-
Notifications
You must be signed in to change notification settings - Fork 672
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): consider acceleration during avoidance maneuver #4186
feat(avoidance): consider acceleration during avoidance maneuver #4186
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #4186 +/- ##
=======================================
Coverage 14.16% 14.16%
=======================================
Files 1586 1586
Lines 109537 109499 -38
Branches 31451 31429 -22
=======================================
- Hits 15518 15515 -3
+ Misses 77130 77102 -28
+ Partials 16889 16882 -7
*This pull request uses carry forward flags. Click here to find out more.
☔ View full report in Codecov by Sentry. |
aa0a6c9
to
24f6bbb
Compare
Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
…ce maneuver Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
24f6bbb
to
cb72159
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Description
🤖 Generated by Copilot at aa0a6c9
This pull request refactors and improves the avoidance logic of the behavior path planner. It simplifies the configuration file by removing two parameters and adding one for limiting lateral acceleration. It also removes unused code and updates the documentation accordingly.
Please approve ⬇️ before this PR.
autowarefoundation/autoware_launch#436
Tests performed
Effects on system behavior
Make it possible to accelerate during avoidance maneuver.
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.
After all checkboxes are checked, anyone who has write access can merge the PR.