changed RouteOptionsUpdater
to use snapping_include_closures=true
for origin of each re-route request
#6050
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Closes #5664.
Adding
snapping_include_closures=true
resolves an occasional problem that returned a route in opposite direction or on parallel road when a driver caused a re-route by entering a closed section of a road. When present, this issue typically caused a loop of reroutes until a driver left the closed section. Withsnapping_include_closures=true
, we'll still try to navigate out of the closed section as soon as possible but without continuous re-routes that could be impossible to execute.cc @mandeepsandhu