MantisBT - ParaView
View Issue Details
0009700ParaViewBugpublic2009-10-12 18:342010-03-01 21:34
Alan Scott 
Utkarsh Ayachit 
immediateminoralways
closedfixed 
 
3.83.8 
0009700: Python trace error - adding clip
ParaView 3.6.2, XP, local server.
* Open disk_out_ref.exo, all variables on, apply.
* Color by Temp.
* Python shell. Trace. Start Trace.
* Clip, Accept, turn Show Plane off.
* Stop Trace. Save Trace.
* Delete the clip filter.
Now, run the saved trace. It will give the following bugs:

<<The clip widget is still visible, and the Show Plane button is still checked.>>
<<The colors in the middle of the object, colored by Temp, have turned to solid blue. Open the Color Editor. The Color Scale has changed to something crazy.>>
No tags attached.
Issue History
2009-10-12 18:34Alan ScottNew Issue
2010-02-01 14:26Utkarsh AyachitPrioritynormal => immediate
2010-02-02 14:50Utkarsh AyachitNote Added: 0019440
2010-02-02 14:52Utkarsh AyachitNote Deleted: 0019440
2010-02-02 14:56Utkarsh AyachitNote Added: 0019441
2010-02-05 13:52Utkarsh AyachitStatusbacklog => tabled
2010-02-05 13:52Utkarsh AyachitAssigned To => Utkarsh Ayachit
2010-02-05 16:03Utkarsh AyachitNote Added: 0019463
2010-02-05 16:03Utkarsh AyachitStatustabled => @80@
2010-02-05 16:03Utkarsh AyachitFixed in Version => 3.8
2010-02-05 16:03Utkarsh AyachitResolutionopen => fixed
2010-02-08 23:12Alan ScottNote Added: 0019490
2010-02-08 23:12Alan ScottStatus@80@ => @20@
2010-02-08 23:12Alan ScottResolutionfixed => reopened
2010-02-09 09:52Utkarsh AyachitNote Added: 0019493
2010-02-09 09:52Utkarsh AyachitStatus@20@ => @80@
2010-02-09 09:52Utkarsh AyachitResolutionreopened => fixed
2010-03-01 21:34Alan ScottNote Added: 0019677
2010-03-01 21:34Alan ScottStatus@80@ => closed

Notes
(0019441)
Utkarsh Ayachit   
2010-02-02 14:56   
The Clip filter by default uses the same LUT as used by the reader for coloring Temp when created through GUI. The trace records the setting up of array to color with, but does not include setting of the LUT (since there's no SM level LUT manager that it can go to. Hence, when the trace is played back, the mapper simply uses a default LUT which is bogus.
(0019463)
Utkarsh Ayachit   
2010-02-05 16:03   
The trace was not handling it correctly when recording
setting of an already existing LookupTable. Added a new GetLookupTableForArray
function to simple that uses the array name to locate/create lookup tables
similar to the GUI.

/cvsroot/ParaView3/ParaView3/Utilities/VTKPythonWrapping/paraview/simple.py,v <-- Utilities/VTKPythonWrapping/paraview/simple.py
new revision: 1.24; previous revision: 1.23
/cvsroot/ParaView3/ParaView3/Utilities/VTKPythonWrapping/paraview/smtrace.py,v <-- Utilities/VTKPythonWrapping/paraview/smtrace.py
new revision: 1.13; previous revision: 1.12

As far as the status of the "Show Plane" check is concerned, it's not possible to control Show Plane through python at all since all the 3D widgets are GUI components and are not accessible through python or batch and the like. Hence the state does not note the change in the status of "Show Plane".
(0019490)
Alan Scott   
2010-02-08 23:12   
Nope, not quite. Now, when I do the steps listed, the whole toolbar for the representation and color selection go gray, and I cannot get them back. Furthermore, it shows that the color is grey. Last, the show plane tool is back on.

If you select disk_out_ref, then go back to the clip, we at least have the toolbar back, but a user shouldn't have to do that.
(0019493)
Utkarsh Ayachit   
2010-02-09 09:52   
When representation was created through python, the GUI would not realize
that the representation has been created since the pqActiveObjects instance
didn't fire the active representation changed signal. Fixed that.

/cvsroot/ParaView3/ParaView3/Qt/Components/pqActiveObjects.cxx,v <-- Qt/Components/pqActiveObjects.cxx
new revision: 1.3; previous revision: 1.2
(0019677)
Alan Scott   
2010-03-01 21:34   
Handling color is now correct.

I will accept the explanation with regards to the Show Plane tool (grumble).

Tested local server, trunk, linux.