Skip to content
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

TST: fix a number of test failures on 32-bit systems #6709

Merged
merged 3 commits into from Oct 22, 2016

Conversation

@rgommers
Copy link
Member

rgommers commented Oct 22, 2016

No description provided.

@rgommers rgommers added this to the 0.19.0 milestone Oct 22, 2016
@ev-br

This comment has been minimized.

Copy link
Member

ev-br commented Oct 22, 2016

I was going to hit the green button under the assumption that 64-bit CI is happy. But the pep8 bot is vigilant!

The marked tests simply error out on 32-bit linux, but
they weren't accurate on 64-bit either.

The changed value for one parameter makes that test pass.
@rgommers rgommers force-pushed the rgommers:fix-failures branch from f55ebf2 to a915248 Oct 22, 2016
@rgommers

This comment has been minimized.

Copy link
Member Author

rgommers commented Oct 22, 2016

hmm, sloppy. fixed

@ev-br

This comment has been minimized.

Copy link
Member

ev-br commented Oct 22, 2016

Meaningful tests have passed already, pep8 run is green, so merging. Thanks Ralf!

@ev-br ev-br merged commit 5e6fc06 into scipy:master Oct 22, 2016
0 of 2 checks passed
0 of 2 checks passed
ci/circleci CircleCI is running your tests
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@rgommers rgommers deleted the rgommers:fix-failures branch Oct 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.