paleolimbot opened a new issue, #244: URL: https://github.com/apache/sedona-db/issues/244
@joonaspessi kindly did some research on the PostGIS implementation of ST_Simplify(), which is not implemented as a thin wrapper around GEOS: 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 I wonder if the `geo` implementation of Simplify (if it exists) already ticks some of these boxes...since it is a more involved function to implement, it is probably worth going straight for a custom or geo-generic-alg based approach. -- 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]
