-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Store auxiliary data with edges #1342
Comments
related to #282. |
What you basically want is a a mini key-value storage engine inside OSRM that maps edge/node ids to properties. Could be worthwhile to invest some time in this, as this often comes up. However implementing this is not trivial. |
right. preferably, the lua script will be able set the keys/values for each way. |
I use the way name workaround at cycle.travel. The results are heavily tokenised (one character for surface type, etc.) which saves some space and download time. But yes, a PBF-like key-val store wouldn't be a bad thing. |
This would also unblock #77 since we could move the speed inside that field. |
I'm also interested in this feature, specifically on the surface, lanes and width property. Maybe it could be customized on the profile? |
If you can handle a second network round-trip, you can use this to get tag data: |
in our "I Bike CPH" bike router app for ios and android, we provide GPS turn-by-turn navigation. we would like to use things like way type, surface, etc. to provide additional cues. this is relevant for bikes, since many bike paths don't have names.
a workaround is to encode the tags as part of the way name in the lua profile, for example "Vesterbrogade {'highway':'primary'}" or "{'highway':'cycleway','surface':'gravel'}. but this takes up extra space.
a more efficient way of storing tags and passing them to the front-end would be great. one idea would be to store the encoded tags as a separate name string, ie store "Vesterbrogade" and "{'highway':'primary'}" separately. since many ways would have the same tags, it would take up less space. but a better solution can probably be found.
The text was updated successfully, but these errors were encountered: