#10932 closed enhancement (fixed)
[Patch] Warn if way has 'turn:lanes=*' but lacks 'lanes=*' tag.
Reported by: | rickmastfan67 | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 20.07 |
Component: | Core validator | Version: | latest |
Keywords: | lanes lanes-tagging | Cc: |
Description
Please throw up a validator warning if a way happens to have 'turn:lanes=*', but doesn't have 'lanes=*'. This is always an error since routers that need the 'lanes=*' tag to be able to properly use the 'turn:lanes' tag. An example would be Way 294921123 (https://www.openstreetmap.org/way/294921123).
Repository Root: http://josm.openstreetmap.de/svn Build-Date: 2015-01-05 02:33:53 Last Changed Author: Klumbumbus Revision: 7925 Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b Relative URL: ^/trunk URL: http://josm.openstreetmap.de/svn/trunk Last Changed Date: 2015-01-04 18:11:11 +0100 (Sun, 04 Jan 2015) Last Changed Rev: 7925 Identification: JOSM/1.5 (7925 en) Windows 7 64-Bit Memory Usage: 354 MB / 1820 MB (179 MB allocated, but free) Java version: 1.7.0_71, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM Dataset consistency test: No problems found Plugins: - ImageryCache (30892) - OpeningHoursEditor (30892) - buildings_tools (30892) - measurement (30892) - mirrored_download (30892) - notes (v0.9.5) - osmarender (30892) - reverter (30892) - turnrestrictions (30892) - undelete (30892) - utilsplugin2 (30892)
Attachments (2)
Change History (13)
comment:1 Changed 8 years ago by
comment:2 Changed 8 years ago by
But what would happen if a user is trying to tag the turn:lanes
, but instead of using |
to separate the lanes, they use ;
for every part of the tag? Here's a case when the lanes=*
tag would be a fail safe, at least for a oneway
highway like highway=motorway
.
And how about if there's a segment of highway that has 5 lanes, but since the user didn't add the lanes=*
, lanes:forward=*
, & lanes:backward=*
tags, but only added a turn:lanes:forward
tag?
There honestly should be at minimum a warning in the 'Other' section. But IMO, it would be better off in the 'Warnings' section.
comment:3 Changed 8 years ago by
Type: | defect → enhancement |
---|
comment:5 follow-up: 7 Changed 3 years ago by
I would say, a warning for ways with *:lanes=*
without lanes=*
and for *:lanes[:*]:forward/backward=*
without lanes:forward/backward=*
does not harm.
comment:6 Changed 3 years ago by
Keywords: | lanes lanes-tagging added |
---|
comment:7 Changed 3 years ago by
Summary: | Validator: Warn if way has 'turn:lanes=*' but lacks 'lanes=*' tag. → [Patch] Warn if way has 'turn:lanes=*' but lacks 'lanes=*' tag. |
---|
Replying to skyper:
I would say, a warning for ways with
*:lanes=*
withoutlanes=*
and for*:lanes[:*]:forward/backward=*
withoutlanes:forward/backward=*
does not harm.
Please, find attached patch file which adds the warning.
comment:8 Changed 3 years ago by
comment:10 Changed 3 years ago by
Milestone: | → 20.07 |
---|
Replying to rickmastfan67:
lanes=*
is not really required, ifturn:lanes
is present, because you can calculate the number of lanes bythe number of
|
inturn:lanes
+1Also note that the lanes key counts only lanes for motorized traffic, while the
:lanes
suffix counts also e.g. bicycle lanes. So the number inlanes
can be different from the number of lanes inturn:lanes
. See Key:lanes and Lanes#Crossing_with_a_designated_lane_for_bicycles