Skip to content

[EPIC] Implement More Geopandas Functions #2230

Description

@petern48

This is an umbrella issue that will encompass many other issues for implementing additional geopandas functionality.

Coverage and remaining work

The coverage snapshot in #3400 (Sedona 2.0.0 development, compared with the GeoPandas 1.1.4 catalog) is 160 of 175 APIs available or partially available (91.4%). See the GeoPandas API coverage reference for the complete inventory, counting rules, and per-API limitations.

Checked items below mean an implementation is available, possibly with limitations; they do not imply complete parameter or behavioral parity. The remaining unimplemented API entries and the intentionally unsupported interfaces are listed below. This epic remains open for that work and for extending partial implementations.

Setup:

Implementation Steps

  1. Pick a GeoSeries function to implement from the (subset) list below or from the full list in the original documentation.
  2. Look for a similar function that has been implemented in Sedona Geopandas already - Chances are, you'll just need to use one of the helper functions _query_geometry_column or _row_wise_operations
  3. Implement function in geoseries.py and base.py. Usually, this is as simple as calling the corresponding Sedona function (e.g ST_IsRing for .is_ring(). The base.py part is important so geodataframe can also execute it.
  4. Add documentation under the base.py implementation by copy and pasting from geopandas docstrings.
  5. Add tests to test_geoseries.py and test_match_geopandas_series.py following similar conventions.
  • In test_geoseries.py, include the example in the docstring as part of the test.
  • For test_match_geopandas_series.py, follow the convention of similar functions. Most of the time, you just run the function on each of the self.geoms and compare output with geopandas using the helper functions.
  1. Create a new issue on GitHub. Either create a subissue of this [EPIC] issue or explicitly mention this issue. Then, submit a PR linked to the issue you just created (not this issue). Reference this epic as Part of #2230; do not use Closes #2230 for partial work. Go ahead and CC @petern48 on your PR too, so I can review
  2. Repeat!

Note about AI use: I have zero problem with contributors using AI to help them write their code faster. HOWEVER, I strongly urge you to at least run your code and tests locally on your laptop, you're not going to have a good time trying to iterate on failed CI runs. If you need help fixing your code to pass the tests, it's fine to ask for help. But if you constantly are blindly pushing changes an LLM told you and waiting for CI to tell you if your code is correct, you will be putting an annoying pain on reviewers, since we need to manually approve your CI to run after each change. I also recommend you learn to do your first one on your own, by following an example PR or implementation. It should be fairly easy to figure out.

Example PR: #2232

Implementation checklist (not exhaustive), with existing functions to model implementations on:

Unary predicates (boolean operations): model off of is_empty

Binary predicates

  • relate (ST_Relate)
  • relate_pattern (non-trivial)

Set theoretic methods: model off of difference

Aggregations (model off of union_all)

  • intersection_all (ST_Intersection_Aggr)
  • voronoi_polygons (ST_VoronoiPolygons)
  • polygonize (ST_Polygonize)
  • delaunay_triangles (ST_DelaunayTriangles)
  • build_area (ST_BuildArea)
  • explode
  • dissolve

Other

  • convex_hull (ST_ConvexHull)
  • concave_hull (ST_ConcaveHull)
  • force_2d (ST_Force_2D)
  • force_3d (ST_Force3D)
  • frechet_distance (ST_FrechetDistance)
  • hausdorff_distance (ST_HausdorffDistance)
  • minimum_bounding_circle (ST_MinimumBoundingCircle)
  • minimum_bounding_radius (ST_MinimumBoundingRadius)

Joins (model off of sjoin)

This doesn't include everything. It's also possible I misgrouped a few functions, but here's a good starting list. Ping me when you we need to add more to the list.


Additional API checklist

Binary predicates (additional): model off of contains

  • geom_equals
  • geom_equals_exact
  • disjoint
  • contains_properly

General methods and attributes

Constructive methods and attributes

Linestring operations

Affine transformations

Overlay operations

  • GeoDataFrame.overlay
  • GeoDataFrame.clip
  • GeoSeries.clip

Plotting

Indexing

  • GeoDataFrame.cx
  • GeoSeries.cx

Tools (top-level functions)

  • geopandas.points_from_xy
  • geopandas.tools.collect
  • geopandas.tools.geocode
  • geopandas.tools.reverse_geocode
  • geopandas.clip
  • geopandas.overlay

Serialization / IO / conversion

Partial implementations and remaining scope

  • sjoin_nearest is implemented by [GH-3182] GeoPandas: Add point-only inner sjoin_nearest #3184 for point-only inner joins, with distance_col and an optional max_distance result filter. It uses the existing Sedona KNN tie configuration, which does not guarantee every equidistant or coincident row. Non-point geometries, left/right joins, exclusive matching, custom suffixes, and complete GeoPandas tie semantics remain outside this initial scope.
  • Other implemented APIs also have parameter, result, or execution limitations. Consult the coverage reference before choosing follow-up work.

Deferred infrastructure projects

These APIs remain unimplemented until the required distributed engine infrastructure is designed and reviewed. They should not be approximated with driver collection, Python UDFs, or session-wide configuration changes.

  • get_precision and set_precision: require persistent per-geometry precision metadata in Sedona serialization/UDTs, native arbitrary-grid and mode support, and precision propagation across operations.

Intentionally unsupported for distributed execution

These APIs return or iterate Python feature objects on the driver and are not planned while Sedona requires scalable distributed execution. Revisit them only if a distributed-compatible contract becomes available. See #3132 for the decision.

  • GeoDataFrame.iterfeatures
  • GeoDataFrame.to_geo_dict
  • GeoDataFrame.__geo_interface__
  • GeoSeries.__geo_interface__

For anyone interested, here's the epic for initial Geopandas Support, which links to all of the rest of the PRs: #2001

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions