cuspatial icon indicating copy to clipboard operation
cuspatial copied to clipboard

[FEA] Modify `read_polygon_shapefile` to return a `GeoSeries` instead of a tuple.

Open thomcom opened this issue 3 years ago • 2 comments

Describe the bug read_polygon_shapefile returns the basic elements of a GeoSeries instead of a GeoSeries. To solidify the API, let's have it return the GeoSeries itself.

thomcom avatar Sep 30 '22 15:09 thomcom

What is the C++ equivalent of a GeoSeries?

harrism avatar Oct 01 '22 08:10 harrism

I'm really referring to the python interface in this issue, which was my oversight. There isn't a parallel for GeoSeries yet at the C++ level.

In the general case the C++ equivalent is: A buffer of xy Points A buffer of xy Points and an offsets buffer of MultiPoint parts A buffer of xy coordinates, an offsets buffer of LineString start positions and an offsets buffer of MultiLineString parts A buffer of xy coordinates, an offsets buffer of Polygon rings, an offsets of Polygon start positions, and a buffer of MultiPolygon parts.

read_polygon_offsets returns the first three objects in the Polygon set.

thomcom avatar Oct 02 '22 18:10 thomcom