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.

Sign up for GitHub

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

Jump to bottom

Centroid returns Point(Infinity, Infinity) for degenerate polygons #225

Closed
jagill opened this issue Jul 30, 2019 · 2 comments
Closed

Centroid returns Point(Infinity, Infinity) for degenerate polygons #225

jagill opened this issue Jul 30, 2019 · 2 comments
Assignees
@alocke
Labels
bug fixed_to_verify
Milestone
2.2.3

Comments

@jagill
Copy link

jagill commented Jul 30, 2019

In cases where degenerate polygons have near-0 area (like "MULTIPOLYGON (((153.492818 -28.13729, 153.492821 -28.137291, 153.492816 -28.137289, 153.492818 -28.13729)))"), the centroid calculation divides by 0.

I've opened up #224 as a possible solution.
@stolstov stolstov mentioned this issue Jul 30, 2019
Merged
alocke added a commit that referenced this issue Jul 31, 2019
@alocke
Fix formatting and added new tests for Issue #225
1c8b326
@jagill jagill mentioned this issue Aug 1, 2019
Closed
@jagill
Copy link
Author

jagill commented Aug 5, 2019

Great! Could we release a new version with these fixes? Then I can verify they fix our issue.

@stolstov stolstov self-assigned this Aug 6, 2019
@stolstov stolstov added the bug label Aug 6, 2019
@stolstov stolstov added this to the 2.2.3 milestone Aug 6, 2019
@stolstov stolstov assigned alocke and unassigned stolstov Aug 19, 2019
@jagill
Copy link
Author

jagill commented Sep 2, 2019

Verified fix.

@jagill jagill closed this as completed Sep 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alocke alocke

Labels
bug fixed_to_verify
Projects
None yet
Milestone
2.2.3
Development

No branches or pull requests
3 participants
@jagill @stolstov @alocke