Skip to content
This repository has been archived by the owner on Mar 9, 2023. It is now read-only.

Contradictory cycleway tagging should report an error #164

Open
westnordost opened this issue Apr 29, 2022 · 3 comments · Fixed by #222
Open

Contradictory cycleway tagging should report an error #164

westnordost opened this issue Apr 29, 2022 · 3 comments · Fixed by #222

Comments

@westnordost
Copy link

westnordost commented Apr 29, 2022

Tagging like

  • cycleway=no + cycleway:left=track
  • cycleway=track + cycleway:left=no
  • cycleway:both=lane+ cycleway:right=track

should report an error.
I.e. wherever cycleway:both/ cycleway contradicts with the cycleway:<direction> tag

This tagging is ambiguous and either one of these tags must be a mistake (or both).

This was referenced Apr 30, 2022
droogmic added a commit to droogmic/osm2lanes that referenced this issue Jun 11, 2022
droogmic added a commit to droogmic/osm2lanes that referenced this issue Jun 12, 2022
droogmic added a commit to droogmic/osm2lanes that referenced this issue Jun 12, 2022
@droogmic
Copy link
Collaborator

So this is now implemented as a warning, because we want to avoid erroring at all costs (it messes up map-wide automations if nothing is produced).
Please check and see if the functionality is as expected?

@westnordost
Copy link
Author

westnordost commented Jun 12, 2022

Warning: unsupported: 'cycleway=track cycleway:both=track'

  1. The warning is not very informative
  2. the warning is returned also if the two values do not contradict

@droogmic
Copy link
Collaborator

indeed, thanks.

@droogmic droogmic reopened this Jun 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants