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

Add LargestEmptyCircle handling of polygon obstacles #939

Merged
merged 8 commits into from
Jul 18, 2023

Conversation

dr-jts
Copy link
Contributor

@dr-jts dr-jts commented Jul 18, 2023

Fixes LargestEmptyCircle to handle polygonal obstacles. The centre of the LEC is constrained to be outside of any polygonal obstacles. Previously polygonal obstacles were only handled as linear constraints, which allowed the LEC centre to lie inside polygons.

Port of locationtech/jts#988

Fixes #855

@dr-jts dr-jts added the Enhancement New feature or feature improvement. label Jul 18, 2023
@dr-jts dr-jts self-assigned this Jul 18, 2023
@dr-jts dr-jts merged commit 2f9e753 into libgeos:main Jul 18, 2023
@dr-jts dr-jts deleted the add_lec_polygon_obstacles branch July 18, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or feature improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LargestEmptyCircle: is the boundary considered as obstacle as well?
1 participant