Skip to content
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_velocity_planner): output stop reason of traffic light whenever a stop point is inserted #2323

Merged

Conversation

tkimura4
Copy link
Contributor

@tkimura4 tkimura4 commented Nov 18, 2022

Description

In the current implementation, when the car stops by the traffic light module and its stop reason is RTC, the stop_reasons of traffic light is not output.
As other behavior modules, I changed that in cases like the above stop_reasons with empty stop_factor_points is output.

Review

1,
Comment out default_enable_list."traffic_light" in rtc_auto_mode_manager.param.yaml

Set a route including stop lines for traffic lights

Confirm the stop_reason of traffic light is output

ros2 topic echo /planning/scenario_planning/status/stop_reasons|grep TrafficLight -A 11

image

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

…whenever a stop point is inserted

Signed-off-by: tomoya.kimura <[email protected]>
@github-actions github-actions bot added the component:planning Route planning, decision-making, and navigation. (auto-assigned) label Nov 18, 2022
Copy link
Contributor

@taikitanaka3 taikitanaka3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
image

Copy link
Contributor

@h-ohta h-ohta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
image

@codecov
Copy link

codecov bot commented Nov 18, 2022

Codecov Report

Base: 10.48% // Head: 10.48% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (2c4844b) compared to base (3c0b335).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2323      +/-   ##
==========================================
- Coverage   10.48%   10.48%   -0.01%     
==========================================
  Files        1249     1249              
  Lines       90888    90910      +22     
  Branches    20860    20870      +10     
==========================================
  Hits         9531     9531              
- Misses      71229    71250      +21     
- Partials    10128    10129       +1     
Flag Coverage Δ *Carryforward flag
differential 4.21% <0.00%> (?)
total 10.46% <0.00%> (ø) Carriedforward from 3c0b335

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files Coverage Δ
...y_planner/src/scene_module/traffic_light/scene.cpp 0.00% <0.00%> (ø)

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.
📢 Do you have feedback about the report comment? Let us know in this issue.

@tkimura4 tkimura4 enabled auto-merge (squash) November 18, 2022 10:20
@tkimura4 tkimura4 merged commit 35e7b79 into autowarefoundation:main Nov 18, 2022
@tkimura4 tkimura4 deleted the feat/always_output_stop_reason branch November 18, 2022 10:35
h-ohta pushed a commit to tier4/autoware.universe that referenced this pull request Dec 15, 2022
…whenever a stop point is inserted (autowarefoundation#2323)

Signed-off-by: tomoya.kimura <[email protected]>

Signed-off-by: tomoya.kimura <[email protected]>
h-ohta added a commit to tier4/autoware.universe that referenced this pull request Dec 15, 2022
…whenever a stop point is inserted (backport autowarefoundation#2323) (#209)

feat(behavior_velocity_planner): output stop reason of traffic light whenever a stop point is inserted (autowarefoundation#2323)

Signed-off-by: tomoya.kimura <[email protected]>

Signed-off-by: tomoya.kimura <[email protected]>

Signed-off-by: tomoya.kimura <[email protected]>
Co-authored-by: Tomoya Kimura <[email protected]>
HansRobo pushed a commit to HansRobo/autoware.universe that referenced this pull request Dec 16, 2022
…whenever a stop point is inserted (autowarefoundation#2323)

Signed-off-by: tomoya.kimura <[email protected]>

Signed-off-by: tomoya.kimura <[email protected]>
Signed-off-by: Kotaro Yoshimoto <[email protected]>
kminoda pushed a commit to kminoda/autoware.universe that referenced this pull request Jan 6, 2023
…whenever a stop point is inserted (autowarefoundation#2323)

Signed-off-by: tomoya.kimura <[email protected]>

Signed-off-by: tomoya.kimura <[email protected]>
Signed-off-by: kminoda <[email protected]>
YoshiRi pushed a commit to YoshiRi/autoware.universe that referenced this pull request Jan 11, 2023
…whenever a stop point is inserted (autowarefoundation#2323)

Signed-off-by: tomoya.kimura <[email protected]>

Signed-off-by: tomoya.kimura <[email protected]>
Signed-off-by: yoshiri <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:planning Route planning, decision-making, and navigation. (auto-assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants