Skip to content
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

Handle route relations for roads #49

Open
pnorman opened this issue Mar 22, 2024 · 3 comments
Open

Handle route relations for roads #49

pnorman opened this issue Mar 22, 2024 · 3 comments

Comments

@pnorman
Copy link
Contributor

pnorman commented Mar 22, 2024

Highway refs are often indicated by route relations, so we need to support them.

One relation may be part of multiple road route relations, each of which has a ref and network tag. They also might be part of no road route relations and have a ref tag. I don't think we need to handle the third case of a ref tag plus refs from relations any different than the first tag.

@westnordost
Copy link
Contributor

Bake the refs of the route relations onto the roads, right?

@pnorman
Copy link
Contributor Author

pnorman commented Mar 31, 2024

Yes, we need the line strings to have appropriate properties. The difficult part is handling multiple relations.

@1ec5
Copy link

1ec5 commented Apr 3, 2024

Route relations intentionally provide more structure than way refs, so the two mechanisms aren’t quite equivalent – the structure should persist in the tile’s features to some extent, or it would defeat the purpose of route relations. openmaptiles/openmaptiles#1620 adopts a predictable route_#_key format for properties dissolved from route relations, allowing clients to process each value without resorting to any parsing.1

If you need to cap the maximum number of concurrent routes along a given way for performance reasons, here are some statistics to help you arrive at a practical limit: https://github.com/ZeLonewolf/openstreetmap-americana/issues/894#issuecomment-1765395845 https://github.com/ZeLonewolf/openstreetmap-americana/issues/894#issuecomment-1888362411.

Footnotes

  1. This would also be a sound server-side solution to the problem of pretty-printing semicolon-delimited value lists.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants