<div dir="ltr">Hi Sandro,<div><br></div><div>What is the actual problem you're trying to solve?</div><div>GEOSMakeValid builds a valid geom out of its input.</div><div>LWMakeValid builds a valid geom out of its input.</div><div>They differ in the details, but that wasn't a problem for years, and there isn't a test to pin either's minor behavior differences, only useful stuff.</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Dec 11, 2020 at 12:19 AM Sandro Santilli <<a href="mailto:strk@kbt.io">strk@kbt.io</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Dec 10, 2020 at 12:48:08PM -0800, Paul Ramsey wrote:<br>
> > On Dec 10, 2020, at 12:41 PM, Sandro Santilli <<a href="mailto:strk@kbt.io" target="_blank">strk@kbt.io</a>> wrote:<br>
> > <br>
> > Is there a good reason to delegate ST_MakeValid<br>
> > to GEOSMakeValid, when available ?<br>
> <br>
> So that we eventually end up with all the behaviour in the same place?<br>
<br>
Eventually would be when we drop support for older GEOS. It's not yet<br>
the time, is it ?<br>
<br>
> > I'm asking because while changing ST_MakeValid to<br>
> > handle NaN I find myself having to do the same changes<br>
> > both in PostGIS and GEOS for people running older GEOS.<br>
> <br>
> If you don't then we instead end up at a place where swapping implementations results in sudden behaviour change, which is Also Bad.<br>
<br>
True. I'll check how does the GEOS side behaves, chances are<br>
it already does NaN stripping (it doens't help that the _unit_<br>
test for GEOS makevalid was left out of autotools build scripts).<br>
<br>
> > How about keeping it all in PostGIS always until we drop<br>
> > support for older GEOS versions ?<br>
> <br>
> Minimize the long term Badness by doing the Extra Work this problem requires.<br>
<br>
What do you mean ? Drop support for old GEOS now ?<br>
<br>
--strk;<br>
_______________________________________________<br>
postgis-devel mailing list<br>
<a href="mailto:postgis-devel@lists.osgeo.org" target="_blank">postgis-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/postgis-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/postgis-devel</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">Darafei "Komяpa" Praliaskouski<br>OSM BY Team - <a href="http://openstreetmap.by/" target="_blank">http://openstreetmap.by/</a><br></div></div>