-
Notifications
You must be signed in to change notification settings - Fork 683
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(behavior_path_planner): add option for combining arc pull out paths #2669
Conversation
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
...ior_path_planner/include/behavior_path_planner/scene_module/pull_out/pull_out_parameters.hpp
Outdated
Show resolved
Hide resolved
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
f6b51ec
to
18ce9af
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.
cap-.2023-01-17-19-23-56.mp4.mp4
LGTM
Codecov ReportBase: 11.38% // Head: 11.36% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #2669 +/- ##
==========================================
- Coverage 11.38% 11.36% -0.03%
==========================================
Files 1277 1277
Lines 89230 89444 +214
Branches 23619 23766 +147
==========================================
+ Hits 10160 10161 +1
- Misses 68297 68497 +200
- Partials 10773 10786 +13
*This pull request uses carry forward flags. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
…ths (autowarefoundation#2669) * feat(behavior_path_planner): add option for combining arc pull out paths Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * divide_pull_out_path Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
* feat(behavior_path_planner): ignore pull over goal near lane start (autowarefoundation#2667) feat(behavior_path_planner) ignore pull over goal near lane start Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * feat(behavior_path_planner): add option for combining arc pull out paths (autowarefoundation#2669) * feat(behavior_path_planner): add option for combining arc pull out paths Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * divide_pull_out_path Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Co-authored-by: Kosuke Takeuchi <kosuke.tnp@gmail.com>
…ths (autowarefoundation#2669) * feat(behavior_path_planner): add option for combining arc pull out paths Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * divide_pull_out_path Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
Signed-off-by: kosuke55 kosuke.tnp@gmail.com
Description
Add option for combining two arc paths of pull_out. This eliminates stops along the way.
pull_out_arc_path-2023-01-17_17.29.00.mp4
Related links
tier4 internal link
Tests performed
Notes for reviewers
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.