<div dir="ltr">These (and others) are now merged to VTK-5-2 :<div><br></div><div><div>    /cvsroot/VTK/VTK/Common/vtkPythonUtil.cxx,v  <--  Common/vtkPythonUtil.cxx</div><div>    new revision: <a href="http://1.83.2.2">1.83.2.2</a>; previous revision: <a href="http://1.83.2.1">1.83.2.1</a></div>
<div>    /cvsroot/VTK/VTK/Common/Testing/Python/CMakeLists.txt,v  <--  Common/Testing/Python/CMakeLists.txt</div><div>    new revision: <a href="http://1.4.2.1">1.4.2.1</a>; previous revision: 1.4</div><div>    /cvsroot/VTK/VTK/Common/Testing/Python/TestExecuteMethodFinalizeCrash.py,v  <--  Common/Testing/Python/TestExecuteMethodFinalizeCrash.py</div>
<div>    new revision: <a href="http://1.1.2.2">1.1.2.2</a>; previous revision: <a href="http://1.1.2.1">1.1.2.1</a></div><div>    /cvsroot/VTK/VTK/Common/Testing/Python/TestTerminationCrash.py,v  <--  Common/Testing/Python/TestTerminationCrash.py</div>
<div>    new revision: <a href="http://1.2.2.2">1.2.2.2</a>; previous revision: <a href="http://1.2.2.1">1.2.2.1</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkAffineRepresentation2D.cxx,v  <--  Widgets/vtkAffineRepresentation2D.cxx</div>
<div>    new revision: <a href="http://1.6.32.2">1.6.32.2</a>; previous revision: <a href="http://1.6.32.1">1.6.32.1</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkBalloonRepresentation.cxx,v  <--  Widgets/vtkBalloonRepresentation.cxx</div>
<div>    new revision: <a href="http://1.8.32.2">1.8.32.2</a>; previous revision: <a href="http://1.8.32.1">1.8.32.1</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkCenteredSliderRepresentation.h,v  <--  Widgets/vtkCenteredSliderRepresentation.h</div>
<div>    new revision: <a href="http://1.3.2.3">1.3.2.3</a>; previous revision: <a href="http://1.3.2.2">1.3.2.2</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkContinuousValueWidgetRepresentation.h,v  <--  Widgets/vtkContinuousValueWidgetRepresentation.h</div>
<div>    new revision: <a href="http://1.2.2.3">1.2.2.3</a>; previous revision: <a href="http://1.2.2.2">1.2.2.2</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkImplicitPlaneRepresentation.cxx,v  <--  Widgets/vtkImplicitPlaneRepresentation.cxx</div>
<div>    new revision: <a href="http://1.9.2.2">1.9.2.2</a>; previous revision: <a href="http://1.9.2.1">1.9.2.1</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkScalarBarRepresentation.h,v  <--  Widgets/vtkScalarBarRepresentation.h</div>
<div>    new revision: <a href="http://1.1.2.1">1.1.2.1</a>; previous revision: 1.1</div><div>    /cvsroot/VTK/VTK/Widgets/vtkScalarBarRepresentation.cxx,v  <--  Widgets/vtkScalarBarRepresentation.cxx</div><div>    new revision: <a href="http://1.1.2.1">1.1.2.1</a>; previous revision: 1.1</div>
<div>    /cvsroot/VTK/VTK/Widgets/vtkSliderRepresentation2D.h,v  <--  Widgets/vtkSliderRepresentation2D.h</div><div>    new revision: <a href="http://1.2.50.1">1.2.50.1</a>; previous revision: 1.2</div><div>    /cvsroot/VTK/VTK/Widgets/vtkSliderRepresentation2D.cxx,v  <--  Widgets/vtkSliderRepresentation2D.cxx</div>
<div>    new revision: <a href="http://1.6.48.2">1.6.48.2</a>; previous revision: <a href="http://1.6.48.1">1.6.48.1</a></div><div>    /cvsroot/VTK/VTK/Widgets/vtkSliderRepresentation3D.h,v  <--  Widgets/vtkSliderRepresentation3D.h</div>
<div>    new revision: <a href="http://1.7.14.1">1.7.14.1</a>; previous revision: 1.7</div><div>    /cvsroot/VTK/VTK/Wrapping/Python/vtk/util/numpy_support.py,v  <--  Wrapping/Python/vtk/util/numpy_support.py</div><div>
    new revision: <a href="http://1.2.2.1">1.2.2.1</a>; previous revision: 1.2</div><div><br></div><div><br></div><div>Thanks for your patience,</div><div>David</div><div><br></div><br><div class="gmail_quote">On Wed, Aug 6, 2008 at 4:25 PM, Prabhu Ramachandran <span dir="ltr"><<a href="mailto:prabhu@aero.iitb.ac.in">prabhu@aero.iitb.ac.in</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Chris Kruszynski wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="Ih2E3d">
On 05-08-08 19:49, Prabhu Ramachandran wrote:<br></div><div class="Ih2E3d">
I have dug deeper into why the crashes occur while they shouldn't, and I have uncovered the cause: the vtkPythonHash is deleted from vtkPythonHashDelete(), which is a Py_AtExit callback. The documentation for Py_AtExit is very clear:<br>

</div></blockquote>
<br>
Excellent!  I noticed the Py_AtExit but did not realize the problem was there.  I've checked in your patch with the test case as well.  Many thanks!<br>
<br>
/cvsroot/VTK/VTK/Common/vtkPythonUtil.cxx,v  <--  Common/vtkPythonUtil.cxx<br>
new revision: 1.90; previous revision: 1.89<br>
/cvsroot/VTK/VTK/Common/Testing/Python/CMakeLists.txt,v  <-- Common/Testing/Python/CMakeLists.txt<br>
new revision: 1.6; previous revision: 1.5<br>
/cvsroot/VTK/VTK/Common/Testing/Python/TestExecuteMethodFinalizeCrash.py,v  <--  Common/Testing/Python/TestExecuteMethodFinalizeCrash.py<br>
initial revision: 1.1<br>
<br>
<br>
I hope there is time to include these in the 5.2 release.  Thanks!<br>
<br>
<br>
<br>
cheers,<br><font color="#888888">
prabhu<br>
</font></blockquote></div><br></div></div>