Copilot commented on code in PR #2355:
URL: https://github.com/apache/sedona/pull/2355#discussion_r2356353414
##########
python/sedona/spark/geopandas/tools/sjoin.py:
##########
@@ -261,7 +261,7 @@ def sjoin(
distance : number or array_like, optional
Distance(s) around each input geometry within which to query the tree
for the 'dwithin' predicate. If array_like, must be
- one-dimesional with length equal to length of left GeoDataFrame.
+ one-dimensional with length equal to length of left GeoDataFrame.
Review Comment:
Fixed the typo 'one-dimesional' to 'one-dimensional'.
##########
python/sedona/spark/geopandas/geodataframe.py:
##########
@@ -23,7 +23,6 @@
import shapely
import warnings
import numpy as np
-import shapely
import geopandas as gpd
Review Comment:
The duplicate import of `shapely` was correctly removed, improving code
cleanliness.
##########
python/sedona/spark/geopandas/tools/sjoin.py:
##########
@@ -276,7 +276,7 @@ def sjoin(
Examples
--------
- >>> groceries_w_communities = geopandas.sjoin(groceries, chicago)
+ >>> groceries_w_communities = sedona.spark.geopandas.sjoin(groceries,
chicago)
Review Comment:
The example import path should be consistent with the actual module
structure. Consider using the standard import pattern like `from
sedona.spark.geopandas.tools import sjoin` or showing the full qualified name
usage.
```suggestion
>>> from sedona.spark.geopandas.tools import sjoin
>>> groceries_w_communities = sjoin(groceries, chicago)
```
--
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]
