<div dir="ltr"><div>I noticed that the OverlayNG coverage union returns an empty polygon if the inputs do not form a coverage, whereas the older algorithm produces an error. I think it would be a good idea to apply an area post-check to the NG version so that we retain the error. I have also noticed cases (TIGER county boundaries) where the NG algorithm takes about 2x as long as the old algorithm but performs better than the old algorithm if the inputs are pre-sorted (as they are in the old algorithm). Maybe the same sorting should be added to the NG version.</div><div><br></div><div>Otherwise I have no reservations about switching.</div><div><br></div><div>Dan<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 18, 2023 at 7:05 PM Martin Davis <<a href="mailto:mtnclimb@gmail.com">mtnclimb@gmail.com</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"><div dir="ltr">geos/coverage/CoverageUnion in fact delegates to geos/operation/overlayng/CoverageUnion.  It's there to provide an API which is the same as the other coverage operations.<br><div><br></div><div>overlayng/CoverageUnion seems to be quite a bit faster than geos/operation/union/CoverageUnion, by my testing.  So it has my vote.(They are both exposed in geosop, so independent verification is welcome.)<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Apr 18, 2023 at 3:56 PM Paul Ramsey <<a href="mailto:pramsey@cleverelephant.ca" target="_blank">pramsey@cleverelephant.ca</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">We have quite a few of them? Which one should be keep and bind into CAPI?<br>
<br>
geos/operation/union/CoverageUnion.h<br>
geos/operation/overlayng/CoverageUnion.h<br>
geos/coverage/CoverageUnion.h<br>
_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a><br>
</blockquote></div>
_______________________________________________<br>
geos-devel mailing list<br>
<a href="mailto:geos-devel@lists.osgeo.org" target="_blank">geos-devel@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/geos-devel" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/geos-devel</a><br>
</blockquote></div>