MantisBT - ParaView
View Issue Details
0006553ParaView(No Category)public2008-03-06 15:082009-05-13 13:59
Berk Geveci 
Clinton Stimpson 
highminorhave not tried
closedfixed 
 
3.43.4 
0006553: Do not create testing thread until it is needed
The testing thread (the one that runs python) should not be created until it is needed. It is causing issues with the interps on the main threadh
No tags attached.
related to 0006554closed Utkarsh Ayachit Revert vtkPVPythonInterp to before thread support 
Issue History
2008-03-06 15:08Berk GeveciNew Issue
2008-03-06 15:08Berk GeveciStatusbacklog => tabled
2008-03-06 15:08Berk GeveciAssigned To => Clinton Stimpson
2008-03-06 15:09Berk GeveciTarget Version => Python support
2008-03-06 15:14Clinton StimpsonNote Added: 0010729
2008-03-06 15:15Berk GeveciRelationship addedrelated to 0006554
2008-03-06 15:56Berk GeveciNote Added: 0010730
2008-03-06 16:46Utkarsh AyachitNote Added: 0010732
2008-03-06 17:00Clinton StimpsonStatustabled => @80@
2008-03-06 17:00Clinton StimpsonResolutionopen => fixed
2008-03-06 17:00Clinton StimpsonNote Added: 0010734
2008-03-18 17:34Alan ScottStatus@80@ => closed
2008-03-18 17:34Alan ScottNote Added: 0010846
2009-05-13 13:58Utkarsh AyachitTarget VersionPython support => 3.4
2009-05-13 13:59Utkarsh AyachitFixed in Version => 3.4
2011-06-16 13:10Zack GalbreathCategory => (No Category)

Notes
(0010729)
Clinton Stimpson   
2008-03-06 15:14   
Do you have a test case to show the problem?

The thread doesn't actually start until needed. However, there are some python calls before then to initialize things that can be postponed until just before the thread is started.
(0010730)
Berk Geveci   
2008-03-06 15:56   
Reminder sent to: Utkarsh Ayachit

Utkarsh,

Can you comment on Clint's note?
(0010732)
Utkarsh Ayachit   
2008-03-06 16:46   
If pqPythonEventSource does not initialize python until the python testing thread starts, that would be great. With the assumption that python testing will never be used if the python shell or python programmable filter are used, we can totally get rid of the multi-threading support.
(0010734)
Clinton Stimpson   
2008-03-06 17:00   
Try this out. I didn't know how to reproduce these problems, so ... hope it works for you.

/cvsroot/ParaView3/ParaView3/Qt/Testing/pqPythonEventSource.cxx,v <-- pqPythonEventSource.cxx
new revision: 1.25; previous revision: 1.24
(0010846)
Alan Scott   
2008-03-18 17:34   
As per Berk.