[Insight-developers] module dependency issue

[Xiaoxiao Liu]

I didn't see the needs but have no objection of changing the module name
from "DeformationField" to"DisplacementField", if you think it helps
to clear out some confusion.



On Mon, Aug 22, 2011 at 9:33 AM, Nicholas Tustison <ntustison at gmail.com>wrote:

> I didn't know that.  That sounds quite appropriate.
> Related---I believe it was Matt M. who pointed out
> that, considering the context, the nomenclature
> "DeformationField" should be replaced with
> "DisplacementField" for our transform classes.  Is that
> change something that should be propagated to the
> module name as well?
>
>
> On Aug 22, 2011, at 9:28 AM, Xiaoxiao Liu wrote:
>
> We actually have a ITKDeformationField  module under Filtering, which
> depends on ITkTransform.
> Does is make sense to move all the displacement field transform classes
> into ITKDeformationField? which I think takes minimum work  too.
>
>
> On Mon, Aug 22, 2011 at 8:50 AM, Nicholas Tustison <ntustison at gmail.com>wrote:
>
>> So, specifically, are we creating a new folder
>> Modules/Registration/DisplacementFieldTransform
>> to house the new module?
>>
>>
>> On Aug 22, 2011, at 8:45 AM, Xiaoxiao Liu wrote:
>>
>> Gaetan's suggestion makes sense to me too.
>> It's always a little risky to introduce extra dependency to a core module.
>> itkBSplineDisplacementFieldTransform seems to be pretty specific. It
>> probably should stay in a module that is more relevant to image
>> registration.
>> Sounds like a good opportunity to take a bunch of displacement field
>> related classes out of Core/Transform.
>>
>>
>> On Mon, Aug 22, 2011 at 8:34 AM, Nicholas Tustison <ntustison at gmail.com>wrote:
>>
>>> That sounds good to me.  I'll wait and let Brian weigh in on the
>>> potential
>>> change and then I can proceed with a patch.
>>>
>>>
>>> On Aug 22, 2011, at 8:06 AM, Gaëtan Lehmann wrote:
>>>
>>> >
>>> > Hi Nick,
>>> >
>>> > I think it would make sense to move all the displacement field
>>> transform to a new module, for example ITKDisplacementFieldTransform.
>>> > That would help to fix the dependency issue with the ITKImageFunction
>>> module as well as this new one.
>>> >
>>> > Does that sound reasonable to you?
>>> >
>>> > Regards,
>>> >
>>> > Gaëtan
>>> >
>>> >
>>> > Le 22 août 11 à 13:49, Nicholas Tustison a écrit :
>>> >
>>> >> Hi Xiaoxiao,
>>> >>
>>> >> Thanks for taking a look at this.  The specific transform class in
>>> question
>>> >> is itkBSplineDisplacementFieldTransform which needs the class
>>> >> itkBSplineScatteredDataPointSetToImageFilter which resides in
>>> >> Filtering/ImageGrid/.  So is your suggestion to move the latter class
>>> to
>>> >> Core/Transform/?
>>> >>
>>> >> Thanks,
>>> >> Nick
>>> >>
>>> >>
>>> >> On Aug 21, 2011, at 9:10 PM, Xiaoxiao Liu wrote:
>>> >>
>>> >>> Hi Nick,
>>> >>>
>>> >>> If the transform class is not only specific to PICSL registration, it
>>> probably should be
>>> >>> a part of the ITKTransform module.
>>> >>> If so, how many new dependencies needs to be introduced to
>>> ITKTransform? And what are they?
>>> >>> You want to be careful not to introduce circular module dependencies
>>> during the process (CMake would warn you
>>> >>> , but it is better to figure out before doing it).
>>> >>>
>>> >>> -Xiaoxiao
>>> >>>
>>> >>>
>>> >>>
>>> >>>
>>> >>>
>>> >>> On Sun, Aug 21, 2011 at 7:41 PM, Nicholas Tustison <
>>> ntustison at gmail.com> wrote:
>>> >>> Hi,
>>> >>>
>>> >>> Part of the PICSL registration refactoring includes the introduction
>>> of a
>>> >>> transform class which has a dependency outside of the Core/Transform
>>> >>> dependency scope.  What is done in this situation?
>>> >>>
>>> >>> Thanks,
>>> >>> Nick
>>> >>> _______________________________________________
>>> >>> Powered by www.kitware.com
>>> >>>
>>> >>> Visit other Kitware open-source projects at
>>> >>> http://www.kitware.com/opensource/opensource.html
>>> >>>
>>> >>> Kitware offers ITK Training Courses, for more information visit:
>>> >>> http://kitware.com/products/protraining.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-developers
>>> >>>
>>> >>>
>>> >>>
>>> >>> --
>>> >>>
>>> >>>
>>> >>> ---------------------------------------------
>>> >>> Xiaoxiao Liu, Ph.D.
>>> >>> R & D Engineer
>>> >>> Kitware Inc.
>>> >>> Clifton Park, NY
>>> >>> Phone: (518) 881-4924  or  (518) 371-3971 x124
>>> >>>
>>> >>>
>>> >>
>>> >> _______________________________________________
>>> >> Powered by www.kitware.com
>>> >>
>>> >> Visit other Kitware open-source projects at
>>> >> http://www.kitware.com/opensource/opensource.html
>>> >>
>>> >> Kitware offers ITK Training Courses, for more information visit:
>>> >> http://kitware.com/products/protraining.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-developers
>>> >
>>> > --
>>> > Gaëtan Lehmann
>>> > Biologie du Développement et de la Reproduction
>>> > INRA de Jouy-en-Josas (France)
>>> > tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
>>> > http://mima2.jouy.inra.fr  http://www.itk.org
>>> > http://www.bepo.fr
>>> >
>>>
>>>
>>
>>
>> --
>>
>>
>> ---------------------------------------------
>> *Xiaoxiao Liu*, Ph.D.
>> R & D Engineer
>> Kitware Inc <http://www.kitware.com/>.
>> Clifton Park, NY
>> Phone: (518) 881-4924  or  (518) 371-3971 x124
>>
>>
>>
>>
>
>
> --
>
>
> ---------------------------------------------
> *Xiaoxiao Liu*, Ph.D.
> R & D Engineer
> Kitware Inc <http://www.kitware.com/>.
> Clifton Park, NY
> Phone: (518) 881-4924  or  (518) 371-3971 x124
>
>
>
>


-- 


---------------------------------------------
*Xiaoxiao Liu*, Ph.D.
R & D Engineer
Kitware Inc <http://www.kitware.com/>.
Clifton Park, NY
Phone: (518) 881-4924  or  (518) 371-3971 x124
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110822/49d3691d/attachment.htm>