[Insight-developers] module dependency issue

[Nicholas Tustison]

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
> 
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.itk.org/mailman/private/insight-developers/attachments/20110822/979d8624/attachment.htm>