fix(traffic_light_multi_camera_fusion): assign multiple regulatory element id for one traffic light

[TomohitoAndo]

Description

In the current implementation, if a traffic light is linked to multiple regulatory elements, the multi-camera fusion only publishes one signal with a regulatory element. I've updated it to publish all associated regulatory elements with the traffic light perception results.

Tests performed

Tested in simple planning simulator.
This was tested in a basic planning simulator. Below is the output when traffic light 2400 is associated with regulatory elements 1489 and 1493.

Before

$ ros2 topic echo /perception/traffic_light_recognition/internal/traffic_signals
stamp:
  sec: 1692786155
  nanosec: 696961024
signals:
- traffic_signal_id: 1489
  elements:
  - color: 1
    shape: 1
    status: 0
    confidence: 0.9967631101608276

After

stamp:
  sec: 1692786410
  nanosec: 596963840
signals:
- traffic_signal_id: 1489
  elements:
  - color: 1
    shape: 1
    status: 0
    confidence: 0.6648706793785095
- traffic_signal_id: 1493
  elements:
  - color: 1
    shape: 1
    status: 0
    confidence: 0.6648706793785095

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.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

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.

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (a69ee45) 15.07% compared to head (35a7d38) 15.07%.
Report is 4 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4727   +/-   ##
=======================================
  Coverage   15.07%   15.07%           
=======================================
  Files        1560     1560           
  Lines      107737   107729    -8     
  Branches    33022    33021    -1     
=======================================
  Hits        16244    16244           
+ Misses      73785    73777    -8     
  Partials    17708    17708           

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <0.00%> (?)
total	15.07% <ø> (+<0.01%)	⬆️	Carriedforward from a69ee45

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

Files Changed	Coverage Δ
...include/traffic_light_multi_camera_fusion/node.hpp	0.00% <ø> (ø)
...ion/traffic_light_multi_camera_fusion/src/node.cpp	0.00% <0.00%> (ø)
planning/behavior_path_planner/src/utils/utils.cpp	15.37% <ø> (+0.01%)	⬆️
sensing/gnss_poser/include/gnss_poser/convert.hpp	0.00% <ø> (ø)
...ensing/gnss_poser/include/gnss_poser/gnss_stat.hpp	0.00% <ø> (ø)
sensing/gnss_poser/src/gnss_poser_core.cpp	0.00% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[TomohitoAndo]

@miursh
Could you review? 🙏

[miursh]

@TomohitoAndo Than you!! I will check this by tonight.

[miursh]

LGTM