[Insight-developers] New valgrind defect in itkGradientDescentLineSearchOptimizerv4Test

[Nicholas Tustison]

Thanks, Kent, for the explanation.  That sounds like an easy fix.


On May 30, 2012, at 10:56 AM, Williams, Norman K wrote:

> If you look at itkSetMacro, it always checks the new value against the old
> value, in order to decide whether to mark the current object as modified,
> which determines whether the current pipeline stage is dirty and needs
> re-evaluation.
> 
> In a constructor, the old value is garbage, hence the valgrind warning.
> The only consequence in practice is that it is checking against a garbage
> value.  I'm not sure the constructor is the right place to mark an object
> 'Modified' though; the base class constructors have already been called,
> and I believe an itk::Object always starts life as 'modified.'
> --
> Kent Williams norman-k-williams at uiowa.edu
> 
> 
> 
> 
> 
> 
> On 5/30/12 8:03 AM, "Nicholas Tustison" <ntustison at gmail.com> wrote:
> 
>> Hi Bill,
>> 
>> If you take a look at the constructor in
>> itkGradientDescentLineSearchOptimizerv4.cxx
>> below, the search method is being set using a function.  Could this be
>> the possible
>> reason for the valgrind error?  I think you were the one who explained to
>> me why this
>> isn't kosher---something about the class not being instantiated at this
>> point to be able
>> do this type of thing.
>> 
>> Nick
>> 
>> /**
>> * Default constructor
>> */
>> GradientDescentLineSearchOptimizerv4
>> ::GradientDescentLineSearchOptimizerv4()
>> {
>> this->m_LowerLimit = itk::NumericTraits< InternalComputationValueType
>>> ::Zero;
>> this->m_UpperLimit = 5.0;
>> this->m_Phi = 1.618034;
>> this->m_Resphi = 2 - this->m_Phi;
>> this->m_Epsilon = 0.01;
>> this->SetSearchMethod( SearchNearBaselineLearningRate );
>> }
>> 
>> 
>> 
>> 
>> 
>> 
>> 
>> On May 30, 2012, at 8:26 AM, Bill Lorensen wrote:
>> 
>>> Folks,
>>> 
>>> Recent changes caused this valgrind defect:
>>> http://open.cdash.org/viewDynamicAnalysisFile.php?id=2757313
>>> 
>>> Bill
>>> _______________________________________________
>>> 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.php
>>> 
>>> 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
>> 
>> _______________________________________________
>> 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.php
>> 
>> 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
> 
> 
> 
> ________________________________
> Notice: This UI Health Care e-mail (including attachments) is covered by the Electronic Communications Privacy Act, 18 U.S.C. 2510-2521, is confidential and may be legally privileged.  If you are not the intended recipient, you are hereby notified that any retention, dissemination, distribution, or copying of this communication is strictly prohibited.  Please reply to the sender that you have received the message in error, then delete it.  Thank you.
> ________________________________