<div dir="ltr"><div><div><div><div><div>Hi all, <br><br></div>I updated my PR.<br></div>I made some quick tests, and it seems this PR can improve speed for PostGIS layers ( with PostGIS 2.2 )<br></div><div>More details in the last comment [1]<br></div><br>Please help to test it with a variety of data types and complexity.<br><br>[1] <a href="https://github.com/qgis/QGIS/pull/2410#issuecomment-160138487">https://github.com/qgis/QGIS/pull/2410#issuecomment-160138487</a><br><br></div>Regards<br></div>Michaël aka "kimaidou"<br></div><div class="gmail_extra"><br><div class="gmail_quote">2015-10-30 11:50 GMT+01:00 kimaidou <span dir="ltr"><<a href="mailto:kimaidou@gmail.com" target="_blank">kimaidou@gmail.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"><div><div><div>Hi all<br><br></div>Following the blog post written by Paul Ramsey [1], I decided to propose using PostGIS 2.2 ST_RemoveRepeatedPoints function instead of ST_SnapToGrid to do the server side simplification<br><br></div><div>We could also investigate other new features, such as the 3rd parameter for ST_Simplify, but we use St_SimplifyPreserveTopology in QGIS, and I do not think this function has got this new parameter too.<br></div><div><br></div>I made a pull request : see [2]<br></div>Please help me to test and review it <br><div><div><div><br>[1] <a href="http://blog.cartodb.com/smaller-faster/" target="_blank">http://blog.cartodb.com/smaller-faster/</a><br>[2] <a href="https://github.com/qgis/QGIS/pull/2410" target="_blank">https://github.com/qgis/QGIS/pull/2410</a><br></div></div></div></div>
</blockquote></div><br></div>