[Insight-developers] a small bug in itkConjugateGradientOptimizer

[Luis Ibanez]

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