-
Notifications
You must be signed in to change notification settings - Fork 701
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(avoidance): ignore objects are in intersection when the ego stops by red traffic signal #6565
fix(avoidance): ignore objects are in intersection when the ego stops by red traffic signal #6565
Conversation
…ic light Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
e8b218a
to
4208a72
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!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6565 +/- ##
=======================================
Coverage 14.78% 14.79%
=======================================
Files 1917 1917
Lines 132211 132217 +6
Branches 39297 39300 +3
=======================================
+ Hits 19552 19555 +3
- Misses 90833 90834 +1
- Partials 21826 21828 +2
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
… by red traffic signal (#6565) fix(avoidance): don't avoid objects are farther than red signal traffic light Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com> Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
… by red traffic signal (autowarefoundation#6565) fix(avoidance): don't avoid objects are farther than red signal traffic light Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
… by red traffic signal (autowarefoundation#6565) fix(avoidance): don't avoid objects are farther than red signal traffic light Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com> Signed-off-by: kaigohirao <kaigo.hirao@proxima-ai-tech.com>
… by red traffic signal (autowarefoundation#6565) fix(avoidance): don't avoid objects are farther than red signal traffic light Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
… by red traffic signal (autowarefoundation#6565) fix(avoidance): don't avoid objects are farther than red signal traffic light Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
Description
Sometimes, the avoidance modue generate unnecessary avoidance path for objects which are in intersection even when the ego stops by red traffic signal.
Related PR: https://tier4.atlassian.net/browse/RT1-5365

In this PR, I fixed detection range so that the module can ignore unrelated object.
Traffic light: RED -> Ignore objects are in intersection
Traffic light: GREEN -> Check objects are in intersection
Tests performed
Effects on system behavior
Improve avoidance behavior.
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.