-
Notifications
You must be signed in to change notification settings - Fork 18
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
Proposal: allow Polygon
to be constructed from a vector of PointTrait
geometries
#168
Comments
Polygon
to be constructed from a vector of PointTrait
sPolygon
to be constructed from a vector of PointTrait
geometries
Polygon
to be constructed from a vector of PointTrait
geometriesPolygon
to be constructed from a vector of PointTrait
geometries
Just syntax? Or you want it to actually wrap that? We could do like the last PR and auto wrap it in a vector of LinearRing and call Polygon again. |
Yeah, the wrapping approach is what I was thinking. |
You'll run into issues like having to check the points form a ring, otherwise they're not a valid polygon. |
I think LinearRing doesn't check validity either ? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
That would allow a much cleaner syntax than the current
Polygon([LinearRing(...)])
...The text was updated successfully, but these errors were encountered: