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 <[email protected]>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 > <[email protected]> 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 <[email protected]> > 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 >
_______________________________________________ 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
