Modify

Opened 5 years ago

Last modified 5 years ago

#9376 new enhancement

For building=roof without layer tag, intersecting with roads, footways, cycleways etc validator should mention that typical roof is above other objects and it should have "layer=1" tag

Reported by: mkoniecz Owned by: team
Priority: normal Milestone:
Component: Core validator Version:
Keywords: Cc:

Description (last modified by mkoniecz)

"Crossing ways (1)" is quite mysterious "Roof crossing with other objects, maybe it should have layer=1 tag?" (or something like this) would be smarter.

Current report is quite confusing (for me, reporter of #9362 and probably others).

This suggestion definitely should be given for crossing with highway=*, and must not be given for crossing with buildings. Not sure about other types of objects.

Note that after adding this special case layerless, intersecting roofs should not appear also in "Crossing ways" category.

I encountered this difficulty on osmwww:browse/way/248676077

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2013-11-25 02:34:44
Last Changed Author: Don-vip
Revision: 6413
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2013-11-25 02:19:43 +0100 (Mon, 25 Nov 2013)
Last Changed Rev: 6413

Identification: JOSM/1.5 (6413 en_GB) Windows 7 32-Bit
Memory Usage: 108 MB / 247 MB (31 MB allocated, but free)
Java version: 1.7.0_45, Oracle Corporation, Java HotSpot(TM) Client VM
Dataset consistency test: No problems found

Plugin: OpeningHoursEditor (29854)
Plugin: notes (v0.6)

Attachments (0)

Change History (5)

comment:1 Changed 5 years ago by mkoniecz

Summary: For building=roof without layer tag, with intersecting with roads, footways, cycleways etc validator should mention that typical roof is above other objects and it should have "layer=1" tagFor building=roof without layer tag, intersecting with roads, footways, cycleways etc validator should mention that typical roof is above other objects and it should have "layer=1" tag

comment:2 Changed 5 years ago by mkoniecz

Description: modified (diff)

comment:3 Changed 5 years ago by mkoniecz

Description: modified (diff)

comment:4 in reply to:  description Changed 5 years ago by aceman

... and must not be given for crossing with buildings.

Why? A building (of any type, without layer tag) crossing into other buildings (or any ways) seems like an error and is currently reported. Roof is no exception without a layer.

comment:5 Changed 5 years ago by mkoniecz

"must not" applies to displaying a new, more specific warning message.

building/building crossing would receive current "Crossing ways".

Modify Ticket

Change Properties
Set your email in Preferences
Action
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 mkoniecz
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.