jiayuasu opened a new pull request, #2732: URL: https://github.com/apache/sedona/pull/2732
## Did you read the Contributor Guide? - Yes, I have read the [Contributor Rules](https://sedona.apache.org/latest/community/rule/) and [Contributor Development Guide](https://sedona.apache.org/latest/community/develop/) ## Is this PR related to a ticket? - Yes, and the PR name follows the format `[GH-XXX] my subject`. Closes #2731 ## What changes were proposed in this PR? Implement 5 GeoSeries functions for the geopandas compatibility module. These were previously blocked by upstream bugs (#2720, #2721, #2722), now fixed in #2730. Distance metrics: - `frechet_distance(other, align, densify)` -> `ST_FrechetDistance` (densify not supported by Sedona, raises `NotImplementedError`) - `hausdorff_distance(other, align, densify)` -> `ST_HausdorffDistance` (supports `densify` parameter via `densityFrac`) Binary predicates: - `geom_equals(other, align)` -> `ST_Equals` Linear referencing: - `interpolate(distance, normalized)` -> `ST_LineInterpolatePoint` (when `normalized=False`, divides distance by `ST_Length` to convert to fraction) - `project(other, normalized, align)` -> `ST_LineLocatePoint` (when `normalized=False`, multiplies fraction by `ST_Length` to convert to absolute distance) Each function follows the established pattern from #2701, #2710, and #2726: - base.py: Docstring with examples + `_delegate_to_geometry_column` call - geoseries.py: ST function call via `_row_wise_operation` - test_geoseries.py: Unit test with expected values + GeoDataFrame delegation check - test_match_geopandas_series.py: Comparison test against real geopandas output ## How was this patch tested? 10 new tests (5 unit tests + 5 match tests) all passing locally. ## Did this PR include necessary documentation updates? - No, this PR does not affect any public API so no need to change the documentation. -- 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]
