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

FIX: PEP8 in tracking and tracking benchmarks/tests #952

Merged
merged 8 commits into from Mar 23, 2016

Conversation

Projects
None yet
2 participants
@manu-tej
Contributor

manu-tej commented Mar 1, 2016

issue #888 and #887 fixed

manu-tej added some commits Mar 1, 2016

@manu-tej manu-tej changed the title from PEP8 fixes in tracking benchmarks/tests to PEP8 fixes in tracking and tracking benchmarks/tests Mar 2, 2016

@manu-tej manu-tej changed the title from PEP8 fixes in tracking and tracking benchmarks/tests to FIX: PEP8 in tracking and tracking benchmarks/tests Mar 2, 2016

assert_array_almost_equal( vec1 + vec2, pf.add_3vecs(vec1, vec2))


def test_winding():

This comment has been minimized.

@arokem

arokem Mar 5, 2016

Member

Whoops. Looks like this test was deleted entirely?

@arokem

This comment has been minimized.

Member

arokem commented Mar 5, 2016

Just that one thing. Otherwise, LGTM.

@arokem

This comment has been minimized.

Member

arokem commented Mar 11, 2016

Anyone else have any comments here? If I don't hear anything in the next couple of days, I'll merge this.

@arokem arokem merged commit 9b58b00 into nipy:master Mar 23, 2016

This was referenced Mar 23, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment