Use general GeoInterface polygons as PolyRegions

[asinghvi17]

This would only require GeoInterface support, and would allow people to use predefined polygons a lot more easily, plus add support for holes in regions!

If you need a nice point in polygon test, Rasters.jl and GeometryOps.jl have some good implementations.

[asinghvi17]

Ah I see you already use PolygonOps - that's the same algorithm which GeometryBasics uses. I think we got the code from there!

[asinghvi17]

Another cool thing to do might be to implement the GeoInterface Feature interface for GeoRegions, so that Rasters.jl can easily consume them.

[natgeo-wong]

Could you link to the documentation on this? I'm very close to publishing v7, so this will probably go into v7.1 or v8.

[asinghvi17]

https://juliageo.org/GeoInterface.jl/dev/guides/developer/#Required-for-Feature(Collection)s

and you could just return a NamedTuple with all relevant attributes as the result of GI.properties(::GeoRegion), as well as the underlying GeometryBasics.Polygon as the result of GI.geometry(::GeoRegion). Maybe consider defining e.g. GI.crs as well?

This would also give a standard interface to get "a polygon" from a georegion, where rectregions could construct the polygon on the fly and return it in GI.geometry. Same for tilted regions, I imagine.

[natgeo-wong]

GeoRegions will construct the polygon automatically when the GeoRegion information is retrieved.

But are you asking if I can integrate and extend GI.properties and other things, that I can do eventually.

[asinghvi17]

Will PR if I end up doing this!