Kevin Kreiser
Kevin Kreiser
who would have thought this would change so frequently
https://github.com/valhalla/valhalla/pull/2660 the reasoning, which I think is still valid, is that we have unidirectional algorithms for time dependence. we added the ability to force bidir and at that point maybe...
https://github.com/valhalla/valhalla/blame/master/src/thor/bidirectional_astar.cc#L517
ah that is an unfortunate misunderstanding. i wasn't attempting to emphasize the word "tracking" like you seem to have understood, a better way for me to have written that sentence...
yeah these are all my fault :smile: thank you for making the issue!
@gknisely just mentioned this to me in passing the other day. i only looked into very very briefly but things like this: https://github.com/valhalla/valhalla/blob/98fa75242d1853dcf2a58e9f29b775cc5d4dedc2/lua/graph.lua#L1182-L1192 are pretty unintuitive. i need time to...
it reminds me very much of administrative boundary processing except in this case we don't store admin info we store a bit that says the edge intersected the boundary. I...
to really tell the full story in my mind, im kind of thinking a bigger picture architecture. what id like to see is we have a binary that pulls out...
right but the max valid value we support is 9 which easily fits in 4 bits. at the end of the day this value gets packed into https://github.com/valhalla/valhalla/blob/master/valhalla/baldr/accessrestriction.h#L67 which is...
some further discussion here: https://github.com/valhalla/valhalla/pull/4485#discussion_r1440040661 seems like since we already had 6 unused values we can shrink the type to 4 bits and get 2 spare in the AccessRestriction::type_ field