I've committed the patch. It will be in vtk 5.8.<div><br></div><div>Thanks!</div><div><br clear="all">David E DeMarle<br>Kitware, Inc.<br>R&D Engineer<br>28 Corporate Drive<br>Clifton Park, NY 12065-8662<br>Phone: 518-371-3971 x109<br>


<br><br><div class="gmail_quote">On Wed, Jun 8, 2011 at 10:32 AM, Andreas Buykx <span dir="ltr"><<a href="mailto:A.Buykx@tnodiana.com">A.Buykx@tnodiana.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">







<div lang="NL" link="blue" vlink="purple">
<div>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">Hi Dave, and vtk-developers,</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">Here’s the patch. Because there are not Cxx tests with vtkSelectVisiblePoints in VTK, and we don’t have Tcl, I tried it in our application where
 it had exactly the expected behavior: no error messages anymore as the ones described below, and visible points selected as before.</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">In what release will this patch appear? We use the latest release 5.6.1, but your nightly build is already 5.9!</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">Thanks,</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D">Andreas</span></p>
<p class="MsoNormal"><span lang="EN-US" style="font-size:11.0pt;color:#1F497D"> </span></p>
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt">From:</span></b><span lang="EN-US" style="font-size:10.0pt"> David E DeMarle [mailto:<a href="mailto:dave.demarle@kitware.com" target="_blank">dave.demarle@kitware.com</a>]
<br>
<b>Sent:</b> Tuesday, June 07, 2011 2:31 PM<br>
<b>To:</b> Andreas Buykx<br>
<b>Cc:</b> <a href="mailto:vtk-developers@vtk.org" target="_blank">vtk-developers@vtk.org</a><br>
<b>Subject:</b> Re: [vtk-developers] vtkSelectVisiblePoints on empty data set</span></p><div><div></div><div class="h5">
<p class="MsoNormal"> </p>
<p class="MsoNormal">That is probably an oversight. Try changing the vtkSelectVisiblePoints::RequestData to do an early check for empty input dataset and make it produce an empty output dataset and return 1. </p>
<div>
<p class="MsoNormal"> </p>
</div>
<div>
<p class="MsoNormal">If that change doesn't break any regression tests on your machine please send the patch to the list and we'll push it to the repository.</p>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><br clear="all">
David E DeMarle<br>
Kitware, Inc.<br>
R&D Engineer<br>
28 Corporate Drive<br>
Clifton Park, NY 12065-8662<br>
Phone: <a href="tel:518-371-3971%20x109" value="+15183713971" target="_blank">518-371-3971 x109</a><br>
<br>
</p>
<div>
<p class="MsoNormal">2011/6/4 Andreas Buykx <<a href="mailto:A.Buykx@tnodiana.com" target="_blank">A.Buykx@tnodiana.com</a>></p>
<div>
<div>
<p class="MsoNormal"><span lang="EN-US">Hi all,</span></p>
<p class="MsoNormal"><span lang="EN-US"> </span></p>
<p class="MsoNormal"><span lang="EN-US">I have a pipeline in my application that should display the ids of visible points, using a.o. the vtkSelectVisiblePoints filter. The input to this filter is a
 – possibly empty - selection of the original data set. </span></p>
<p class="MsoNormal"><span lang="EN-US">When the set is in fact empty, I get a lot of the following errors:</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">ERROR: In <path-deleted>/VTK/src/Filtering/vtkExecutive.cxx, line 753</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">vtkStreamingDemandDrivenPipeline (0x32b8700): Algorithm vtkSelectVisiblePoints(0x3168ea0) returned failure for request: vtkInformation (0x3340180)</span></p>


<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  Debug: Off</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  Modified Time: 23448</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  Reference Count: 1</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  Registered Events: (none)</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  Request: REQUEST_DATA</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  FROM_OUTPUT_PORT: 0</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  FORWARD_DIRECTION: 0</span></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span lang="EN-US" style="font-size:9.0pt;font-family:"Courier New"">  ALGORITHM_AFTER_FORWARD: 1</span></p>
<p class="MsoNormal"><span lang="EN-US"> </span></p>
<p class="MsoNormal"><span lang="EN-US">I looked in the code and found that this filter returns a 0 from RequestData if the input dataset is empty. In my situation an empty input dataset is a valid
 input condition and I’d like to prevent all these errors. What are my options?</span></p>
<p class="MsoNormal"><span lang="EN-US"> </span></p>
<p class="MsoNormal"><span lang="EN-US">Thanks for your help!</span></p>
<p class="MsoNormal" style="text-autospace:none">
<b><span lang="EN-US" style="font-size:10.0pt;color:black">Andreas Buykx</span></b></p>
<p class="MsoNormal" style="text-autospace:none">
<span lang="EN-US" style="font-size:10.0pt;color:black">Senior Software Engineer</span></p>
<p class="MsoNormal" style="text-autospace:none">
<span lang="EN-US" style="font-size:10.0pt;color:black"> </span></p>
<p class="MsoNormal" style="text-autospace:none">
<b><span lang="EN-US" style="font-size:11.5pt;color:#CB2027">TNO DIANA BV </span>
</b></p>
<p class="MsoNormal" style="text-autospace:none">
<i><span lang="EN-US" style="font-size:8.0pt;color:black">Software Developers and Analysis Consultants for Civil and Geotechnical Engineering
</span></i></p>
<p class="MsoNormal" style="text-autospace:none">
<span lang="EN-US" style="font-size:8.0pt;color:black"> </span></p>
<p class="MsoNormal" style="text-autospace:none">
<span lang="EN-US" style="font-size:9.0pt;color:black">Delftechpark 19a, 2628 XJ, Delft, The Netherlands
</span></p>
<p class="MsoNormal" style="text-autospace:none">
<span lang="EN-US" style="font-size:9.0pt;color:black">Tel: <a href="tel:%2B31%2088%2034262%2015" target="_blank">
+31 88 34262 15</a> (Direct) </span><span lang="EN-US" style="color:black">│ Tel:
<a href="tel:%2B31%2088%2034262%2000" target="_blank">+31 88 34262 00</a> (Switchboard) │
<a href="tel:%2B31%2088%2034262%2099" target="_blank">+31 88 34262 99</a> (Fax) </span>
</p>
<p class="MsoNormal" style="text-autospace:none">
<span style="font-size:9.0pt;color:black"><a href="http://tnodiana.com/" target="_blank"><span lang="EN-US">http://tnodiana.com</span></a></span><b><i><span style="font-size:9.0pt">
</span></i></b></p>
<p class="MsoNormal" style="text-autospace:none">
<b><i><span lang="EN-US" style="font-size:9.0pt"> </span></i></b></p>
<p class="MsoNormal"><b><i><span lang="EN-US" style="font-size:9.0pt">…be green keep it on screen</span></i></b></p>
<p class="MsoNormal"><span lang="EN-US"> </span></p>
</div>
<p class="MsoNormal"><span style="font-family:"Courier New"">____________________________________________________________<br>
TNO DIANA BV is a limited liability company registered in the trade register of the Chamber of Commerce as TNO DIANA BV with registered number 27252655.<br>
____________________________________________________________<br>
This e-mail and its contents are subject to the DISCLAIMER at <a href="http://tnodiana.com/content/Disclaimer" target="_blank">
http://tnodiana.com/content/Disclaimer</a><br>
____________________________________________________________</span></p>
</div>
<p class="MsoNormal" style="margin-bottom:12.0pt"><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>
</p>
</div>
<p class="MsoNormal"> </p>
</div>
</div>
</div></div></div><div><div></div><div class="h5">
<font face="monospace">____________________________________________________________<br>
TNO DIANA BV is a limited liability company registered in the trade register of the Chamber of Commerce as TNO DIANA BV with registered number 27252655.<br>
____________________________________________________________<br>
This e-mail and its contents are subject to the DISCLAIMER at <a href="http://tnodiana.com/content/Disclaimer" target="_blank">http://tnodiana.com/content/Disclaimer</a><br>
____________________________________________________________</font></div></div></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>