<div dir="ltr"><div><div><div><div><div>It seems some methods have not been thought with the 'output param must be set' mindset<br></div>i've managed to fix most methods in all vtkCell implementation (i've mainly Check linear 2D cells, as other implementation seemed to be alright )  but i've left the Contour and Clip methods out, not knowing if it should reset some parameters.<br></div><br></div></div>i've encountered some pcoords[0] = -10 ( really old code, 1994 ), is it relevant in any way ?<br><br></div><div>See my new merge request :<br><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/247">https://gitlab.kitware.com/vtk/vtk/merge_requests/247</a><br><br></div><div>please let me know what you think, i hope that i've not overdone it.<br></div><div><div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr">Mathieu Westphal<br><br>PS : subId for non composite cell should be set to -1 instead of 0 shouldn't it ?<br></div></div></div>
<br><div class="gmail_quote">On Thu, May 28, 2015 at 9:40 AM, Mathieu Westphal <span dir="ltr"><<a href="mailto:mathieu.westphal@kitware.com" target="_blank">mathieu.westphal@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><div><div><div><div><div>Here is the results of my enquiries on different cell types for evaluatePosition:<br></div>vtkEmptyCell : pcoords not set at all, nor any output parameters.<br></div>vtkVertex : pcoords[2] is not set<br></div>vtkPolyVertex : pcoords[1] and pcoords[2] are not set<br></div>vtkPolygon : pcoords[2] is not set<br></div><div>vtkQuad :  pcoords[2] is not set</div><div><br></div>All other vtkCell implementation does set all pcoords, dist2 and closestPoints<br><br></div>Do we set it to 0 ? do you know why vtkTriangle compute it this way when this is a 2D cell? (vtkTriangle.cxx:142)<span class=""><font color="#888888"><br><br></font></span><div><div class="gmail_extra"><span class=""><font color="#888888"><div><div><div dir="ltr">Mathieu Westphal<br></div></div></div></font></span><div><div class="h5">
<br><div class="gmail_quote">On Wed, May 27, 2015 at 6:10 PM, Berk Geveci <span dir="ltr"><<a href="mailto:berk.geveci@kitware.com" target="_blank">berk.geveci@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Ah! I confirm that this is a bug. Any code that generically works on 2D and 3D cells will trip over this in mixed 2D-3D cells situations as well as if the outer code does not initialize pcoords to 0 initially. So any cell type that uses a 2D parametric coordinate system should initialize the 3rd value to 0. Please submit a fix for quad and I'll review/approve it. If you have some cycles, I recommend checking the other 2D cells too...</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 27, 2015 at 11:48 AM, Joachim Pouderoux <span dir="ltr"><<a href="mailto:joachim.pouderoux@kitware.com" target="_blank">joachim.pouderoux@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Sorry, here is the correct entry link: <a href="http://www.vtk.org/Bug/view.php?id=15493" target="_blank">http://www.vtk.org/Bug/view.php?id=15493</a><br></div><div class="gmail_extra"><br clear="all"><div><div><div dir="ltr"><b>Joachim Pouderoux</b><br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><font size="1"><i>PhD, Technical Expert</i></font><br><b><font size="1"><a href="http://www.kitware.fr" target="_blank">Kitware SAS</a></font></b><br></blockquote>
</div></div></div><div><div>
<br><div class="gmail_quote">2015-05-27 17:45 GMT+02:00 Berk Geveci <span dir="ltr"><<a href="mailto:berk.geveci@kitware.com" target="_blank">berk.geveci@kitware.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hi Mathieu,<div><br></div><div>I don't see that issue in the tracker...</div><div><br></div><div>-berk</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div>On Wed, May 27, 2015 at 10:04 AM, Mathieu Westphal <span dir="ltr"><<a href="mailto:mathieu.westphal@kitware.com" target="_blank">mathieu.westphal@kitware.com</a>></span> wrote:<br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div><div dir="ltr"><div><div><div><div>I've just submitted a bug in mantis<br><a href="http://www.vtk.org/Bug/view.php?id=15491" target="_blank">http://www.vtk.org/Bug/view.php?id=15491</a><br><br></div>pcoords[2] is not overwrited when i think should.<br></div><br>The thing is, this but may be present in every 2D cell type.<br></div>I've checked vtkTriangle, but this one seems ok, pcoords[2] is overwriten.<br><br></div>What do you think about it ?<span><font color="#888888"><br><br clear="all"><div><div><div><div><div><div><div><div dir="ltr">Mathieu Westphal<br></div></div></div>
</div></div></div></div></div></font></span></div>
<br></div></div>_______________________________________________<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>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br></div>
<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>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" target="_blank">http://markmail.org/search/?q=vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
<br></blockquote></div><br></div></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div></div></div></div></div>
</blockquote></div><br></div></div></div></div>