<div dir="ltr"><div>Hi Kathleen,<br><br></div>Thanks for the patch. I'll take a look at it.<br><br>Andy<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 2, 2013 at 2:06 PM, Kathleen S. Biagas <span dir="ltr"><<a href="mailto:biagas2@llnl.gov" target="_blank">biagas2@llnl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I have a patch for vtkPyramid.cxx which corrects a problem with the<br>
EvaluatePosition method.  I also have a small test that demonstrates the<br>
problem: it creates a pyramid with vertices at (1.0, 1.0, 1.0), (-1.0,<br>
1.0, 1.0), (-1.0, -1.0, 1.0), (1.0, -1.0, 1.0),(0.0, 0.0, 0.0), then<br>
calls EvaluatePosition for each of the following Points printing a<br>
message whether the point is considered inside or outside the cell:<br>
<br>
(0.0, 0.0, -0.03)<br>
(0.0, 0.0, -0.005)<br>
(1.005, 1.005, 1.0)<br>
(0.0, 0.0, 0.5)<br>
<br>
<br>
I couldn't determine how this small test case would fit within your<br>
testing framework, or I would push it to gerrit with the vtkPyramid.cxx<br>
patch.  Shall I commit the patch anyway?<br>
<br>
Kathleen<br>
<br>
<br>
Here's various results of the tests with different vtk versions:<br>
(all run on rhel-6, using gcc-4.4)<br>
<br>
Results of test when run with VTK-5.8:<br>
<br>
Point 0 0 -0.03 is outside the pyramid.<br>
Point 0 0 -0.005 is outside the pyramid.<br>
Point 1.005 1.005 1 is outside the pyramid.<br>
Point 0 0 0.5 is inside the pyramid.<br>
<br>
Results of test when run with snapshot of master from this morning<br>
(04-02-12):<br>
<br>
Point 0 0 -0.03 is inside the pyramid.<br>
Point 0 0 -0.005 is inside the pyramid.<br>
Point 1.005 1.005 1 is outside the pyramid.<br>
Point 0 0 0.5 is inside the pyramid.<br>
<br>
Results of test with proposed change to vtkPyramid.cxx:<br>
<br>
Point 0 0 -0.03 is outside the pyramid.<br>
Point 0 0 -0.005 is outside the pyramid.<br>
Point 1.005 1.005 1 is outside the pyramid.<br>
Point 0 0 0.5 is inside the pyramid.<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>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://www.vtk.org/mailman/listinfo/vtk-developers" target="_blank">http://www.vtk.org/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br></div>