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

Ways with tag bicycle = designated should be tagged #512

Closed
nnhubbard opened this issue Oct 16, 2018 · 3 comments · Fixed by #604
Closed

Ways with tag bicycle = designated should be tagged #512

nnhubbard opened this issue Oct 16, 2018 · 3 comments · Fixed by #604
Milestone

Comments

@nnhubbard
Copy link
Contributor

nnhubbard commented Oct 16, 2018

Some ways that are paths are not designated as a highway = cycleway, instead the use the tag bicycle = designated. See https://www.openstreetmap.org/way/107483312#map=15/45.6287/-122.4478

screen shot 2018-10-16 at 12 38 16 pm

These types of paths should include data in the .mbtiles file so that they can be styled as a bike path.

Other locations are correctly created with highway = cycleway, see: https://www.openstreetmap.org/way/88732778

screen shot 2018-10-16 at 12 39 14 pm

@MartinMikita
Copy link
Collaborator

Thanks for your discovery.

According to the OSM wiki about cycling, this is correct and expected tagging.

Pull requests are welcome!

@MartinMikita MartinMikita added this to the Later milestone Oct 18, 2018
@nnhubbard
Copy link
Contributor Author

I guess instead of making it a cycleway, the bicycle = designated tag should be imported so that those can be styled differently if needed.

@nnhubbard nnhubbard changed the title Ways with tag bicycle = designated should be created as cycleway Ways with tag bicycle = designated should be tagged Oct 22, 2018
@zstadler
Copy link
Contributor

zstadler commented Apr 4, 2019

I agree with @nnhubbard comment above.

PR #573 is adding a bicycle tag, but it combines bicycle=designated info bicycle=yes:

WHEN bicycle IN ('yes', 'designated') THEN true

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

Successfully merging a pull request may close this issue.

4 participants