archaeodav / felttegn

Public repository for FeltTegn, a Qgis Plugin for processing survey data from archaeological excavations conducted by Danish Museums
GNU General Public License v3.0
5 stars 1 forks source link

Anlæg kan ikke deles, da det ikke er MultiPolygon #9

Closed nesnejretep closed 3 years ago

nesnejretep commented 3 years ago

Anlæg (og grøfter) kan med den valgte geometritype ikke deles i flere dele, og det mener jeg er et stort problem ved anlæg der f.eks. skæres af dræn. En løsning kunne være at anvende "Multipolygon" til disse i stedet for "Polygon" som det er tilfældet lige nu.

KajFRasmussen commented 3 years ago

Ja den er skidt. Det bliver MEGET bøvlet at lave forundersøgelser, hvor det i reglen ofte sker anlæg skærer igennem hinanden. Som det er bliver man nødt til at lave passager udenom f.eks. dræn, så områder kan hænge sammen...

Desuden er der ofte krydsende anlæg som det er meningsbærende at skille korrekt. Her giver det slet ikke mening at have almindelige polygoner.

KajFRasmussen commented 3 years ago

En lworkaraound for brugeren efter den ulovlige deling, kan være at skille synderen op i enkelanlæg som derfor får hver sin "fid", men samme anlægs- / grøft-nummer. Det er et komforttab, men data kan heldigvis stadig knyttes til andet data og analyseres.

KajFRasmussen commented 3 years ago

Anlægslag og Feltlag er nu af Multipolygon typen, som opfører sig mere som vi er vant til :-)

archaeodav commented 3 years ago

We're not going to force multipolygon at this stage- with unvalidated data like this it can cause problems with topological operations that we're using to validate geometries.

What we are going to do is add an option to convert to multipolygon with the upcoming create master / import to master function that will do some other topological stuff like trimming Anlæg polygons to Felt extents and handling duplicate geometries.