Opened 6 weeks ago
Last modified 4 weeks ago
#24148 new defect
Validator warns about "overlapping buildings" when a building overlaps a building:part
Reported by: | tguen | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core validator | Version: | tested |
Keywords: | template_report buiding building:part overlap | Cc: | tguen |
Description
What steps will reproduce the problem?
- download a building with an overlapping building:part
- click the "validator" button in the validator results panel
What is the expected result?
Warning does not appear
What happens instead?
Warning for "overlapping buildings"
Please provide any additional information below. Attach a screenshot if possible.
tooltip for warning says:
Test: tag checker (MapCSS based)
Appears to be defined here:
resources/data/validator/geometry.mapcss:195
I can't think of any reason this would be appropriate. Anyone know why it was added?
Revision:19307 Is-Local-Build:true Build-Date:2025-02-15 00:14:11 Identification: JOSM/1.5 (19307 SVN en) Linux Arch Linux Memory Usage: 356 MB / 7960 MB (137 MB allocated, but free) Java version: 21.0.6+7, Arch Linux, OpenJDK 64-Bit Server VM Look and Feel: javax.swing.plaf.metal.MetalLookAndFeel Screen: :0.0 3840x2160x[Multi depth]@120Hz (scaling 1.00×1.00) Maximum Screen Size: 3840×2160 Best cursor sizes: 16×16→16×16, 32×32→32×32 Environment variable LANG: en_US.UTF-8 System property file.encoding: UTF-8 System property sun.jnu.encoding: ANSI_X3.4-1968 Locale info: en_US Numbers with default locale: 1234567890 -> 1234567890 Desktop environment: KDE VM arguments: [-Dawt.useSystemAAFontSettings=on, --add-exports=java.base/sun.security.action=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.plugins.jpeg=ALL-UNNAMED, --add-exports=java.desktop/com.sun.imageio.spi=ALL-UNNAMED] Dataset consistency test: No problems found Last errors/warnings: - 00030.337 W: java.io.IOException: Attribution is not loaded yet - 00030.337 W: java.io.IOException: Attribution is not loaded yet - 00030.337 W: java.io.IOException: Attribution is not loaded yet - 00030.337 W: java.io.IOException: Attribution is not loaded yet - 00030.337 W: java.io.IOException: Attribution is not loaded yet - 00030.338 W: java.io.IOException: Attribution is not loaded yet - 00030.338 W: java.io.IOException: Attribution is not loaded yet - 00030.338 W: java.io.IOException: Attribution is not loaded yet - 00030.338 W: java.io.IOException: Attribution is not loaded yet - 00030.338 W: java.io.IOException: Attribution is not loaded yet
Attachments (0)
Change History (3)
comment:1 by , 4 weeks ago
comment:2 by , 4 weeks ago
Yes in principle it is the rule that the object with building=*
includes all building:part=*
and therefore should not overlap. I have to admit that the concept can make it hard to get the footprint of the building at ground level. Additionally the wiki does not handle the situation in case of underground levels exceeding all other parts.
comment:3 by , 4 weeks ago
Keywords: | buiding building:part overlap added |
---|
https://wiki.openstreetmap.org/wiki/Key:building:part
Dont use building:part without building
It's not accepted, it seems