[Insight-developers] itkMRIBiasFieldCorrectionFilter.h

William A. Hoffman bill.hoffman@kitware.com
Mon, 20 Aug 2001 21:56:23 -0400


It is legal to use Numerics in the entire system, so that would not be a 
problem.
Even Common uses Numerics.

-Bill


At 04:17 PM 8/20/2001 -0700, Jisung Kim wrote:
>As I described before, the filter requires not only
>the itkMRASlabIdentifitier but also other classes in
>the Numerics and Numerics/Statistics directories.
>Therefore, even after I move the filter to the
>Algorithms directory, there would be include errors
>because of such classes that are in the Numerics
>directories.
>
>--- "William A. Hoffman" <bill.hoffman@kitware.com>
>wrote:
> > I think the problem is slightly different.
> > itkMRIBiasFieldCorrectionFilter.h:  ->#include
> > "itkMRASlabIdentifier.h"
> >
> > itkMRIBiasFieldCorrectionFilter is in BasicFilters,
> > but should be moved to
> > Algorithms.
> > In this case the filter is using something from
> > Algorithms and not just the
> > test case.
> >
> > -Bill
> >
> > At 01:49 AM 8/18/2001 -0700, Jisung Kim wrote:
> > >Hi.
> > >
> > >Sorry for my late response. I am back in my home
> > >country for getting the new U.S. visa and have some
> > >trouble to connect to the Internet.
> > >
> > >The problem seems to be the same kind of problem
> > that
> > >Damion had with blox image test programs (?) (see
> > july
> > >dev-list archive). If I'm
> > >correct, The cmakelist.txt file in the
> > >Testing/Code/BasicFilters specifies only the
> > >Code/Common directory for inclusion. However, the
> > >itkMRIBiasCorrectionFilter needs
> > >itkOnePlusOneOpimizer,
> > >itkMultivariateLegendrePolynomial,
> > >itkCompositValleyFunction (Code/Numerics)
> > >itkFastRandomUnitNormalVariateGenerator
> > >(Code/Numerics/Statistics), and
> > itkMRASlabIdentifier
> > >(Code/Algorithms).
> > >
> > >There were discussions over if the test directories
> > >should include all sub-directories under the Code
> > >directory or not. I think people said that we will
> > >discuss this issue after the beta release.
> > >At that time, Damion said he would temporarily move
> > >everything to the same directory where the blox
> > >image filter(?) is.
> > >
> > >I can't think of any quick fix for this problem.
> > One
> > >option would be move everything to the BasicFilters
> > >directory. This approch can fix the problem but I
> > >don't think it's good way to solve the problem.
> > >Since the problem was directly caused by adding the
> > >itkMRIBiasCorrectionFilter.h to the
> >
> >Testing/Code/BasicFilters/itkBasicFiltersHeaderTest.cxx,
> > >commenting out the include line will hide the
> > problem.
> > >
> > >For the discussion, LAPACK has three levels of
> > >routines: driver routines (solve complete
> > problems),
> > >computational routines (perform distinct
> > computational
> > >tasks), and auxiliary routines.
> > >(for more information, please refer to the LAPACK
> > user
> > >guide -
> > http://www.netlib.org/lapack/lug/node22.html)
> > >
> > >I think we may define levels for filters according
> > to
> > >their complexity, purpose, and dependency. For
> > >example, BasicFilters directory may only include
> > >general "independent" filters such as
> > LogImageFilter,
> > >and the "AdvancedFilters" directory include more
> > >complex and application-like filters such as
> > >itkMRIBiasCorrectionFilter.
> > >
> > >I am also discussing with Luis about this problem
> > to
> > >find a solution. Sorry again and I will try to
> > >response timely manner.
> > >
> > >--- Bill Hoffman <bill.hoffman@kitware.com> wrote:
> > > > This file seems to be including other files that
> > are
> > > > not checked into cvs.
> > > > If that were fixed, I think we would have at
> > least
> > > > some architectures that
> > > > do not have red on them...
> > > >
> > > > -Bill
> > > >
> > > >
> > > > _______________________________________________
> > > > Insight-developers mailing list
> > > > Insight-developers@public.kitware.com
> > > >
> >
> >http://public.kitware.com/mailman/listinfo/insight-developers
> > >
> > >
> > >=====
> > >Jisung Kim
> > >bahrahm@yahoo.com
> > >106 Mason Farm Rd.
> > >129 Radiology Research Lab., CB# 7515
> > >Univ. of North Carolina at Chapel Hill
> > >Chapel Hill, NC 27599-7515
> > >
> > >__________________________________________________
> > >Do You Yahoo!?
> > >Make international calls for as low as $.04/minute
> > with Yahoo! Messenger
> > >http://phonecard.yahoo.com/
> >
> >
> > _______________________________________________
> > Insight-developers mailing list
> > Insight-developers@public.kitware.com
> >
>http://public.kitware.com/mailman/listinfo/insight-developers
>
>
>=====
>Jisung Kim
>bahrahm@yahoo.com
>106 Mason Farm Rd.
>129 Radiology Research Lab., CB# 7515
>Univ. of North Carolina at Chapel Hill
>Chapel Hill, NC 27599-7515
>
>__________________________________________________
>Do You Yahoo!?
>Make international calls for as low as $.04/minute with Yahoo! Messenger
>http://phonecard.yahoo.com/
>
>_______________________________________________
>Insight-developers mailing list
>Insight-developers@public.kitware.com
>http://public.kitware.com/mailman/listinfo/insight-developers