[vtk-developers] Regression of vtkLegendBoxActor

Aashish Chaudhary aashish.chaudhary at kitware.com
Tue Oct 19 13:27:36 EDT 2010


Hi Dave,

On Tue, Oct 19, 2010 at 1:16 PM, David Gobbi <david.gobbi at gmail.com> wrote:

> Hi Aashish,
>
> Some of the dashboard tests are failing due to changes to line 796 of
> Hybrid/vtkLegendBoxActor.cxx:
>
>    posX = p1[0] + this->Padding +
>                 0.25*symbolSize*(p2[0] - p1[0] - 2.0*this->Padding);
>
> If the "0.25" is changed back to "0.5", then the tests pass.  I don't
> know if that is the correct fix, though.
>
> I will have a look. All the tests that I ran at the time passed. There
could be a logical issue here.


> There also seems to be an error at line 917:
>
>      if( this->Icon[i])
>        {
>        renderedSomething +=
> this->IconActor[i]->RenderOpaqueGeometry(viewport);
>        }
>
> I'm guessing that this should be "if( this->IconImage[i] )" instead of
> "this->Icon[i]"

 David
>

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.

Thanks,



-- 
| Aashish Chaudhary
| R&D Engineer
| Kitware Inc.
| www.kitware.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20101019/20e332f6/attachment.html>


More information about the vtk-developers mailing list