<div class="gmail_quote">On Wed, Aug 25, 2010 at 2:26 PM, pat marion <span dir="ltr"><<a href="mailto:pat.marion@kitware.com">pat.marion@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi David,<br><br>Sorry I didn't get back to you on this.  I don't really want to argue for or against this change, but here are some comments...<br><br>vtkKdTreePointLocator is a leaf class that implements the 
vtkAbstractPointLocator interface.  It would be best if new interface 
methods were be added at the abstract layer and then implemented in each leaf
 class.<br><br>A potential problem with your proposed implementation is that 
it assumes the vtkDataSet is a vtkPointSet, which may not be true. <br><br>I 
definitely agree that the proposed api is convenient, but it doesn't add
 new power to the class.<br><font color="#888888">
<br>
Pat<br><br></font><div class="gmail_quote"><div><div></div><div class="h5">On Wed, Aug 25, 2010 at 9:15 AM, David Doria <span dir="ltr"><<a href="mailto:daviddoria%2Bvtk@gmail.com" target="_blank">daviddoria+vtk@gmail.com</a>></span> wrote:<br>
</div></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex"><div><div></div><div class="h5">
<div><div class="gmail_quote">
<blockquote style="border-left:1px solid rgb(204, 204, 204);margin:0px 0px 0px 0.8ex;padding-left:1ex" class="gmail_quote">
<div class="gmail_quote">
<div>
<div class="gmail_quote">Because I bet it is a common thing that many users have to deal with fighting all of the time. It just seems like it should be provided. What would be the harm in giving access a few ways?</div></div>




</div>
<div class="gmail_quote"><br><font color="#888888">
<div>David</div></font></div></blockquote></div>
</div><div><br>Ok so we have 1 yay and 1 nay - any other comments?</div>
<div> </div>
<div>I would like to propose that we standardize some kind of voting system for "thumbs up/thumbs down" type of feature additions like this. What always seems to happen without this type of system is someone posts a suggestion, a couple of people make small comments, and then it dies. There is no reason small things like this should take more than a day or two to "accept", "accept with major revisions", or "reject" - but they seem to take months.</div>



<div><br></div><div>I have created a vote for the new FindPointsWithinRadius function here:</div><div><span style="font-family:arial,sans-serif;font-size:12.5px;border-collapse:collapse"><a href="http://www.surveymonkey.com/s/GNFMZ8K" style="color:rgb(0, 0, 204)" target="_blank">http://www.surveymonkey.com/s/GNFMZ8K</a></span></div>



<div><br></div><div><span style="font-family:arial,sans-serif;font-size:12.5px;border-collapse:collapse">The problem may remain that very few people vote (the same as very few people reply on the mailing list). There needs to be a system where if no one votes then the default action is to move forward. I support adding things unless someone is opposed (with a good reason) versus NOT adding things unless lots of people are actively supportive.</span></div>





<div><br><font color="#888888">David</font></div>
<br></div></div><div class="im">_______________________________________________<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></div></blockquote></div><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><br></div><div>Pat didn't want to argue for or against this change, but his comments speak for themselves. I think his comments should count as -2 in your voting scheme, unless you re-implement your patch to account for both comments.</div>
<div><br></div><div>David C.</div><div><br></div>