Thanks Bill! I'm dlrdave on the gitlab site. Looks like Ben added me as a reviewer.<div><br></div><div>D</div><div><br><br>On Friday, October 16, 2015, Bill Lorensen <<a href="mailto:bill.lorensen@gmail.com">bill.lorensen@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I just submitted a merge request to add triangle strips to the writer.<br>
<a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/777" target="_blank">https://gitlab.kitware.com/vtk/vtk/merge_requests/777</a><br>
<br>
Dave C., I did not see your name as a reviewer on the gitlab site.<br>
<br>
Bill<br>
<br>
On Thu, Oct 15, 2015 at 1:17 PM, Bill Lorensen <<a href="javascript:;" onclick="_e(event, 'cvml', 'bill.lorensen@gmail.com')">bill.lorensen@gmail.com</a>> wrote:<br>
> I looked at the STLWriter code:<br>
><br>
> 1) It reports an error if a polygon is not a triangle.<br>
> 2) It reports nothing if there are triangle strips.<br>
><br>
> At a minimum I think it should handle triangle strips. No need to run<br>
> triangle filter, I'll decompose the strips on the fly.<br>
><br>
> Working on a patch and a unit test.<br>
><br>
> Bill<br>
><br>
> On Thu, Oct 15, 2015 at 9:45 AM, Bill Lorensen <<a href="javascript:;" onclick="_e(event, 'cvml', 'bill.lorensen@gmail.com')">bill.lorensen@gmail.com</a>> wrote:<br>
>> I think the writer should handle triangle strips. I'll take a stab at adding<br>
>> that functionality.<br>
>><br>
>> On Oct 15, 2015 9:13 AM, "Ben Boeckel" <<a href="javascript:;" onclick="_e(event, 'cvml', 'ben.boeckel@kitware.com')">ben.boeckel@kitware.com</a>> wrote:<br>
>>><br>
>>> On Wed, Oct 14, 2015 at 22:34:25 -0400, David Cole via vtkusers wrote:<br>
>>> > Shouldn't the STL writer detect that its input contains non-triangles,<br>
>>> > and<br>
>>> > **automatically** insert a triangle filter to produce correct output?<br>
>>> > Or,<br>
>>> > at the very least, spit out an error message saying "here, insert this<br>
>>> > code<br>
>>> > before calling Update/Write, and it will work like you want it to" ... ?<br>
>>><br>
>>> Well, it used to drop vertices past the third on the floor (silently),<br>
>>> but now it should be erroring out about non-triangles at least:<br>
>>><br>
>>>     commit 9a00db82900bd15442cc471143c4e5dd1ac09a2d<br>
>>>     Author: Ben Boeckel <<a href="javascript:;" onclick="_e(event, 'cvml', 'ben.boeckel@kitware.com')">ben.boeckel@kitware.com</a>><br>
>>>     Date:   Tue Feb 25 17:16:58 2014 -0500<br>
>>><br>
>>>         STLWriter: Raise an error for non-triangles<br>
>>><br>
>>>         The STL format only supports triangles and previously, it just<br>
>>>         truncated polygons silently. Cause an error to occur when this<br>
>>>         happens so that users know.<br>
>>><br>
>>>         Change-Id: I9c2f96d887074943130a8442d6796cb0c2ce1bbf<br>
>>><br>
>>> The commit is in VTK 6.2.0.<br>
>>><br>
>>> > (Similarly for rendering crappy polydata using the polydata mapper: it<br>
>>> > requires up-to-date normals to render with proper lighting and shading,<br>
>>> > and<br>
>>> > seems like the perfect candidate for **automatically** inserting a<br>
>>> > polydata<br>
>>> > normals filter. This might be harder to detect that the normals are not<br>
>>> > up<br>
>>> > to date, but sure would have saved my team some time recently...)<br>
>>><br>
>>> It should error out like the STL writer for bad input at the least.<br>
>>><br>
>>> > Would these types of improvements be welcome additions to VTK, or am I<br>
>>> > missing some case where you would NOT want this type of behavior?<br>
>>><br>
>>> As long as it lets you know that your data is bad and that (and how) it<br>
>>> is being fixed is warned about, I don't see any reason why such an<br>
>>> auto-fixer filter couldn't exist (whether it is named vtkSTLWriter or<br>
>>> something else is a different question I don't have enough feel too<br>
>>> strongly about).<br>
>>><br>
>>> --Ben<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<br>
>>> <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
>>><br>
>>> Please keep messages on-topic and check the VTK FAQ at:<br>
>>> <a href="http://www.vtk.org/Wiki/VTK_FAQ" target="_blank">http://www.vtk.org/Wiki/VTK_FAQ</a><br>
>>><br>
>>> Search the list archives at: <a href="http://markmail.org/search/?q=vtkusers" target="_blank">http://markmail.org/search/?q=vtkusers</a><br>
>>><br>
>>> Follow this link to subscribe/unsubscribe:<br>
>>> <a href="http://public.kitware.com/mailman/listinfo/vtkusers" target="_blank">http://public.kitware.com/mailman/listinfo/vtkusers</a><br>
><br>
><br>
><br>
> --<br>
> Unpaid intern in BillsBasement at noware dot com<br>
<br>
<br>
<br>
--<br>
Unpaid intern in BillsBasement at noware dot com<br>
</blockquote></div>