<div dir="ltr">Hi,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jul 24, 2013 at 11:04 AM, aperi2007 <span dir="ltr"><<a href="mailto:aperi2007@gmail.com" target="_blank">aperi2007@gmail.com</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">On 24/07/2013 17:55, Larry Shaffer wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I think QGIS should offer the option to on-the-fly try to fix those geometries. This was discussed previously [0]<br>
</blockquote>
<br>
The MakeValid is not a fast process.<br>
Try to apply it on-the-fly will slow again the labelling flow.<br></blockquote><div><br></div><div>Yes, but only for those features that are tested to be invalid (already done). Also, as I noted, this should be optional, and probably disabled by default.<br>
</div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Another problem is tht it could change little , but not less important, part of a geometry.<br>
so if the user will decide to edit it .<br>
Instead of editing the original geometry it will edit a changed geometry.<br>
And save in the shapefile (o r other container) another more changed geometry.<br></blockquote><div><br></div><div>The labeling engine sends to the backend PAL library *copies* of the feature geometries. So, the original geometry is never edited, i.e. the copy would be what is run through ST_MakeValid. Once the labels are created, those copied geometries are dumped and never rendered on the canvas.<br>
<br></div><div>There should probably be some logging somewhere, if the geometry is invalid, but usually the copied geometry has been clipped to the extent rectangle. In other words, invalid geometries of the copies at that point have nothing to do with the original features, which may be valid. So, logging invalids at that point doesn't make sense.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Don't forgot that the original ST_MakeValid of postgis (and also spatialite) will return also the<br>
removed parts of original geometry. (point , arc or other it was)<br></blockquote><div><br></div><div>This could be an issue, where a multi-geometry is return for a single input. It seems this could be overcome with some checks, and eventually fallback to rejecting the geometry, if necessary.<br>
</div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
To not lost of view the original geometry.<br></blockquote><div><br></div><div>Sorry. I don't understand your meaning here.<br><br><br></div><div>Regards,<br><br></div><div>Larry<br></div><div><br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

______________________________<u></u>_________________<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/<u></u>mailman/listinfo/qgis-<u></u>developer</a><br>
</blockquote></div><br></div></div></div>