joonaspessi opened a new pull request, #243: URL: https://github.com/apache/sedona-db/pull/243
Implements `ST_SimplifyPreserveTopology(geometry, tolerance)` function using geos. **Notice about ST_Simplify** Actually started the implementation of `ST_Simplify` but noticed that the PostGIS has different behavior for collapsed geometries and collection types in comparison to [GEOS implementation](https://libgeos.org/doxygen/classgeos_1_1simplify_1_1DouglasPeuckerSimplifier.html). Achieving full PostGIS compatibility (including all the params) would probably require custom Rust implementation. - Both PostGIS and Sedona-DB uses now GEOS for ST_SimplifyPreserveTopology - https://github.com/postgis/postgis/blob/master/postgis/lwgeom_geos.c#L935 - Entry point - https://github.com/postgis/postgis/blob/master/postgis/lwgeom_geos.c#L966 - GEOS function call - PostGIS uses custom implementation for ST_Simplify - https://github.com/postgis/postgis/blob/master/postgis/lwgeom_functions_analytic.c#L58 - Entry point - https://github.com/postgis/postgis/blob/master/liblwgeom/lwgeom.c#L1851 - Core custom implementation This might be actually nice function to be ported into Rust. -- 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]
