<div dir="ltr">Thank you Simon,<div><br></div><div>Please submit that patch via gerrit to facilitate automated code review and eventual integration into the master branch.<br></div><div><br></div><div>For instructions see: <a href="http://www.vtk.org/Wiki/VTK/Git/Develop">http://www.vtk.org/Wiki/VTK/Git/Develop</a> and contact me if you need assistance.</div>

</div><div class="gmail_extra"><br clear="all"><div>David E DeMarle<br>Kitware, Inc.<br>R&D Engineer<br>21 Corporate Drive<br>Clifton Park, NY 12065-8662<br>Phone: 518-881-4909</div>
<br><br><div class="gmail_quote">On Tue, Mar 18, 2014 at 9:21 AM, Simon Kolotov <span dir="ltr"><<a href="mailto:simonkolotov@gmail.com" target="_blank">simonkolotov@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr">Hi.<br><div class="gmail_quote"><div dir="ltr"><div style="font-family:georgia,serif"><br></div><div style="font-family:georgia,serif">When using the vtk-stripper I came across a problem already encountered by many, namely that the stripper can't connect polylines.<br>





<br><b>Carl Hetherington </b>proposed a patch for this problem in 2002<div class="gmail_default" style="font-family:georgia,serif;display:inline">.  (I'm curious as to why Carl Hetherington's patch from 2002 was not included 
into the release version of vtk. Is there a hidden problem with it?)<br><br></div></div><div style="font-family:georgia,serif">

However, even with his patch there were some problems, and the stripper did not manage to correctly connect all the polylines.<br><br></div><div style="font-family:georgia,serif">Having invested  considerable time to find and correct this issue I'd be happy to share it with others.<br>





<br></div><div style="font-family:georgia,serif">Attached is the corrected vtkSlicer.cxx file, with both Carl's and my corrections<div class="gmail_default" style="font-family:georgia,serif;display:inline">, as well as my diff relative to Carl's patch.</div>



<br><br></div><div style="font-family:georgia,serif">

My correction to Carl's patch is relatively minor, namely doing InitTraversal to the newLines structure at every inner loop so as not to miss previously discarded lines (All my additions are around line 557)<br></div>





<div style="font-family:georgia,serif"><br><br></div><div style="font-family:georgia,serif">Respectfully,<br clear="all"></div><div><div dir="ltr"><div style="font-family:georgia,serif;display:inline">

Simon Kolotov</div></div></div>
</div>
<br></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>
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>