[Insight-users] UnaryFunctorImageFilter and MetaDataDictionnary

[Luis Ibanez]

Hi Julien,

My Apologies for the typo in your name,
in the previous email.

I should pay more attention...


     Luis


-------------------------------------------------------------------------
On Wed, Oct 21, 2009 at 6:41 PM, Luis Ibanez <luis.ibanez at kitware.com> wrote:
> Hi Julie,
>
> Yes,
> please send me the patch and I will run an Experimental build with it.
>
> If it comes out Green, I'll commit the patch.
>
>
>    Thanks
>
>
>            Luis
>
>
>
> ---------------------------------------------------------------------------------------------
> On Tue, Oct 20, 2009 at 4:55 AM, Julien Michel <julien.michel at c-s.fr> wrote:
>> Julien Michel a écrit :
>>>
>>> Dear ITK developpers,
>>>
>>> Lately I bumped again into a bug which I first reported in 2007 on ITK
>>> users list :
>>> http://www.cmake.org/pipermail/insight-users/2007-March/021468.html
>>>
>>> I found back the issue I reported in Mantis
>>> (http://public.kitware.com/Bug/view.php?id=4625)
>>> and it seems it was not fixed because one can not be sure that the
>>> MetaDataDictionary still makes sense after some filter (which is a good
>>> point).
>>>
>>> I think we still need to copy the Dictionnary in most of the cases. Maybe
>>> we could add a CopyMetaDataDictionnary flag in the base class set to true by
>>> default and which could be used to deactivate the copy of the dictionnary
>>> when it does not make sense.
>>>
>>> Does it sounds like an acceptable solution to you ?
>>>
>>> Many thanks,
>>
>> I am really sorry to insist but this is a major issue for us. We are using
>> the metadata dictionnary to store valuable information for downstream
>> filters, and if there is one single UnaryFunctorImageFilter (which is
>> common) in this pipeline, these filters will not work.
>>
>> Besides, this is not really a MetaDataDictionnary issue, it is more the lack
>> of call to the CopyData() method in the GenerateOutputInformation(). We
>> could store these information elsewhere and still fail to got them
>> propagated through the pipeline.
>>
>> I can provide a patch, and will be glad to discuss this issue with ITK
>> developpers.
>>
>> Many thanks,
>>
>> Julien
>> --
>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> Julien MICHEL - Ingénieur d'études - Traitement d'images
>> CS Systèmes d'Information - Division ESPACE
>> Département Information Géographique & Image
>> Téléphone : +33 561 17 64 27
>> Email : julien.michel at c-s.fr
>>
>> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>>
>> _____________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Please keep messages on-topic and check the ITK FAQ at:
>> http://www.itk.org/Wiki/ITK_FAQ
>>
>> Follow this link to subscribe/unsubscribe:
>> http://www.itk.org/mailman/listinfo/insight-users
>>
>