Skip to content

Conversation

@riron1206
Copy link

Summary

Addressed an issue where route_roadblock_correction would hang or take an excessive amount of time due to BFS search on certain maps.
This PR makes the search depth limits (search_depth_backward, search_depth_forward) configurable via Hydra config, allowing users to reduce the search space and avoid infinite loops.

Changes

  1. Configuration: Added pdm_search_depth_backward and pdm_search_depth_forward to default_metric_caching.yaml (defaults kept at 15 and 30).
  2. Parameter Propagation: Updated constructors and method calls in MetricCacheProcessor, PDMClosedPlanner, AbstractPDMClosedPlanner, and AbstractPDMPlanner to propagate these parameters down to route_utils.
  3. Bug Fix: Fixed a bug in AbstractPDMClosedPlanner where arguments were not passed to super().__init__.

Impact

  • PDM Planner related classes in navsim/planning.
  • Metric caching execution flow.
  • No behavioral change if default values are used. Lower values can prevent hang-ups and reduce computation time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant