[ITK Community] [Insight-users] Generate gaussian noise image

[Bradley Lowekamp]

Hello,

Thanks for sharing, I didn't see a a commit with the changes you made. But I got you seeding of the NormalVariateGenerate. Did you write any tests? There is where some time is going to need to be spent.

I have an initial moving of the code into ITK up in gerrit:
http://review.source.kitware.com/#/c/14646/1

Eyes on the code are well come as I try to get some time to write some tests.

Brad


On Mar 7, 2014, at 12:35 PM, Laurent Chauvin <lchauvin at bwh.harvard.edu> wrote:

> Hello Bradley,
> 
> Here is the link of the code I updated.
> https://github.com/lchauvin/itkNoiseFilter
> 
> Thank you.
> -Laurent
> 
> 
> On Fri, Mar 7, 2014 at 6:41 AM, Bradley Lowekamp <blowekamp at mail.nih.gov> wrote:
> Laurent,
> 
> 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.
> 
> 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.
> 
> Thanks,
> Brad
> 
> 
> 
> On Mar 6, 2014, at 10:21 PM, Matt McCormick <matt.mccormick at kitware.com> wrote:
> 
>> Hi Laurent,
>> 
>> On Thu, Mar 6, 2014 at 8:49 PM, Laurent Chauvin <lchauvin at bwh.harvard.edu> wrote:
>> The code presented in the article is a bit out-dated. There is some minor changed to do to make it compatible with ITKv4.
>> 
>> It would be great see your fixes integrated so yourself and others can easily benefit from this work!
>>  
>> 
>> I made these changes locally, and it's compiling fine. However I only tested the AdditiveGaussianNoiseFilter for now.
>> I can quickly check the other ones.
>> 
>> What are the next steps to include this code in ITK ?
>> 
>> 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.
>> 
>> Thanks,
>> Matt
>> 
>> [1]  http://www.kitware.com/blog/home/post/557
>> 
>> [2] http://www.itk.org/Wiki/ITK/Policy_and_Procedures_for_Adding_Remote_Modules#Procedure_for_Adding_a_Remote_Module
>> _____________________________________
>> 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://www.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-users
> 
> 
> The information in this e-mail is intended only for the person to whom it is
> addressed. If you believe this e-mail was sent to you in error and the e-mail
> contains patient information, please contact the Partners Compliance HelpLine at
> http://www.partners.org/complianceline . If the e-mail was sent to you in error
> but does not contain patient information, please contact the sender and properly
> dispose of the e-mail.
> 
> 
> 
> 
> -- 
> Laurent Chauvin, MS
> Surgical Navigation and Robotics Laboratory, Radiology Department
> Brigham And Women's Hospital, Harvard Medical School
> http://wiki.ncigt.org/index.php/User:Lchauvin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/community/attachments/20140307/7405a978/attachment-0002.html>
-------------- next part --------------
_____________________________________
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://www.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-users