Skip to content

RJD-1666 Change PathWithLaneId subscription to /api/routing/route#1621

Draft
gmajrobotec wants to merge 3 commits intomasterfrom
RJD-1666/remove_pathwithlaneid_subscription
Draft

RJD-1666 Change PathWithLaneId subscription to /api/routing/route#1621
gmajrobotec wants to merge 3 commits intomasterfrom
RJD-1666/remove_pathwithlaneid_subscription

Conversation

@gmajrobotec
Copy link
Copy Markdown
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

@github-actions
Copy link
Copy Markdown

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
Copy Markdown
Member

HansRobo commented Jun 5, 2025

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 6, 2025

@sonarqubecloud
Copy link
Copy Markdown

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