From df669f1afae8398696a9ca67abf298a82507f4a1 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 23 Feb 2026 21:55:29 +0000 Subject: [PATCH 1/3] Initial plan From a65f481c83da0850ab3096527918fb9aa6af2b9b Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 23 Feb 2026 21:58:08 +0000 Subject: [PATCH 2/3] Rename pin_write_gdf_parquet to pin_write_gdf_geoparquet for consistent suffixes Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> --- src/geopins/drivers/gdf/dispatch.py | 4 ++-- src/geopins/drivers/gdf/filetypes/parquet.py | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/geopins/drivers/gdf/dispatch.py b/src/geopins/drivers/gdf/dispatch.py index d598e0e..1a0f961 100644 --- a/src/geopins/drivers/gdf/dispatch.py +++ b/src/geopins/drivers/gdf/dispatch.py @@ -7,7 +7,7 @@ from geopins.drivers.gdf.filetypes.gpkg import pin_read_gdf_gpkg, pin_write_gdf_gpkg from geopins.drivers.gdf.filetypes.parquet import ( pin_read_gdf_geoparquet, - pin_write_gdf_parquet, + pin_write_gdf_geoparquet, ) from geopins.drivers.infer import infer_driver_info from geopins.interfaces import PinReadKwargDict, PinWriteKwargDict @@ -140,7 +140,7 @@ def pin_write_gdf( # noqa: PLR0913 if type_ in ("geopackage", "gpkg"): return pin_write_gdf_gpkg(x, board=board, **kwargs) elif type_ == "parquet": - return pin_write_gdf_parquet(x, board=board, **kwargs) + return pin_write_gdf_geoparquet(x, board=board, **kwargs) else: raise_driver_not_supported(type_, cls=board.__class__, mode="write") raise AssertionError # Change to assert_never after deprecating 3.11 support diff --git a/src/geopins/drivers/gdf/filetypes/parquet.py b/src/geopins/drivers/gdf/filetypes/parquet.py index b4d83fb..f7c4e94 100644 --- a/src/geopins/drivers/gdf/filetypes/parquet.py +++ b/src/geopins/drivers/gdf/filetypes/parquet.py @@ -52,7 +52,7 @@ def pin_read_gdf_geoparquet( return gpd.read_parquet(filename) -def pin_write_gdf_parquet( # noqa: PLR0913 +def pin_write_gdf_geoparquet( # noqa: PLR0913 # N.B. match pins.boards.BaseBoard.pin_write signature x: GeoDataFrame, name: str | None = None, From 32b86965f3a4eebc397efb4a2abb24391223a0bc Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 23 Feb 2026 22:00:59 +0000 Subject: [PATCH 3/3] Use _parquet suffix consistently: rename pin_read_gdf_geoparquet to pin_read_gdf_parquet Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> --- src/geopins/drivers/gdf/dispatch.py | 8 ++++---- src/geopins/drivers/gdf/filetypes/parquet.py | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/geopins/drivers/gdf/dispatch.py b/src/geopins/drivers/gdf/dispatch.py index 1a0f961..03ff5c2 100644 --- a/src/geopins/drivers/gdf/dispatch.py +++ b/src/geopins/drivers/gdf/dispatch.py @@ -6,8 +6,8 @@ from geopins.drivers.exceptions import raise_driver_not_supported from geopins.drivers.gdf.filetypes.gpkg import pin_read_gdf_gpkg, pin_write_gdf_gpkg from geopins.drivers.gdf.filetypes.parquet import ( - pin_read_gdf_geoparquet, - pin_write_gdf_geoparquet, + pin_read_gdf_parquet, + pin_write_gdf_parquet, ) from geopins.drivers.infer import infer_driver_info from geopins.interfaces import PinReadKwargDict, PinWriteKwargDict @@ -74,7 +74,7 @@ def _pin_read_gdf( if filetype == "gpkg": return pin_read_gdf_gpkg(board=board, **kwargs) elif filetype == "parquet": - return pin_read_gdf_geoparquet(board=board, **kwargs) + return pin_read_gdf_parquet(board=board, **kwargs) else: raise_driver_not_supported(filetype, cls=board.__class__, mode="read") raise AssertionError # Change to assert_never after deprecating 3.11 support @@ -140,7 +140,7 @@ def pin_write_gdf( # noqa: PLR0913 if type_ in ("geopackage", "gpkg"): return pin_write_gdf_gpkg(x, board=board, **kwargs) elif type_ == "parquet": - return pin_write_gdf_geoparquet(x, board=board, **kwargs) + return pin_write_gdf_parquet(x, board=board, **kwargs) else: raise_driver_not_supported(type_, cls=board.__class__, mode="write") raise AssertionError # Change to assert_never after deprecating 3.11 support diff --git a/src/geopins/drivers/gdf/filetypes/parquet.py b/src/geopins/drivers/gdf/filetypes/parquet.py index f7c4e94..f83a63b 100644 --- a/src/geopins/drivers/gdf/filetypes/parquet.py +++ b/src/geopins/drivers/gdf/filetypes/parquet.py @@ -16,7 +16,7 @@ from pins.meta import Meta -def pin_read_gdf_geoparquet( +def pin_read_gdf_parquet( name: str, version: str | None = None, hash: str | None = None, # noqa: A002 @@ -52,7 +52,7 @@ def pin_read_gdf_geoparquet( return gpd.read_parquet(filename) -def pin_write_gdf_geoparquet( # noqa: PLR0913 +def pin_write_gdf_parquet( # noqa: PLR0913 # N.B. match pins.boards.BaseBoard.pin_write signature x: GeoDataFrame, name: str | None = None,