Skip to content

Rician Noise Transform #2054

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
lyndonboone opened this issue Apr 20, 2021 · 2 comments · Fixed by #2066
Closed

Rician Noise Transform #2054

lyndonboone opened this issue Apr 20, 2021 · 2 comments · Fixed by #2066

Comments

@lyndonboone
Copy link
Contributor

Is your feature request related to a problem? Please describe.
Working with augmentation for DL pipelines, it would be nice to have a noise transform that resembles MRI noise more closely.

Describe the solution you'd like
Would it be possible to add a transform that models Rician noise, as a complement to the current Gaussian noise transform? Example of a Rician noise transform implemented in DIPY:
https://dipy.org/documentation/1.1.1./reference/dipy.sims/#dipy.sims.voxel.add_noise
https://github.com/dipy/dipy/blob/c846bf5a23b7e95343a9cf231df2653473602456/dipy/sims/voxel.py (line 60)

I'd be more than happy to try and submit a pull request.

@wyli
Copy link
Contributor

wyli commented Apr 20, 2021

welcome! please have a look at the contributing doc in case you haven't seen it https://github.com/Project-MONAI/MONAI/blob/master/CONTRIBUTING.md#preparing-pull-requests

@lyndonboone
Copy link
Contributor Author

Thanks for the reply, @wyli ! I just opened up a WIP pull request for the proposed transform ( #2066 )

@wyli wyli mentioned this issue Apr 28, 2021
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants