<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Laurent,<div><br></div><div>Giving the interest and generality of this filter. It should be integrated in a proper ITK module and not a Remote. We should be able to get plenty of eyes on to bring it up to ITK standards. Additionally, I really want easy access to this in SimpleITK.</div><div><br></div><div>Please submit what you have to gerrit/github/e-mail me and I'll make time to polish it up, and experiment with my proposed scan line based reproducibility.</div><div><br></div><div>Thanks,</div><div>Brad</div><div><br></div><div><br></div><div><br><div><div>On Mar 6, 2014, at 10:21 PM, Matt McCormick <<a href="mailto:matt.mccormick@kitware.com">matt.mccormick@kitware.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"><div dir="ltr">Hi Laurent,<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 6, 2014 at 8:49 PM, Laurent Chauvin <span dir="ltr"><<a href="mailto:lchauvin@bwh.harvard.edu" target="_blank">lchauvin@bwh.harvard.edu</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">The code presented in the article is a bit out-dated. There is some minor changed to do to make it compatible with ITKv4.</div>
</blockquote><div><br></div><div>It would be great see your fixes integrated so yourself and others can easily benefit from this work!</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div><br></div><div>I made these changes locally, and it's compiling fine. However I only tested the AdditiveGaussianNoiseFilter for now.</div>
<div>I can quickly check the other ones.</div><div><br></div><div>What are the next steps to include this code in ITK ?</div></div></blockquote><div><br></div><div>Reorganize the code into an ITK Module [1], then submit a small patch [2]. Please let us know if you have any questions or issues.</div>
<div><br></div><div>Thanks,</div><div>Matt</div><div><br></div><div>[1] <a href="http://www.kitware.com/blog/home/post/557">http://www.kitware.com/blog/home/post/557</a></div><div><br></div><div>[2] <a href="http://www.itk.org/Wiki/ITK/Policy_and_Procedures_for_Adding_Remote_Modules#Procedure_for_Adding_a_Remote_Module">http://www.itk.org/Wiki/ITK/Policy_and_Procedures_for_Adding_Remote_Modules#Procedure_for_Adding_a_Remote_Module</a></div>
</div></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://www.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-users<br></blockquote></div><br></div></body></html>