-
Notifications
You must be signed in to change notification settings - Fork 673
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_path_planner): fix turn signal endless loop #5444
Conversation
Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
8a95453
to
0ca67d3
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #5444 +/- ##
==========================================
- Coverage 14.76% 14.76% -0.01%
==========================================
Files 1663 1663
Lines 115772 115767 -5
Branches 35925 35923 -2
==========================================
- Hits 17099 17096 -3
Misses 79310 79310
+ Partials 19363 19361 -2
*This pull request uses carry forward flags. Click here to find out more.
☔ View full report in Codecov by Sentry. |
@danielsanchezaran could you please review? |
@danielsanchezaran When close the issue, we really appreciate it if you upload the video that problem is solved. |
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!
(don't merge before @.danielsanchezaran san's approval 🙇
@kosuke55 CC @kyoichi-sugahara |
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!
…ndation#5444) Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
…ndation#5444) Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
…ndation#5444) Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
Description
fix turn signal endless loop
#5440
Tests performed
https://evaluation.tier4.jp/evaluation/reports/c40eba8d-7d8e-5aa6-9c33-071b5b7841ac?project_id=prd_jt
1665/1667
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.