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

Specify crossing:markings in Poland #1030

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Specify crossing:markings in Poland #1030

merged 1 commit into from
Dec 15, 2023

Conversation

Zaczero
Copy link
Contributor

@Zaczero Zaczero commented Oct 15, 2023

No description provided.

@github-actions
Copy link

🍱 Preview the tagging presets of this pull request here: https://pr-1030--ideditor-presets-preview.netlify.app/id/dist/#locale=en.

@1ec5 1ec5 mentioned this pull request Oct 28, 2023
@tyrasd tyrasd added this to the v6.6 milestone Dec 15, 2023
@tyrasd
Copy link
Member

tyrasd commented Dec 15, 2023

Hey, thanks for this. I only noticed that the value dashes is also relatively common, it seems to be used for bicycle crossings like this one. Is it correct that these should have been tagged using dots as the value instead?

@Zaczero
Copy link
Contributor Author

Zaczero commented Dec 15, 2023

image

I strongly believe that all bicycle crossings are indeed dots. I suppose the issue arises from the interpretation of what constitutes a dot and what constitutes a dash. I believe the mapping of dashes is incorrect, but OSM documentation does not state that clearly.

image

In Poland, there is a single way to mark cycle crossings, so either all of them should be dots, or all of them should be dashes. I believe the sooner.

@tyrasd tyrasd added the field label Dec 15, 2023
@tyrasd tyrasd merged commit 3f8c56b into openstreetmap:main Dec 15, 2023
5 checks passed
@Zaczero
Copy link
Contributor Author

Zaczero commented Dec 15, 2023

❤️

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

Successfully merging this pull request may close these issues.

2 participants