<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hi Michael and Brian,<div><br></div><div>If you take a look at the valgrind error that Brad linked</div><div>to, it's basically the same as the two other registration</div><div>method valgrind errors where we perform optimization </div><div>within the method class itself because they have to be </div><div>handled individually. What's interesting is that the only </div><div>other similar class where we don't get a valgrind error is</div><div>the itkTimeVaryingBSplineVelocityFieldImageRegistrationMethod</div><div>class but I just looked and there was a bug in which we</div><div>were calling the metric GetValueAndDerivative() function</div><div>twice. I fixed that here</div><div><br></div><div><a href="http://review.source.kitware.com/#/c/4823/">http://review.source.kitware.com/#/c/4823/</a></div><div><br></div><div>but I bet you that by fixing it we'll add another valgrind </div><div>error. </div><div><br></div><div>What I can't figure out is why. For standard registration</div><div>in which we're using the base class, </div><div>ImageToImageRegistrationMethod, the optimizer calls</div><div>the GetValueAndDerivative() function and there's no</div><div>valgrind error. What I'm wondering if is if the valgrind </div><div>checking sees the repeated resizing of the metric </div><div>derivative as a problem.</div><div><br></div><div>Nick</div><div><br></div><div><br></div><div><br><div><div>On Mar 26, 2012, at 9:20 AM, Bradley Lowekamp wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Hello,</div><div><br></div><div>I was just closely looking at the dashboard, and noticed that the valgrind number of defect increased to 3. This one looks like the new one.</div><div><br></div><a href="http://open.cdash.org/viewDynamicAnalysisFile.php?id=2726550">http://open.cdash.org/viewDynamicAnalysisFile.php?id=2726550</a><div><br></div><div><br></div><div>If there other two defects are not going to be addressed and they are considered false positives, they really should be suppressed, so that new defects can more easily be seen.</div><div><br></div><div>Brad</div><div><br><div>
<div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; font-size: 12px; "><span class="Apple-style-span" style="border-collapse: separate; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; -webkit-text-decorations-in-effect: none; text-indent: 0px; -webkit-text-size-adjust: auto; text-transform: none; orphans: 2; white-space: normal; widows: 2; word-spacing: 0px; "><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">========================================================</font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">Bradley Lowekamp<span class="Apple-converted-space"> </span><span class="Apple-converted-space"> </span></font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">Medical Science and Computing for</font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">Office of High Performance Computing and Communications</font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; ">National Library of Medicine<span class="Apple-converted-space"> </span></font></div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; "><font face="Helvetica" size="3" style="font: normal normal normal 12px/normal Helvetica; "><a href="mailto:blowekamp@mail.nih.gov">blowekamp@mail.nih.gov</a></font></div><br class="Apple-interchange-newline"></span></div><br class="Apple-interchange-newline">
</div>
<br></div></div>_______________________________________________<br>Powered by <a href="http://www.kitware.com">www.kitware.com</a><br><br>Visit other Kitware open-source projects at<br><a href="http://www.kitware.com/opensource/opensource.html">http://www.kitware.com/opensource/opensource.html</a><br><br>Kitware offers ITK Training Courses, for more information visit:<br>http://kitware.com/products/protraining.php<br><br>Please keep messages on-topic and check the ITK FAQ at:<br>http://www.itk.org/Wiki/ITK_FAQ<br><br>Follow this link to subscribe/unsubscribe:<br>http://www.itk.org/mailman/listinfo/insight-developers<br></blockquote></div><br></div></body></html>