<div dir="ltr"><div><div><div><div><div>It should be applied only when the test IsValid fail.<br></div>This should limit the slowness.<br></div>Also pay attention that something the result of a MakeValid on an Invalid poligon could be a MultiPolygon or also a Collection.<br>
</div>So to use it in an automatic manner need to apply also a ST_Multi(CollectionExtract(ST_MakeValid(geometry),3)).<br></div>To be sure that it is always a geometry of the same kind.<br><br></div>A.<br><br><div><div><div>
<br></div></div></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014/1/16 Larry Shaffer <span dir="ltr"><<a href="mailto:larrys@dakotacarto.com" target="_blank">larrys@dakotacarto.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi Paolo,<br><div><div class="gmail_extra"><br><br><div class="gmail_quote"><div class="im">On Wed, Jan 15, 2014 at 11:35 PM, Paolo Cavallini <span dir="ltr"><<a href="mailto:cavallini@faunalia.it" target="_blank">cavallini@faunalia.it</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi all.<br>
To the best of my knowledge, our make clean command, available through<br>
the Processing subplugin:<br>
<br>
<a href="http://plugins.qgis.org/plugins/processinglwgeomprovider/" target="_blank">http://plugins.qgis.org/plugins/processinglwgeomprovider/</a><br>
<br>
is the most effective method to clean up invalid geometries.<br>
I would therefore suggest to include it in master, in order to have it<br>
available straight away for everybody.<br>
This depends on lwgeom, available at least on Debian and Windows.<br></blockquote><div><br></div></div><div>I have recently made a Homebrew formula for Mac that allows liblwgeom to be built without Postgres/PostGIS installs (a bit of a hack, actually). So, the standalone lib is now readily available on Mac OS X as well [0].<br>

</div><div><br></div><div>+1 for this to be included in QgsGeometry or similar class.<br><br>It would help solve labeling issues where the extents-clipped geometry is invalid. Would slow rendering down, of course, but would be good to have as an option for faster machines and print jobs. Note: the validity fix would be applied to the duplicates of feature geometries, which are used by PAL.<br>

</div><div><br>[0] <a href="https://github.com/Homebrew/homebrew/blob/master/Library/Formula/liblwgeom.rb" target="_blank">https://github.com/Homebrew/homebrew/blob/master/Library/Formula/liblwgeom.rb</a><span class="HOEnZb"><font color="#888888"><br>
<br></font></span></div><span class="HOEnZb"><font color="#888888"><div>Larry<br><br><br>
</div></font></span><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Opinions? Any blockers?<br>
<br>
All the best.<br>
<span><font color="#888888">--<br>
Paolo Cavallini - <a href="http://www.faunalia.eu" target="_blank">www.faunalia.eu</a><br>
QGIS & PostGIS courses: <a href="http://www.faunalia.eu/training.html" target="_blank">http://www.faunalia.eu/training.html</a><br>
_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org" target="_blank">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br>
</font></span></blockquote></div></div><br></div></div></div>
<br>_______________________________________________<br>
Qgis-developer mailing list<br>
<a href="mailto:Qgis-developer@lists.osgeo.org">Qgis-developer@lists.osgeo.org</a><br>
<a href="http://lists.osgeo.org/mailman/listinfo/qgis-developer" target="_blank">http://lists.osgeo.org/mailman/listinfo/qgis-developer</a><br></blockquote></div><br><br clear="all"><br>-- <br>-----------------<br>Andrea Peri<br>
. . . . . . . . . <br>qwerty אטלעש<br>-----------------<br>
</div>