[vtk-developers] No api for vtkAlgorithm::SetInputData()

[Chris Harris]

I am of the opinion that if this is a pretty uncommon use case lets
not pollute the API with another method ( that we may struggle to find
a decent name for ). We should just document the trivial producer
approach.

My 2 cents

Chris

On Fri, Apr 13, 2012 at 10:22 AM, Bill Lorensen <bill.lorensen at gmail.com> wrote:
> Why not reactivate GetProducerPort()? And do not add a new Set method.
>
> On Fri, Apr 13, 2012 at 9:18 AM, Berk Geveci <berk.geveci at kitware.com> wrote:
>> I am fine with making this change with the exception of calling it
>> "SetInputData". The moment you did that, you will start seeing hidden method
>> warning in all subclasses that define SetInputData with another type as
>> input. SetInputDataObject? SetInputDataAsDataObject?
>>
>> -berk
>>
>> On Thu, Apr 12, 2012 at 6:29 PM, Pat Marion <pat.marion at kitware.com> wrote:
>>>
>>> Here is a example, which may not be very common, but I had it in my
>>> code...
>>>
>>> void foo(vtkAlgorithm* aFilter, vtkDataObject* aData)
>>> {
>>>   vtkDataObject* aDataCopy = copyData(aData);
>>>   aFilter->SetInputConnection(aDataCopy->GetProducerPort());
>>>   ...
>>> }
>>>
>>> In VTK 6, I can't find a generic way to connect a data object to an
>>> algorithm.
>>> I have to create a trivial producer (reimplement
>>> vtkAlgorithm::SetInputDataInternal)
>>>
>>> I'm curious if there is a reason not to change protected
>>> SetInputDataInternal to public SetInputData?
>>>
>>> Most subclasses define SetInputData as:
>>>
>>> void vtkPolyDataAlgorithm::SetInputData(int index, vtkDataObject* input)
>>> {
>>>   this->SetInputDataInternal(index, input);
>>> }
>>>
>>>
>>> Pat
>>>
>>> _______________________________________________
>>> Powered by www.kitware.com
>>>
>>> Visit other Kitware open-source projects at
>>> http://www.kitware.com/opensource/opensource.html
>>>
>>> Follow this link to subscribe/unsubscribe:
>>> http://www.vtk.org/mailman/listinfo/vtk-developers
>>>
>>>
>>
>>
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Follow this link to subscribe/unsubscribe:
>> http://www.vtk.org/mailman/listinfo/vtk-developers
>>
>>
>
>
>
> --
> Unpaid intern in BillsBasement at noware dot com
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtk-developers
>