Modify

Opened 10 years ago

Closed 8 years ago

#4656 closed enhancement (fixed)

[Patch] Building inside Building

Reported by: anonymous Owned by: team
Priority: normal Milestone:
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 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 9 years ago by e2jk

Affected too.

Changed 8 years ago by simon04

Attachment: 4656.patch added

comment:2 Changed 8 years ago by simon04

Summary: Building inside Building[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 8 years ago by bastiK

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

comment:4 Changed 8 years ago by simon04

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

comment:5 Changed 8 years ago by simon04

Resolution: fixed
Status: newclosed

In [4409/josm]:

fix #4656 - new validator test: building inside building

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.