Hi Dave,<div><br></div><div>I pushed my updated branch and now all the tests should pass (continuous looks ok after my second push). Apart from fixing the issue I added a new test as well (there were none before for that class). </div>
<div><br></div><div>With the new branch I added a feature to have a background (and in future I might put a support to have a texture for the background) for the legend as we need it for our current project. I will be keeping an eye on the dashboards tonight. </div>
<div><br></div><div>Thanks again. </div><div><br></div><div>Regards, </div><div><br></div><div><br><br><div class="gmail_quote">On Tue, Oct 19, 2010 at 1:27 PM, Aashish Chaudhary <span dir="ltr"><<a href="mailto:aashish.chaudhary@kitware.com">aashish.chaudhary@kitware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi Dave, <br><br><div class="gmail_quote"><div class="im">On Tue, Oct 19, 2010 at 1:16 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com" target="_blank">david.gobbi@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Aashish,<br>
<br>
Some of the dashboard tests are failing due to changes to line 796 of<br>
Hybrid/vtkLegendBoxActor.cxx:<br>
<br>
    posX = p1[0] + this->Padding +<br>
                 0.25*symbolSize*(p2[0] - p1[0] - 2.0*this->Padding);<br>
<br>
If the "0.25" is changed back to "0.5", then the tests pass.  I don't<br>
know if that is the correct fix, though.<br>
<br></blockquote></div><div>I will have a look. All the tests that I ran at the time passed. There could be a logical issue here.  </div><div class="im"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


There also seems to be an error at line 917:<br>
<br>
      if( this->Icon[i])<br>
        {<br>
        renderedSomething += this->IconActor[i]->RenderOpaqueGeometry(viewport);<br>
        }<br>
<br>
I'm guessing that this should be "if( this->IconImage[i] )" instead of<br>
"this->Icon[i]" </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><font color="#888888">
  David<br>
</font></blockquote></div></div><div><br></div>That sounds about right. I will fix this too. The idea is you can have icons, symbols and or a background plane. In fact I am still working on it so its not done (background plane) yet. <div>

<br></div><div>Thanks, </div><div><br></div><font color="#888888"><div><br clear="all"><br>-- <br>| Aashish Chaudhary <br>| R&D Engineer         <br>| Kitware Inc.            <br>| <a href="http://www.kitware.com" target="_blank">www.kitware.com</a>    <br>


</div>
</font></blockquote></div><br><br clear="all"><br>-- <br>| Aashish Chaudhary <br>| R&D Engineer         <br>| Kitware Inc.            <br>| <a href="http://www.kitware.com">www.kitware.com</a>    <br>
</div>