Merged onto the branch. Thanks John for reporting this and trying the fix!<div><br></div><div><div>/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPVArrayCalculator.cxx,v <-- Servers/Filters/vtkPVArrayCalculator.cxx</div>
<div>new revision: 1.2.2.2; previous revision: 1.2.2.1</div><div>/cvsroot/ParaView3/ParaView3/VTK/Common/vtkFunctionParser.cxx,v <-- VTK/Common/vtkFunctionParser.cxx</div><div>new revision: 1.45.2.2; previous revision: 1.45.2.1</div>
<br><div class="gmail_quote">On Tue, Apr 13, 2010 at 5:51 PM, John Patchett <span dir="ltr"><<a href="mailto:patchett2002@gmail.com">patchett2002@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Initial results show performance is repaired, on par with the March<br>
Head build...<br>
Thank You very much,<br>
--John.<br>
<br>
On Tue, Apr 13, 2010 at 2:20 PM, Utkarsh Ayachit<br>
<div class="im"><<a href="mailto:utkarsh.ayachit@kitware.com">utkarsh.ayachit@kitware.com</a>> wrote:<br>
</div><div><div></div><div class="h5">> John,<br>
><br>
> Attached is a patch. Can you verify that it overcomes the performance<br>
> issues you are seeing? If so, I'll commit it to CVS and get in 3.8<br>
> branch as well.<br>
><br>
> Utkarsh<br>
><br>
> On Tue, Apr 13, 2010 at 3:23 PM, John Patchett <<a href="mailto:patchett2002@gmail.com">patchett2002@gmail.com</a>> wrote:<br>
>> I've noticed the calculator has started performing substantially<br>
>> slower in the ParaView-3-8 branch when taking the magnitude of a<br>
>> vector as compared to the Head before the branch.<br>
>> I've attached a graph summarizing the performance degradation. I used<br>
>> the Mandelbrot source and took the magnitude of the coords vector to<br>
>> collect data for the graph.<br>
>> One of my machines averages nearly a 6x slowdown and the other a 4.5x slowdown.<br>
>><br>
>> The *-march is a Head build from 3/23/2010.<br>
>> 3-8 is the branch as of yesterday afternoon.<br>
>><br>
>> I (believe) the change that affected the calculator performance<br>
>> happened after the branch as I noticed it after I updated the branch (<br>
>> I wouldn't bet too much on this though)... I don't know if the Head<br>
>> is affected.<br>
>><br>
>> Thanks,<br>
>> --John Patchett.<br>
>><br>
>> _______________________________________________<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>
>> Please keep messages on-topic and check the ParaView Wiki at: <a href="http://paraview.org/Wiki/ParaView" target="_blank">http://paraview.org/Wiki/ParaView</a><br>
>><br>
>> Follow this link to subscribe/unsubscribe:<br>
>> <a href="http://www.paraview.org/mailman/listinfo/paraview" target="_blank">http://www.paraview.org/mailman/listinfo/paraview</a><br>
>><br>
>><br>
><br>
_______________________________________________<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>
Please keep messages on-topic and check the ParaView Wiki at: <a href="http://paraview.org/Wiki/ParaView" target="_blank">http://paraview.org/Wiki/ParaView</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.paraview.org/mailman/listinfo/paraview" target="_blank">http://www.paraview.org/mailman/listinfo/paraview</a><br>
</div></div></blockquote></div><br></div>