Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests for geometry operator correctness and performance #775

Open
jipclaassens opened this issue Jul 25, 2024 · 7 comments
Open

Tests for geometry operator correctness and performance #775

jipclaassens opened this issue Jul 25, 2024 · 7 comments
Assignees
Labels
OVSRV09 testcase available on ovsrv09

Comments

@jipclaassens
Copy link
Member

image

@jipclaassens
Copy link
Member Author

bg_split_union_polygon()

image

@jipclaassens
Copy link
Member Author

En waar zijn deze 4 lakes ?

image

@jipclaassens
Copy link
Member Author

jipclaassens commented Jul 25, 2024

Met bg_union_polygon()

image

@jipclaassens jipclaassens changed the title bg_union_polygon loses polygons Tests for boost geometry operator correctness Jul 29, 2024
@jipclaassens
Copy link
Member Author

jipclaassens commented Jul 29, 2024

OVSRV09:
C:\ProjDir_Jip\_Tools\GeoDMS-Test\Polygons\main.dms

LargeSetWithSmallPolygons == 132,199 panden gebouwd na 1990 in Amsterdam, dus vaak niet aangrenzend
SmallSetWithLargePolygons == 345 gemeente (y2022), dus meestal aangrenzend.


LargeSetWithSmallPolygons

  • bg_overlay_count_islands = 135,337
  • bp_overlay_count_islands = 135,336
  • bg_overlay_area = 3.28845e7 m2
  • bp_overlay_area = 0 km2
  • bg_inflate_ratio_correctness = 132199/132199
  • bg_deflate_ratio_correctness = 132127/132199
  • bg_polygon_count_islands = 132,199
  • bp_polygon_count_islands = 132,199
  • bg_polygon_area = 1.31718e8 km2
  • bp_polygon_area =1.31718e8 km2
  • bg_union_area = 1.17084e7 km2
  • bp_union_area = 1.17135e7 km2
  • bg_split_count_islands = 132,192
  • bp_split_count_islands = 132,192
  • bg_split_area = 1.31717e8 km2
  • bp_split_area = 1.31717e8 km2
  • bg_split_union_count_islands = 4294
  • bp_split_union_count_islands = 4294
  • bg_split_union_area= 2.63671e7 km2
  • bp_split_union_area= 2.63670e7 km2

SmallSetWithLargePolygons

  • bg_overlay_count_islands = 43,000
  • bp_overlay_count_islands = 43,010
  • bg_overlay_area = 3.17851e10 km2
  • bp_overlay_area = 0 km2
  • bg_inflate_ratio_correctness = 345/345
  • bg_deflate_ratio_correctness = 345/345
  • bg_polygon_count_islands = 345
  • bp_polygon_count_islands = 345
  • bg_polygon_area = 3.5288e10 km2
  • bp_polygon_area =3.5288e10 km2
  • bg_union_area = 3.35641e10 km2
  • bp_union_area = 3.50804e10 km2
  • bg_split_count_islands = 705
  • bp_split_count_islands = 705
  • bg_split_area = 3.5288e10 km2
  • bp_split_area = 3.5288e10 km2
  • bg_split_union_count_islands = 290
  • bp_split_union_count_islands = 200
  • bg_split_union_area = 3.30058e10 km2
  • bp_split_union_area = 3.51374e10 km2

@jipclaassens jipclaassens changed the title Tests for boost geometry operator correctness Tests for geometry operator correctness and performance Aug 6, 2024
@jipclaassens
Copy link
Member Author

image

@basvanbemmel
Copy link

Gaat dit nu om een nieuwe operator of om de (niet juiste) werking van een bestaande operator (operator correctness and performance?).

@jipclaassens
Copy link
Member Author

jipclaassens commented Sep 2, 2024

Gaat dit nu om een nieuwe operator of om de (niet juiste) werking van een bestaande operator (operator correctness and performance?).

@basvanbemmel beide. De cgal_ en geos_ varianten zijn nieuw. Daarnaast een aantal nieuwe bg_ operatoren en hernoeming van de bp_ varianten. Daarnaast blijkt op dit moment de bg_union operatoren niet helemaal juist te zijn.

Oftewel, cgal/geos/bg is allemaal experimenteel.

de bp_ (oftewel de oude polygoon operatoren) zijn nog steeds juist. Maarten heeft de naamgeving veranderd en de performance verbeterd

@jipclaassens jipclaassens added the OVSRV09 testcase available on ovsrv09 label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OVSRV09 testcase available on ovsrv09
Projects
None yet
Development

No branches or pull requests

3 participants