Modify ↓

Opened 3 years ago

Closed 21 months ago

#4656 closed enhancement (fixed)

[Patch] Building inside Building

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

Description

I would like the validator to recognize buildings inside buildings. The import I am working on have done this at several occations, and browsing up all instances to create multipolygons are coumbersome. I use the validator actively to reduce the number of errors in the dataset, and recognizing these buildings in buildings is just one of many things that this validator should be able to do.

Attachments (1)

4656.patch​ (4.9 KB) - added by simon04 21 months ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 3 years ago by e2jk

Affected too.

Changed 21 months ago by simon04

comment:2 Changed 21 months ago by simon04

  • Summary changed from Building inside Building to [Patch] Building inside Building

Patch attached, multipolygons are not supported. The performance seems all right: ≈10k buildings of Innsbruck are validated within a few seconds.

Please test+report+comment! :)

comment:3 Changed 21 months ago by bastiK

You'll get plenty of testing, when it is committed... :)

comment:4 Changed 21 months ago by simon04

All right. So lets commit and await comments :-)

comment:5 Changed 21 months ago by simon04

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

In [4409/josm]:

fix #4656 - new validator test: building inside building

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.

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.