-
Notifications
You must be signed in to change notification settings - Fork 57
Poisson, Gaussian, Complex Gaussian with tests #420
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
base: mg/noises/background
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have written some comments on the Gaussian noise class. Some of the comments also apply to the other classes. Once you have had a chance to address them, I can go through the remaining classes in more detail. Let me know if anything is unclear.
Offset = Background | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that we usually have two empty lines between different classes
Builds on mg/noises/background to implement torch testing with the backend test class.