Opened 2 years ago
Last modified 2 years ago
#22928 new enhancement
Warn about bicycle=use_sidepath with conflicting cycleway tags — at Version 6
Reported by: | skyper | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core validator | Version: | |
Keywords: | bicycle use_sidepath cycleway | Cc: |
Description (last modified by )
cycleway=*
together with bicycle=use_sidepath
is only valid with cycleway=separate
and cycleway:both=separate
. Additionally cycleway:right/left=*
together with bicycle:forward/backward=use_sidepath
is only valid with cycleway:right/left=separate
. See https://community.openstreetmap.org/t/widerspruchlich-getaggte-rad-und-fusswege/98297/5 and https://community.openstreetmap.org/t/widerspruchlich-getaggte-rad-und-fusswege/98297/9.
There might be some more combinations with oneway=yes
.
Change History (6)
comment:1 by , 2 years ago
comment:2 by , 2 years ago
Please clarify the description. I use bicycle=use_sidepath often and I see no need to add a tag like cycleway=separate. I agree that a combination like cycleway=both + bicycle=use_sidepath should produce a warning.
comment:3 by , 2 years ago
the problem is not the presense or missing of the tag cycleway=separate.
use_sidepath is only to use with an separate mapped cycleway.
so any other value which is defining a cycleway (track/lane/shared/shared_lane/...) along the way without an separate geometry is wrong
also cycleway=no is wrong, because there have to be a separate way!
Have a look in my decision table. (null) stands for no cycleway tag present
follow-up: 6 comment:5 by , 2 years ago
Probably good to exclude any highway that contains bicycle:conditional
, bicycle:forward/backward/both_ways:conditional
for this check.
There might be some more combinations with
oneway=yes
.
Be additionally aware of oneway:bicycle
, oneway:conditional
, oneway:bicycle:conditional
in this case :)
comment:6 by , 2 years ago
Description: | modified (diff) |
---|
Replying to GerdP:
Please clarify the description. I use bicycle=use_sidepath often and I see no need to add a tag like cycleway=separate. I agree that a combination like cycleway=both + bicycle=use_sidepath should produce a warning.
Hope the description is a bit better, now.
Replying to Famlam:
Probably good to exclude any highway that contains
bicycle:conditional
,bicycle:forward/backward/both_ways:conditional
for this check.
I would expect cycleway:conditional
or similar then.
There might be some more combinations with
oneway=yes
.
Be additionally aware of
oneway:bicycle
,oneway:conditional
,oneway:bicycle:conditional
in this case :)
Yes, I am aware of oneway:bicycle
. For *:conditional
again, I would expect all tags to carry the suffix.
I hope this table is complete
cycleway (null) o.k.
cycleway(:both) separate o.k.
cycleway:left/right separate/separate o.k
cycleway:left/right no/separate o.k
cycleway:left/right separate/no o.k
cycleway:left/right no/(null) o.k
cycleway:left/right (null)/no o.k
cycleway:left/right separate/(null) o.k
cycleway:left/right (null)/separate o.k
cycleway(:both) !separate wrong
cycleway(:both) no wrong
cycleway:left/right separate/!separate wrong
cycleway:left/right !separate/separate wrong
cycleway:left/right !separate/no wrong
cycleway:left/right no/!separate wrong
cycleway:left/right no/no wrong
cycleway:left/right !separate/(null) wrong
cycleway:left/right (null)/!separate wrong