Opened 3 years ago

Last modified 3 years ago

#20029 new defect

missing tag - turn:lanes:forward without lanes - although lanes:forward and lanes:backward are correct

Reported by: Polarbear-j Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: turn:lanes:forward lanes Cc:

Description (last modified by Polarbear-j)

How to reproduce:

create New Layer
draw street, tag highway and name
tag lanes:forward=2 and lanes:backward=1

try upload: validator ok

add turn:lanes:forward=left|through

try upload: Validator: "missing tag - turn:lanes:forward without lanes (1)"
Thus the validator requires the sum of lanes:forward + lanes:backward to be added in lanes, which is redundant and error-prone.
In particular it requires the sum only in the presence of other *:lanes tags, which is inconsequent.

Might be a side effect of #19609 ?
Might be unrelated but in the startup messages I found:
2020-11-03 20:39:15.377 INFO: Could not load tool definition turnlanes-tagging
from Status:

Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2020-11-03 00:09:37 +0100 (Tue, 03 Nov 2020)
Build-Date:2020-11-03 02:30:51
Relative:URL: /trunk

Identification: JOSM/1.5 (17292 en_GB) Mac OS X 10.14.6
OS Build number: Mac OS X 10.14.6 (18G6032)
Memory Usage: 1559 MB / 3641 MB (1372 MB allocated, but free)
Java version: 1.8.0_251-b08, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM

+ turnlanes-tagging (288)
+ turnrestrictions (35583)
validator.layer=false[{title=Addresses, active=true, url=resource://data/validator/addresses.mapcss},

{title=Tag combinations, active=true, url=resource://data/validator/combinations.mapcss},
{title=Deprecated features, active=true, url=resource://data/validator/deprecated.mapcss},
{title=Geometry, active=true, url=resource://data/validator/geometry.mapcss},
{title=Highways, active=true, url=resource://data/validator/highway.mapcss},
{title=Multiple values, active=true, url=resource://data/validator/multiple.mapcss},
{title=Numeric values, active=true, url=resource://data/validator/numeric.mapcss},
{title=Religion, active=true, url=resource://data/validator/religion.mapcss},
{title=Relations, active=true, url=resource://data/validator/relation.mapcss},
{title=Unnecessary tags, active=true, url=resource://data/validator/unnecessary.mapcss},
{title=Wikipedia, active=true, url=resource://data/validator/wikipedia.mapcss},
{active=true, title=Territories, url=resource://data/validator/territories.mapcss}


Attachments (0)

Change History (3)

comment:1 Changed 3 years ago by Polarbear-j

Description: modified (diff)

comment:2 Changed 3 years ago by skyper

lanes=* should always be present. lanes:forward and lanes:backward are additional tags.

Adding both, lanes:forward and lanes:backward, used to be optional but there are situations where only one is not completely sufficient. E.g. it is common practice to add both.

comment:3 Changed 3 years ago by Polarbear-j

I strongly disagree.
The wiki says on :

In the common case of two driving directions either :forward or :backward is added to the end of the key; i.e., 
<key>:lanes:forward describes the properties of the lanes in the same direction as the osm-way, whereas 
<key>:lanes:backward described the properties of the lanes in the opposite direction of the osm-way.

lanes=N as the simple sum of lanes:forward=F + lanes:backward=B is an extra effort for the mapper, since N needs to be adjusted according to any changes of F and B.
On the other side it is a very simple calculation for the data consumer, as all values are available directly in the tagged object.

I'm not a friend of letting the data consumer do complex geographical is_in calculations, as with the idea of tagging the address on the building outline and leaving the POI without. But here the calculation is simple enough for the consumer and an extra burden for the mapper.

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain team.
as The resolution will be set.
to The owner will be changed from team to the specified user.
The owner will change to Polarbear-j
as duplicate The resolution will be set to duplicate.The specified ticket will be cross-referenced with this ticket
The owner will be changed from team to anonymous.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.