Nils
I'll leave that here too: https://github.com/Project-OSRM/osrm-backend/issues/6631. Description: https://wiki.openstreetmap.org/wiki/Tag:junction%3Droundabout#Turbo_roundabouts OSM Example: https://www.openstreetmap.org/way/1168475314. Route Example: https://valhalla.openstreetmap.de/directions?profile=car&wps=6.206252612268473%2C52.15738412198366%2C6.202636595191105%2C52.16075073005974
since https://github.com/valhalla/valhalla/pull/2651, we use `OSRMCarTurnDuration` to add transition cost for traffic light and turns. but we only add this to the `Cost::secs` in e.g. `AutoCost::TransitionCost` while we add our "own"...
in costmatrix we already had a special check for trivial routes when checking a connecting edge, but not for the case where the connecting edge is the correlated edge on...
we should consider the attributes of the opposing edge when we create shortcuts, ie don't contract nodes if the opposing ones have attributes which prohibit contraction. right now we just...
I just found a serious bug in our TomTom converter (for Valhalla at least) where we turned every simple turn restriction into a complex one accidentally, which was around 10...
as preparation for https://github.com/valhalla/valhalla/issues/4568 in that issue I promised to look into the current path pruning conditions which we apply to origin/destination correlated edges to allow closed/destonly/not_thru, but otherwise prohibit...
fixes #3807 Will see another day if it'll work, just leaving some comments now after looking over the bidirectional ones.
fixes #3958 Lets a user know which source/target id is the problem. Though on second thought it's not that super informative; if one pair is off, there's very likely more....
I think it's currently a bit cumbersome to know anything about new config options (as a user). lately new configs all have defaults so it'll not really error out anymore...
showing https://github.com/valhalla/valhalla/issues/4433. it's not fixed yet, just a test written to showcase the problem. I recently did a band-aid for this, just so we get the right shape out of...