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 [email protected] > > > > > > > On 5/30/12 8:03 AM, "Nicholas Tustison" <[email protected]> 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. > ________________________________ _______________________________________________ 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
