fix: use hazard in auto command when emergency_handling is false

[tkimura4]

Signed-off-by: tomoya.kimura tomoya.kimura@tier4.jp

Related Issue(required)

Description(required)

Fix porting miss in vehicle_cmd_gate.
https://github.com/tier4/AutowareArchitectureProposal.iv/blob/main/control/vehicle_cmd_gate/src/vehicle_cmd_gate.cpp#L309-310

When use_emergency_handling is false, use the auto_commands for hazard_light

Review Procedure(required)

Set use_emergency_handling to false in control.launch.py
Launch planning simulator, and start to run the vehicle.
Transition the vehicle state to Emergency in some way, and confirm that the hazard lights are not enabled.

Related PR(optional)

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

• Read commit-guidelines
• Assign PR to reviewer

If you are adding new package following items are required:

• Documentation with description of the package is available
• A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

• Commits are properly organized and messages are according to the guideline
• PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

• All open points are addressed and tracked via issues or tickets

CI Checks

• Build and test for PR / build-and-test-pr: Required to pass before the merge.
• Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
• Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

[codecov]

Codecov Report

Merging #487 (2b43d94) into main (6e3b9ea) will increase coverage by 7.90%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #487      +/-   ##
==========================================
+ Coverage   10.95%   18.86%   +7.90%     
==========================================
  Files         704        6     -698     
  Lines       49656      477   -49179     
  Branches     6655      203    -6452     
==========================================
- Hits         5442       90    -5352     
+ Misses      39852      360   -39492     
+ Partials     4362       27    -4335     

Impacted Files	Coverage Δ
control/vehicle_cmd_gate/src/vehicle_cmd_gate.cpp	0.00% <0.00%> (ø)
...ene_module/occlusion_spot/occlusion_spot_utils.hpp
...ator/include/planning_evaluator/metrics/metric.hpp
...toware_auto_geometry/test/src/test_convex_hull.cpp
.../motion_velocity_smoother/src/trajectory_utils.cpp
map/lanelet2_extension/lib/detection_area.cpp
...autoware_utils/test/src/geometry/test_geometry.cpp
...clude/obstacle_avoidance_planner/marker_helper.hpp
...nclude/lidar_apollo_instance_segmentation/util.hpp
...vehicle_model/sim_model_delay_steer_acc_geared.cpp
... and 689 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6e3b9ea...2b43d94. Read the comment docs.

[yukkysaito]

@tkimura4 If use_emergency_handling is false, does this mean that the hazard lights will turn on according to emergency stop even when emergency stop is not working?

[tkimura4]

@yukkysaito
Currently, yes. I fixed it on this PR.

[yukkysaito]

@tkimura4 Ok 👍 pls fix PR title

[tkimura4]

@yukkysaito
I changed title and commit message.
Please approve one more.