-
Notifications
You must be signed in to change notification settings - Fork 437
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
PEP8 in sims #884 #983
PEP8 in sims #884 #983
Conversation
@@ -66,7 +68,7 @@ def test_add_noise(): | |||
N = 50 | |||
S0 = 100 | |||
|
|||
options = dict(func=f, | |||
options=dict(func=f, |
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.
This was actually fine before, I believe.
Thanks. A few small comments. |
Fixed both. |
@@ -78,7 +80,7 @@ def test_add_noise(): | |||
vol = orbital_phantom(gtab, **options) | |||
|
|||
for snr in [10, 20, 30, 50]: | |||
vol_noise = orbital_phantom(gtab, snr=snr, **options) | |||
vol_noise = orbital_phantom(gtab, snr = snr, **options) |
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.
Sorry - missed this one before. This =
shouldn't have whitespace around it.
Real close here. One more teensy comment. |
Fixing it in 5! |
Did you get a chance to fix this one? |
Fixes dipy#884
This one needs a rebase. |
Hey @theaverageguy : mind giving this one a rebase? It's got some conflicts with other things that got merged into master since. |
I assume this has been abandoned, but feel free to reopen if you still want to work on this. |
Fixes #884