search

MarcoDuiker / QGIS_BGT_Import

The QGIS BGT Import plugin for importing Dutch BGT data.
Other
4 stars 1 forks source link

Geen (juiste) verwerking van MultiSurface? #10

Closed HermanAssink closed 4 years ago

HermanAssink commented 4 years ago

Marco, in o.a. bgt_overigbouwwerk.gml en bgt_pand.gml komen geometrieën van het type MultiSurface voor. Deze worden door de BGT Import plugin als Polygon in de Geopackage geladen. Bij Geoprocessing-operaties worden deze geometrieën als invalid aangemerkt. Is dit een bug in de plugin of doe ik iets fout?

MarcoDuiker commented 4 years ago

Kun jij mij voorzien van een gml en een object id van een object waar het misgaat?

HermanAssink commented 4 years ago

In bijgevoegde gml vind je MultiSurface geometrieën. Deze gml is gedownload met de beta-versie van de nieuwe BGT downloadfunctie (https://download.pdok.io/lv/bgt/viewer/) bgt_overigbouwwerk.zip

HermanAssink commented 4 years ago

Geometrievalidatie in QGIS geeft de foutmelding "Gat ligt buiten schil", bijv bij gml-id=baf17d3a7-1bdb-444b-852e-6940c79a49d5

MarcoDuiker commented 4 years ago

Inderdaad. Zorgwekkend genoeg laat ogr (die de feitelijke import verzorgt) hier een steekje vallen. De MultiSurface wordt ingelezen als een polygoon met gat. Hoe vaak zou dit ook in andere omstandigheden misgaan?

De oplossing zit in het forceren naar de Multi-vorm van een geometrie. Dan leest ogr het wel goed in. Ik zal dit in de plugin verwerken.

MarcoDuiker commented 4 years ago

hmm, het blijkt dat ogr het wel netjes doet. De plugin code behoefde een kleine aanpassing omdat deze de geometrie naar het single type forceerde.

HermanAssink commented 4 years ago

Dankjewel. Dit is een handige plugin waarmee je met "één druk op de knop" een BGT download kunt importeren in een Geopackage en met stijl en al geladen krijgt in QGIS.

MarcoDuiker commented 4 years ago

Inmiddels heb ik een nieuwe plugin in deze repo gezet (tag 3.12). Kun jij deze uittesten voordat ik hem aanbied voor de QGIS repo.

MarcoDuiker commented 4 years ago

Fixed in versie 3.13. Inmiddels ingediend bij de QGIS repo. Deze zal daar vanzelf verschijnen.