<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Vivien,<div><br></div><div>We are still looking at your patch. &nbsp;I need to dig into what coordinate frames the various calculations are being done.</div><div><br></div><div>However, with a change like this where the output of a test needs to change, it is best to correct the test. &nbsp;This may simply require submitting a new baseline image. This is a two step process as the baseline images are maintained using a content based hashing system. &nbsp;The input file in the repository is just a small text file containing the md5 hash for the new image. &nbsp;That file needs to be updated. &nbsp;Then the new image needs to be uploaded to the system which serves the baseline images.</div><div><br></div><div>I "think" this can all be done at the gerrit-push stage. &nbsp;Just put the updated baseline image where it needs to go and the build process and gerrit-push process should create the md5 version of the file and push the new data to the server.&nbsp;</div><div><br></div><div>Here is link that describes the process</div><div><br></div><div><a href="http://www.itk.org/Wiki/ITK/Git/Develop/Data">http://www.itk.org/Wiki/ITK/Git/Develop/Data</a></div><div>&nbsp;</div><div>Jim</div><div><br><div><div>On May 16, 2012, at 4:24 AM, Vivien Delmon wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div>Hi,<br><br>I pushed my first path into guerrit here: <a href="http://review.source.kitware.com/#/c/5639/">http://review.source.kitware.com/#/c/5639/</a><br><br>Hans J. Johnson<span class="Apple-tab-span" style="white-space:pre">        </span>requested for a build in which a test fail which is normal since my patch changes the geometry used by the RayCastInterpolator. What should I do now ? correct this test to take into account the physical coordinate system of the image and submit a change ?<br><br>-- <br>Vivien Delmon<br>PhD student at CREATIS, Lyon<br>Supported by ANRT and ELEKTA<br><a href="http://www.creatis.insa-lyon.fr/rio">http://www.creatis.insa-lyon.fr/rio</a><br><br>_______________________________________________<br>Powered by www.kitware.com<br><br>Visit other Kitware open-source projects at<br>http://www.kitware.com/opensource/opensource.html<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></div></blockquote></div><br><div apple-content-edited="true">
<span class="Apple-style-span" style="font-family: Helvetica; font-size: 12px; "><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><b><font color="#000000" size="2" face="Arial">Jim Miller</font></b><font size="2" face="Arial Unicode MS"></font><br><font color="#000000" size="2" face="Arial">Senior Scientist</font><br><font color="#000000" size="2" face="Arial">GE Research</font>&nbsp;<br><font class="Apple-style-span" face="Arial"><span class="Apple-style-span" style="font-size: small; ">Interventional and Therapy</span></font></div><div><font class="Apple-style-span" face="Arial"><span class="Apple-style-span" style="font-size: small; "><br></span></font><font color="#0000FF" size="2" face="Arial">GE imagination at work</font></div></div></span>
</div>
<br></div></body></html>