<div dir="ltr">Hi,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 26, 2014 at 12:44 PM, Larry Shaffer <span dir="ltr"><<a href="mailto:larrys@dakotacarto.com" target="_blank">larrys@dakotacarto.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"><div dir="ltr">Hi Martin and Mathieu,<br><div><div class="gmail_extra"><br><div class="gmail_quote">
On Wed, Feb 26, 2014 at 6:41 AM, Mathieu Pellerin <span dir="ltr"><<a href="mailto:nirvn.asia@gmail.com" target="_blank">nirvn.asia@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"><div dir="ltr"><div><div>Happy to report that following Martin's last commit (<a href="https://github.com/qgis/QGIS/commit/daf1e0b6881cdb77df6f6a9dc988ad92e0f9019d" target="_blank">https://github.com/qgis/QGIS/commit/daf1e0b6881cdb77df6f6a9dc988ad92e0f9019d</a>), I cannot see any issues with vector/raster layer rendering, as well as labelling. All renders like 2.2, except the user experience is 10 times better :)<br>


<br></div>Larry, I'm pretty sure your labeling tests will mostly turn green now.<br></div></div></blockquote><div><br></div><div class="gmail_extra">Yes and no. :-)<br><br>The server and composer tests, and comparisons with canvas output ,seem to pass OK now, but not so for the canvas [0]. Now, I don't know if it is the unit test setup that needs updated to work with new QgsMapSettings or something in PAL, so I have to do some investigation first.<br>
</div></div></div></div></div></blockquote><div><br></div><div>Turns out I just had to rebuild the TestQgsPalLabeling base test class to utilize QgsMapSettings (instead of deprecated QgsMapRenderer methods) [0] and ensure to set the correct map units for the CRS. Canvas tests now match 2.2 control images within a 0 - 6 pixel tolerance [1].<br>
<br>Close enough for me!<br><br>[0] <a href="https://github.com/qgis/QGIS/commit/b6d943e">https://github.com/qgis/QGIS/commit/b6d943e</a><br>[1] <a href="http://drive.dakotacarto.com/qgis/PALTestReport_2014-02-26_20-56-33.pdf">http://drive.dakotacarto.com/qgis/PALTestReport_2014-02-26_20-56-33.pdf</a><br>
<br></div><div>Regards,<br><br></div><div>Larry<br></div><div><br> </div></div><br></div></div></div>