Copilot commented on code in PR #2274:
URL: https://github.com/apache/sedona/pull/2274#discussion_r2271598005
##########
python/sedona/spark/geopandas/geodataframe.py:
##########
@@ -1607,17 +1612,17 @@ def to_file(
... "geometry": [Point(0, 0), LineString([(0, 0), (1, 1)])],
... "int": [1, 2]
... })
- >>> gdf.to_file(filepath, driver="geoparquet")
+ >>> gdf.to_file("output.parquet", driver="geoparquet")
With selected drivers you can also append to a file with ``mode="a"``:
- >>> gdf.to_file(filepath, driver="geojson", mode="a")
+ >>> gdf.to_file("output.geojson", driver="geojson", mode="a")
When the index is of non-integer dtype, ``index=None`` (default) is
treated as True,
writing the index to the file.
- >>> gdf = GeoDataFrame({"geometry": [Point(0, 0)]}, index=["a", "b"])
- >>> gdf.to_file(filepath, driver="geoparquet")
+ >>> gdf = GeoDataFrame({"geometry": [Point(0, 0), Point(1, 1)]},
index=["a", "b"])
Review Comment:
The example creates a GeoDataFrame with 2 geometry points but only 2 index
values for what appears to be a single-row example. The original code had
Point(0, 0) with index ["a", "b"] suggesting a mismatch. Consider using a
single point with single index or two points with appropriate indices.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
