[Insight-developers] a small bug in itkConjugateGradientOptimizer

[Ashish Poddar]

Hi Luis,

Thank you for the quick action. probably similar change is required
for Levenberg Marquardt Optimizer too.

with regards,
Ashish.

On 5/16/05, Luis Ibanez <luis.ibanez at kitware.com> wrote:
> 
> Hi Ashish,
> 
> Thanks for pointing this out.
> 
> You are right, the GetValue() method should be const.
> 
> A fix has now been committed to the CVS repository.
> 
> Please let us know if you encounter any other problem.
> 
>     Thanks
> 
>        Luis
> 
> ----------------------
> Ashish Poddar wrote:
> > hi,
> >
> > I am not sure whether it qualifies as a bug or not, but surely affects
> > the re-usability and pluggability model of ITK Library.
> >
> > the GetValue() function in ConjugateGradientOptimizer class currently is
> >
> > MeasureType GetValue();
> >
> > but in case of RegularStepGradientDescentOptimizer class its defined by macro as
> >
> > MeasureType GetValue() const;
> >
> > which is an interface mis-match... This I encountered when i replaced
> > regular step gradient descent optimizer by conjugate gradient
> > optimizer. In the observer I was using a const reference of the
> > optimizer and displaying the value (just the example which is
> > available for the same nothing new =D)...
> >
> > with regards,
> > Ashish.
> >
> 
> 


-- 
Ashish Poddar
Have an acceptable reason for accepting anything.
Y:ashish_poddar | MSN:ashish_poddar at yahoo.com