-
Notifications
You must be signed in to change notification settings - Fork 657
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
edge walk failed #4684
Labels
Comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
valhalla/src/thor/route_matcher.cc
Lines 132 to 143 in 1b864c9
why do we need followed_edges ? I feel like this is a useless cache, use start_de will skip some edge from the current node
There's a scenario.
when we get the wrong path and try to step back, it is the same correlated_index but not the same node (because there are two very similar edges)
if use start_de from followed_edges will skip some edge
The text was updated successfully, but these errors were encountered: