-
Notifications
You must be signed in to change notification settings - Fork 684
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
docs(lane_change): stopping position when an object exists ahead #5523
Conversation
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
@rej55 |
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
@TakaHoribe |
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
One comment: the function of "stop for lane change" should be explained before describing each use case.
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
dd91379
to
25bc279
Compare
fixed in 25bc279 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5523 +/- ##
==========================================
- Coverage 14.86% 14.83% -0.03%
==========================================
Files 1666 1667 +1
Lines 116258 119302 +3044
Branches 36266 38459 +2193
==========================================
+ Hits 17277 17700 +423
- Misses 79476 81472 +1996
- Partials 19505 20130 +625
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
…owarefoundation#5523) * docs(lane_change): Stopping position when an object exists ahead Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * add stuck params Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * add backward_length_buffer_for_blocking_object Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * add stop for lane change Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> --------- Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
Description
(Currently, due to a bug, it does not actually work as described in the following document, but it does show the ideal behavior.)
Tests performed
Not applicable.
Effects on system behavior
Not applicable.
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.