Feature/enable specify entity type in autosink

[hakuturu583]

Description

Abstract

Enabled selection of Entity to be Sinked in TrafficSink.

Background

In TrafficSink, arbitrary Entity was despawned when it approached TrafficSink, so there were cases where EgoEntity was despawned and testing could not continue.

Details

To enable this feature, set auto_sink, a member of traffic_simulator::Configuration structure, to true, and set std::set<uint8_t> to specify the target EntityType of the entity to be despawned.

e.g

auto configuration = traffic_simulator::Configuration(map_path);
configuration.auto_sink = true;
configuration.sinkable_entity_type = {traffic_simulator::EntityType::VEHICLE}

If you want to check the uint_8 type number assigned to the EntityType you want to specify, please check the following URL.

https://github.com/tier4/scenario_simulator_v2/blob/master/simulation/traffic_simulator_msgs/msg/EntityType.msg

References

N/A

Destructive Changes

N/A

Known Limitations

• In feat(traffic_controller, api): add rviz marker for TrafficSink #1335, we found that the TrafficSink constructor was not designed well enough to be merged into the PR, but we will not fix it in this PR.
• API to install TrafficSink without auto_sink function to automatically install TrafficSink at the end of the road is not implemented.

[github-actions]

Checklist for reviewers ☑️

All references to "You" in the following text refer to the code reviewer.

• Is this pull request written in a way that is easy to read from a third-party perspective?
• Is there sufficient information (background, purpose, specification, algorithm description, list of disruptive changes, and migration guide) in the description of this pull request?
• If this pull request contains a destructive change, does this pull request contain the migration guide?
• Labels of this pull request are valid?
• All unit tests/integration tests are included in this pull request? If you think adding test cases is unnecessary, please describe why and cross out this line.
• The documentation for this pull request is enough? If you think adding documents for this pull request is unnecessary, please describe why and cross out this line.

[yamacir-kit]

Instead of adding a less generic type like EntityTypeComparator, change it to add comparison operator overloads to EntityType.

[sonarqubecloud]

Quality Gate passed

Issues
1 New issue
1 Accepted issue

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud