zhangfengcdt opened a new issue, #2273:
URL: https://github.com/apache/sedona/issues/2273
Environment
- Component: Python Documentation
- Files Affected:
- python/sedona/spark/geopandas/geoseries.py
- python/sedona/spark/geopandas/geodataframe.py
- mkdocs.yml
- README.md
Bug Description
Users attempting to follow the documentation examples encounter various
errors.
Steps to Reproduce
1. Navigate to Sedona Python GeoPandas documentation
2. Copy any of the affected example code snippets
3. Attempt to run the code in a Python environment
4. Observe various errors depending on the specific example
Expected Behavior
All documentation examples should execute successfully when copied and run
by users, with
all necessary imports included and variables properly defined.
Actual Behavior
Examples fail with various errors including import errors, undefined
variables, and
incorrect class references.
Detailed Issues Found
Critical Issues
Incorrect Class References
- geoseries.py:1761 - Uses geopandas.GeoSeries.from_wkb() instead of
Sedona's
GeoSeries.from_wkb()
- geoseries.py:1904 - Uses geopandas.GeoSeries.from_xy() instead of
Sedona's
GeoSeries.from_xy()
- geodataframe.py:962 - Uses geopandas.GeoDataFrame() instead of Sedona's
GeoDataFrame()
- geodataframe.py:1425 - Uses geopandas.read_file() instead of appropriate
Sedona method
Missing Import Statements
- geoseries.py:344 - Example uses pd.Series without import pandas as pd
- geodataframe.py:961 - Missing from sedona.spark.geopandas import
GeoDataFrame
Undefined Variables
- geoseries.py:446,566 - Examples reference s.crs but s variable not
defined in context
- geodataframe.py:1614,1618,1624 - Uses undefined filepath variable
Medium Priority Issues
Inconsistent Formatting
- geoseries.py:322 - Uses Examples: instead of standard Examples\n--------
format
Inconsistent Import Aliases
- geoseries.py:2451 - Uses geopandas.read_file() instead of gpd.read_file()
- geodataframe.py:1426 - Inconsistent usage between geopandas and gpd alias
Data/Index Mismatches
- geodataframe.py:1623 - Creates GeoDataFrame with 1 geometry but 2 index
values
--
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]
