To be totally frank, I don't feel enough ownership of the imaging classes to care :-) It doesn't sound like a big deal but someone that uses these classes more regularly should also chime in.<br><br><div class="gmail_quote">
On Fri, Nov 19, 2010 at 4:55 PM, David Gobbi <span dir="ltr"><<a href="mailto:david.gobbi@gmail.com">david.gobbi@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Hi Berk,<br>
<br>
Since I've caught your ear, I might as well get your opinion on one<br>
other matter.  It also has to do with image stencils.<br>
<br>
Right now some filters have a SetStencil() method that takes a<br>
vtkImageStencilData object directly.  I'm converting these to<br>
SetStencilConnection() methods.  But I'm thinking of adding the<br>
SetStencilConnection() method directly to vtkImageAlgorithm, and<br>
here is why: I want to be able to probe any ImageAlgorithm to see<br>
if that algorithm accepts a stencil as an input (and I want to make<br>
more of the existing image filters use stencils, I've written an iterator<br>
to simplify this.)<br>
<br>
In my proposal, an image algorithm that uses a stencil will do this in<br>
its constructor:<br>
<br>
vtkSomeFilter::vtkSomeFilter()<br>
{<br>
  this->SetNumberOfInputs(2);<br>
  this->SetStencilPortNumber(1);<br>
}<br>
<br>
The default value of StencilPortNumber set to -1 by vtkImageAlgorithm<br>
(i.e. no stencil input), so the existence of a stencil port can be<br>
checked.  The vtkImageAlgorithm::SetStencilConnection() method<br>
will connect its argument to the port, or do nothing if the port is -1.<br>
You can see the new vtkImageAlgorithm code on Gerrit:<br>
<br>
<a href="http://review.source.kitware.com/388" target="_blank">http://review.source.kitware.com/388</a><br>
<br>
If you think that SetStencilConnection() belongs in the individual<br>
filters (e.g. like SetSelectionConnection()) then that is fine. The<br>
only answer I will not accept is "derive a new base class from<br>
vtkImageAlgorithm". ;)<br>
<font color="#888888"><br>
  David<br>
</font></blockquote></div><br>