Skip to content

Conversation

gmajrobotec
Copy link
Contributor

@gmajrobotec gmajrobotec commented Jun 5, 2025

Description

Abstract

This PR removes /planning/scenario_planning/lane_driving/behavior_planning/path_with_lane_id subscription in FieldOperatorApplication and replaces it with /api/routing/route subscription. It also includes all needed changes in traffic_simulator to use new message.

Details

  1. Removed /planning/scenario_planning/lane_driving/behavior_planning/path_with_lane_id subscription.
  2. Add /api/routing/route subscription.
  3. Change EgoEntity::getRouteLanelets to use new message structure.

Destructive Changes

--

Known Limitations

--

References

RJD-1666

Copy link

github-actions bot commented Jun 5, 2025

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.

@gmajrobotec gmajrobotec added the bump patch If this pull request merged, bump patch version of the scenario_simulator_v2 label Jun 5, 2025
@HansRobo
Copy link
Member

HansRobo commented Jun 5, 2025

Copy link

sonarqubecloud bot commented Jun 6, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump patch If this pull request merged, bump patch version of the scenario_simulator_v2 wait for regression test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants