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

Render lcn routes tagged on ways directly #430

Merged
merged 3 commits into from
Sep 29, 2020
Merged

Conversation

brainwad
Copy link
Contributor

@brainwad brainwad commented Sep 9, 2020

LCN routes are often tagged with a simple lcn= tag on the ways, rather than with route relations, especially because they often aren't numbered/named. Other tiers of cycle routes are much-less-often tagged this way, see taginfo: lcn rcn ncn icn.

The most common value is yes, but the next most common is proposed, which should be rendered similarly to relation-based routes with state=proposed, presumably.

LCN routes are often tagged with a simple lcn=yes tag on the ways, rather than with route relations, especially because they often aren't numbered/named.
lcn='s second-most-common value is 'proposed', which presumably means the same thing as state=proposed on a relation-based LCN route.
@brainwad brainwad changed the title Render lcn routes tagged in the old style Render lcn routes tagged on ways directly Sep 9, 2020
Don't render the negative tagging of lcn on ways.
@brainwad brainwad marked this pull request as ready for review September 9, 2020 19:35
@Phyks Phyks merged commit 783093c into cyclosm:master Sep 29, 2020
@Phyks
Copy link
Member

Phyks commented Sep 29, 2020

Hi! Sorry, this one slipped out of my focus. Looks good to me, merging! Thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants