Opened 2 years ago
Closed 2 years ago
#22556 closed enhancement (wontfix)
[patch] Complain about building=roof + addr:*
Reported by: | Famlam | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core validator | Version: | |
Keywords: | roof fuel address | Cc: |
Description (last modified by )
Currently there are about 50k cases where addr:* tags are placed on a building with building=roof
(not to be confused with building:part=roof)
Addresses are typically not located on (in?) the roof of the building, but rather point at the object that's under the roof. Additionally, as the majority of these tags seem to be for amenity=fuel
(60k in combination with building=roof
), the wiki page of amenity=fuel
also indicates one should make a separate area for the roof.
https://wiki.openstreetmap.org/wiki/Tag:amenity%3Dfuel
https://wiki.openstreetmap.org/wiki/Tag:building%3Droof
Hence, I would propose to add either:
way[building=roof][amenity=fuel] { throwOther: tr("{0} together with {1}, usually {1} is located underneath the {2}", "{0.tag}", "{1.tag}", "{0.value}"); group: tr("suspicious tag combination"); }
or
way[building=roof][/^addr:/] { throwOther: tr("{0} together with {1}, usually addresses refer to objects underneath the {2}", "{0.tag}", "{1.tag}", "{0.value}"); group: tr("suspicious tag combination"); }
Attachments (0)
Change History (9)
comment:1 by , 2 years ago
comment:2 by , 2 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
I'll close it as WontFix for now then.
comment:4 by , 2 years ago
Resolution: | wontfix |
---|---|
Status: | closed → reopened |
How about an informational warning (Others)?
I'm in favor, of course :)
comment:5 by , 2 years ago
Description: | modified (diff) |
---|
comment:6 by , 2 years ago
Description: | modified (diff) |
---|---|
Keywords: | address added |
use one more placeholder
comment:7 by , 2 years ago
@taylor.smock, what's your opinion on a 'throwOther' warning as Skyper proposes? If you oppose, I'll close the issue again, otherwise it'd probably stay open forever :)
comment:8 by , 2 years ago
A gas station POI is not necessarily located underneath the roof; that would be a osmwiki:Tag:man_made=fuel_pump. The wiki mentions a "fueling area", which has been interpreted differently by different mappers. However, this is no different than how some mappers dual-tag the nearby building as a convenience store while others prefer to keep it as a separate node inside the building. By this logic, a convenience store is also inside the building, not identical to it, but a warning about dual-tagging any building with an amenity would be quite noisy.
comment:9 by , 2 years ago
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
I don't mind seeing this in an external validator, but I really don't think adding it to JOSM will do anyone any favors, especially since a fairly common tagging practice.
Out of a semi-random sampling of building=roof
and amenity=fuel
(osm-download.bounds=33.1375512;-113.7304687;45.0269505;-92.2851562
), 4160 have both building=roof
and amenity=fuel
.
There were about 40300 building=roof
objects and ~12700 amenity=fuel
objects. So about 1/3 amenity=fuel
objects also happen to be on building=roof
objects.
It doesn't help that most of the time the roof(s) are also the fueling area.
I get where you are coming from, with a caveat that in some cases the
building=roof
section will be larger than the service building and cover or almost cover the service building (I have a couple of examples in my area, although one is in the process of being torn down and rebuilt).I'm disinclined to apply the patch for
amenity=fuel
objects because of that.You might also want to see how people actually tag fuel stations -- I've seen people put all the tags on the "roof" of the fueling area, so I suspect osmwiki:Tag:amenity=fuel does not conform to current tagging practices. This also means that putting the
addr
tags on the roof makes sense. In some cases.