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

MultiPolygons are incorrectly encoded as a single Polygon #4

Closed
bcamper opened this issue Jan 30, 2015 · 3 comments
Closed

MultiPolygons are incorrectly encoded as a single Polygon #4

bcamper opened this issue Jan 30, 2015 · 3 comments

Comments

@bcamper
Copy link
Contributor

bcamper commented Jan 30, 2015

A MultiPolygon is being incorrectly encoded as a single Polygon. It appears the rings of the multipolygon are being flattened into one polygon, so for instance, a multipolygon with two single-ring polygons is being encoded as a single polygon with two rings, which means the second polygon is misinterpreted as a hole in the first polygon.

Instead, since the format doesn't natively support multipolygons, it probably makes the most sense to simply "denormalize" the multipolygon into multiple, individual polygon features, and copy the feature properties from the original multipolygon to each of the new individual polygons.

@bcamper
Copy link
Contributor Author

bcamper commented Jan 30, 2015

Top image: correctly encoded multipolygon.
Bottom image: incorrectly encoded multipolygon, which renders as one polygon.

screen shot 2015-01-30 at 1 02 39 pm
screen shot 2015-01-30 at 1 02 32 pm

@bcamper
Copy link
Contributor Author

bcamper commented Jan 30, 2015

cc @hkrishna @rmarianski

@hkrishna
Copy link
Contributor

hkrishna commented Feb 2, 2015

Closing this issue. #5 explodes multipolygons

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants