Modify

Opened 20 months ago

Closed 9 months ago

#7034 closed defect (fixed)

building inside building false positiv

Reported by: julien.balas@… Owned by: team
Priority: normal Component: Core validator
Version: Keywords:
Cc:

Description

i don't know if the validator handle incorrectly "wall=no" or if the algorithm is too strict, but ther are some false alerts.

for example
http://www.openstreetmap.org/browse/way/81189416
and
http://www.openstreetmap.org/browse/way/81188636

Attachments (4)

area_inside_MP.osm (1.9 KB) - added by mdk 20 months ago.
Here is an example .osm file. Just validate.
7034.osm (23.2 KB) - added by JulienBalas 20 months ago.
a small test file. None of the small way "linked" to the big way should be reported as "in"
test.osm (2.0 KB) - added by jcr83 18 months ago.
Another false positive example.
covered_yard.osm (4.0 KB) - added by julien@… 18 months ago.
2 tests cases for a covered yard

Download all attachments as: .zip

Change History (11)

comment:1 Changed 20 months ago by mdk

An other fase positive is a building inside the inner of a building multipolygon:

 _______________̣_______    MP outer
|  MP building=yes     |
|  __________________  |   MP inner
| |                  | |
| |  ______________  | |
| | |              | | |   Building inside MP inner
| | | building=yes | | |   Validator mark this as "building in building" error
| | |              | | |
| | |______________| | |
| |__________________| |
|______________________|

mdk

comment:2 Changed 20 months ago by skyper

Think, this has to do with #4582.

It is always better to have a small .osm file test file, as objects might change on the server.

Changed 20 months ago by mdk

Here is an example .osm file. Just validate.

Changed 20 months ago by JulienBalas

a small test file. None of the small way "linked" to the big way should be reported as "in"

Changed 18 months ago by jcr83

Another false positive example.

comment:3 Changed 18 months ago by Don-vip

In [4738/josm]:

see #7034 - building inside building false positives

comment:4 Changed 18 months ago by Don-vip

Problem fixed for 7034.osm and test.osm. The first example needs another fix.

comment:5 Changed 18 months ago by Don-vip

  • Resolution set to fixed
  • Status changed from new to closed

In [4747/josm]:

fix #7034 - building inside building false positives

Changed 18 months ago by julien@…

2 tests cases for a covered yard

comment:6 Changed 18 months ago by julien@…

  • Resolution fixed deleted
  • Status changed from closed to reopened

The first example "area inside mp" seems to be also fixed, a building in a yard is correctly handled.
I'v added 1 new test file about covered yard (very common in french school)
The 2 example should not trigger the rule.

comment:7 Changed 9 months ago by Don-vip

  • Resolution set to fixed
  • Status changed from reopened to closed

In 5522/josm:

fix #7034, fix #8067 - Rework "Building inside Building" test from scratch

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


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

 
Note: See TracTickets for help on using tickets.