<div class="gmail_quote">On Sun, Oct 17, 2010 at 3:24 PM, Bill Lorensen <span dir="ltr"><<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Sorry, I did mean to send this to <a href="mailto:arb@vtk.org">arb@vtk.org</a>. gmail was overly agressive.<br></blockquote><div><br></div><div>Do you mean you didn't mean to send this to arb@...? I was wondering why it was an ARB issue.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="h5"><br>
Marcus,<br>
<br>
I noticed a while back on the wiki examples that the y axis label does<br>
not fit on the window for vtk charts. I'm not sure when this<br>
regression occurred since we don't update VTK every night.<br></div></div></blockquote><div><br></div><div>I remember you emailing me about this, and I gave you an overview of what I was doing at the time. Due to my travel schedule, and some other events, my topic branch to address this has been a little delayed. I had most of it staged last week, but was working on fixing a few more regressions before merging it. I think I got most of it finished yesterday.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="h5">
<br>
I looked at the VTKData git repository. On 8/23, many Charts baselines<br>
were changed.<br>
<br>
Please compare this baseline:<br>
<a href="http://vtk.org/gitweb?p=VTKData.git;a=blob;f=Baseline/Charts/TestScatterPlot.png;hb=e293ebc8db06c21a574f42911b27b9b8c92c6a95" target="_blank">http://vtk.org/gitweb?p=VTKData.git;a=blob;f=Baseline/Charts/TestScatterPlot.png;hb=e293ebc8db06c21a574f42911b27b9b8c92c6a95</a><br>

with this baseline<br>
<a href="http://vtk.org/gitweb?p=VTKData.git;a=blob;f=Baseline/Charts/TestScatterPlot.png;hb=fcdb83896ab1de8ec8efcb10bf5a079f2c7219f7" target="_blank">http://vtk.org/gitweb?p=VTKData.git;a=blob;f=Baseline/Charts/TestScatterPlot.png;hb=fcdb83896ab1de8ec8efcb10bf5a079f2c7219f7</a><br>

<br>
The former looks correct to me. The Y-Axis lies completely in the<br>
window, while the later looks wrong to me. The Y-Axis label is<br>
truncated.<br></div></div></blockquote><div><br></div><div>The problem is that it was only correct for short strings, for longer ones they would still fall off the edge. The newer code uses font metrics to determine the chart layout and works for all font sizes and string lengths. The issue was that only the Qt label render strategy was respecting the rotation, but the freetype strategy was returning the wrong metrics and not rotating the font. </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div><div class="h5">
<br>
This occurs consistently on all of the baselines that hav ea Y-Axis labe;.<br><br></div></div></blockquote><div>It looks a lot better when using the Qt label render strategy, and as I said I was working on getting this into shape with the freetype rendering (which is default for most VTK applications. The StringToImage topic branch contains a lot of these changes, and there are a few more I will push shortly and integrate tomorrow.</div>
<div><br></div><div>I didn't want to cause all of the baseline images to need multiple updates, but once I merge the changes in tomorrow many tests will require updates. I think the new baselines look far better, and I am hoping to land some more string rendering improvements in master over the next few weeks. I did not intend to leave the axis rendering like this for a release.</div>
<div><br></div><div>I hope that clears things up, these changes would have likely made it in last week if it weren't for some other large changes also being integrated along with some other timing issues.</div><div><br>
</div><div>Marcus</div></div>