search

ObjectVision / GeoDMS

Source code for the GeoDMS software
Other
7 stars 1 forks source link

operator voor het omdraaien van de winding order #302

Open MaartenHilferink opened 1 year ago

MaartenHilferink commented 1 year ago

Discussed in https://github.com/ObjectVision/GeoDMS/discussions/63

Originally posted by **jipclaassens** January 9, 2023 Maarten Hilferink [28 Feb 2022 at 17:09](https://trello.com/c/Ov4YGx7k/53-operator-voor-het-omdraaien-van-de-winding-order#comment-621cf3c628c2fe12a2dc3890) wellicht ook al OK met split_polygon [Reply](https://trello.com/c/Ov4YGx7k/53-operator-voor-het-omdraaien-van-de-winding-order#) Maarten Hilferink [28 Feb 2022 at 17:08](https://trello.com/c/Ov4YGx7k/53-operator-voor-het-omdraaien-van-de-winding-order#comment-621cf38a0a150a42f2153ebc) eerst bg_inflate, bg_intersect, remove_lakes, dan cleaning heroverwegen. [Reply](https://trello.com/c/Ov4YGx7k/53-operator-voor-het-omdraaien-van-de-winding-order#) Jip Claassens [28 Feb 2022 at 17:05](https://trello.com/c/Ov4YGx7k/53-operator-voor-het-omdraaien-van-de-winding-order#comment-621cf2d7ec98367313762cbb) of een fix geometry
jipclaassens commented 5 months ago

Clean polygon geometry fixes most of the issues (using a partitioned_union_polygon(geometry, id(.)) ).

Fix geometry in ArcGIS of BAG panden: image

Fix geometry in ArcGIS of BAG panden after apply the partitioned_union_polygon: image (1)

MaartenHilferink commented 3 months ago

@jipclaassens Kun je de resterende geometry die volgens ArcGIS gerepareerd moest worden (feature 354574, of een equivalent uit een recente BAG) voor en na partitioned_union_polygon(geometry, id(.)) ). hier posten ? En dan dit issue weer aan mij assignen? Benieuwd hoe het zit met die self intersection en of dat in een test-case gereproduceerd kan worden.

n.b. wellicht is dit issue al opgelost met commit dd 07-03-2024 met een "adapt for polygons with touching rings" -> v14.13.2

MaartenHilferink commented 3 months ago

Voor ophalen en heroriënteren van meren hebben we dus outer_multi_polygon(geometry) - geometry. Verder zou ik topologische issues met polygonen zoveel mogelijk bij de bron oplossen, zoals we o.a. met de latlong coördinaat volgorde (die omgekeerd o.a. een verkeerde winding order geeft).

jipclaassens commented 2 months ago

Na partitioned_union_polygon(geometry_mm, id(.)) nog steeds 2 featues in regio Amsterdam die incorrect ring ordering hebben in v14.16.0

image

before and after files download here: https://surfdrive.surf.nl/files/index.php/s/Zd7BIIgOCIrcz7W