[Paraview] Calculator performing poorly in ParaView-3-8 branch

[Dave Partyka]

Merged onto the branch. Thanks John for reporting this and trying the fix!

/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPVArrayCalculator.cxx,v  <--
 Servers/Filters/vtkPVArrayCalculator.cxx
new revision: 1.2.2.2; previous revision: 1.2.2.1
/cvsroot/ParaView3/ParaView3/VTK/Common/vtkFunctionParser.cxx,v  <--
 VTK/Common/vtkFunctionParser.cxx
new revision: 1.45.2.2; previous revision: 1.45.2.1

On Tue, Apr 13, 2010 at 5:51 PM, John Patchett <patchett2002 at gmail.com>wrote:

> Initial results show performance is repaired, on par with the March
> Head build...
> Thank You very much,
> --John.
>
> On Tue, Apr 13, 2010 at 2:20 PM, Utkarsh Ayachit
> <utkarsh.ayachit at kitware.com> wrote:
> > John,
> >
> > Attached is a patch. Can you verify that it overcomes the performance
> > issues you are seeing? If so, I'll commit it to CVS and get in 3.8
> > branch as well.
> >
> > Utkarsh
> >
> > On Tue, Apr 13, 2010 at 3:23 PM, John Patchett <patchett2002 at gmail.com>
> wrote:
> >> I've noticed the calculator has started performing substantially
> >> slower in the ParaView-3-8 branch when taking the magnitude of a
> >> vector as compared to the Head before the branch.
> >> I've attached a graph summarizing the performance degradation.  I used
> >> the Mandelbrot source and took the magnitude of the coords vector to
> >> collect data for the graph.
> >> One of my machines averages nearly a 6x slowdown and the other a 4.5x
> slowdown.
> >>
> >> The *-march is a Head build  from 3/23/2010.
> >> 3-8 is the branch as of yesterday afternoon.
> >>
> >> I (believe) the change that affected the calculator performance
> >> happened after the branch as I noticed it after I updated the branch (
> >> I wouldn't bet too much on this though)...  I don't know if the Head
> >> is affected.
> >>
> >> Thanks,
> >> --John Patchett.
> >>
> >> _______________________________________________
> >> Powered by www.kitware.com
> >>
> >> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
> >>
> >> Please keep messages on-topic and check the ParaView Wiki at:
> http://paraview.org/Wiki/ParaView
> >>
> >> Follow this link to subscribe/unsubscribe:
> >> http://www.paraview.org/mailman/listinfo/paraview
> >>
> >>
> >
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Please keep messages on-topic and check the ParaView Wiki at:
> http://paraview.org/Wiki/ParaView
>
> Follow this link to subscribe/unsubscribe:
> http://www.paraview.org/mailman/listinfo/paraview
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.paraview.org/pipermail/paraview/attachments/20100413/44a277e8/attachment.htm>