But, Bill... shouldn't we keep the crappy looking text for backwards compatibility...?<div><br></div><div>:-)</div><div><br></div><div><br></div><div>(Just teasing, now, don't flame me too harshly.....)</div><div><br>
<br><div class="gmail_quote">On Sun, Oct 17, 2010 at 11:28 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;">
Thanks. You are correct I did not mean to send it to the arb.<br>
<br>
And, I guess I forgot I had mentioned this before.<br>
<br>
I'll look forward to the new code.<br>
<font color="#888888"><br>
Bill<br>
</font><div><div></div><div class="h5"><br>
On Sun, Oct 17, 2010 at 6:24 PM, Marcus D. Hanwell<br>
<<a href="mailto:marcus.hanwell@kitware.com">marcus.hanwell@kitware.com</a>> wrote:<br>
> On Sun, Oct 17, 2010 at 3:24 PM, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>><br>
> wrote:<br>
>><br>
>> Sorry, I did mean to send this to <a href="mailto:arb@vtk.org">arb@vtk.org</a>. gmail was overly agressive.<br>
><br>
> Do you mean you didn't mean to send this to arb@...? I was wondering why it<br>
> was an ARB issue.<br>
>><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>
><br>
> I remember you emailing me about this, and I gave you an overview of what I<br>
> was doing at the time. Due to my travel schedule, and some other events, my<br>
> topic branch to address this has been a little delayed. I had most of it<br>
> staged last week, but was working on fixing a few more regressions before<br>
> merging it. I think I got most of it finished yesterday.<br>
>><br>
>> I looked at the VTKData git repository. On 8/23, many Charts baselines<br>
>> were changed.<br>
>><br>
>> Please compare this baseline:<br>
>><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>
>><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>
><br>
> The problem is that it was only correct for short strings, for longer ones<br>
> they would still fall off the edge. The newer code uses font metrics to<br>
> determine the chart layout and works for all font sizes and string lengths.<br>
> The issue was that only the Qt label render strategy was respecting the<br>
> rotation, but the freetype strategy was returning the wrong metrics and not<br>
> rotating the font.<br>
>><br>
>> This occurs consistently on all of the baselines that hav ea Y-Axis labe;.<br>
>><br>
> It looks a lot better when using the Qt label render strategy, and as I said<br>
> I was working on getting this into shape with the freetype rendering (which<br>
> is default for most VTK applications. The StringToImage topic branch<br>
> contains a lot of these changes, and there are a few more I will push<br>
> shortly and integrate tomorrow.<br>
> I didn't want to cause all of the baseline images to need multiple updates,<br>
> but once I merge the changes in tomorrow many tests will require updates. I<br>
> think the new baselines look far better, and I am hoping to land some more<br>
> string rendering improvements in master over the next few weeks. I did not<br>
> intend to leave the axis rendering like this for a release.<br>
> I hope that clears things up, these changes would have likely made it in<br>
> last week if it weren't for some other large changes also being integrated<br>
> along with some other timing issues.<br>
> Marcus<br>
</div></div><div><div></div><div class="h5">_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br></div>