Skip to content

TEST: Nifti test bug; STY: PEP8 test_nifti1 #340

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

Merged
merged 3 commits into from
Sep 3, 2015

Conversation

effigies
Copy link
Member

While fixing tests on #332 PEP8, I cleaned the file and ran into a bug.

One outstanding PEP8 issue because I couldn't parse a comment enough to reword.

((2, minf), HDE, HDE, (2, minf))):
# Good slope and inter - you guessed it
((2, 0), None, (None, None), (2, 0)),
((2, 1), None, (None, None), (2, 1)),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The colon/comma swap was the bug. Last two conditions weren't iterated over but were unassigned values in the loop body.

Updated the values in the unrun tests to not fail, but this should be checked for validity.

@matthew-brett
Copy link
Member

LGTM

@effigies
Copy link
Member Author

Okay. Slightly modified your inline comment to fit in 79 characters, and moved the another_aff definition up with nice_aff but otherwise used your fix exactly.

@effigies
Copy link
Member Author

effigies commented Sep 3, 2015

Anything left to do on this one?

@matthew-brett
Copy link
Member

Thanks for the reminder - and the rewrite.

matthew-brett added a commit that referenced this pull request Sep 3, 2015
MRG: Nifti test bug; STY: PEP8 test_nifti1

PEP8 refactoring; restoring a couple of tests accidentally chopped.  Improving one test.
@matthew-brett matthew-brett merged commit 5e10a6d into nipy:master Sep 3, 2015
@effigies effigies deleted the nifti_test_bug branch September 3, 2015 21:18
@effigies effigies mentioned this pull request Nov 6, 2015
grlee77 pushed a commit to grlee77/nibabel that referenced this pull request Mar 15, 2016
MRG: Nifti test bug; STY: PEP8 test_nifti1

PEP8 refactoring; restoring a couple of tests accidentally chopped.  Improving one test.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants