8000 Fix ARM failures (gh-413) by rgommers · Pull Request #2879 · numpy/numpy · GitHub
[go: up one dir, main page]

Skip to content

Fix ARM failures (gh-413) #2879

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
Jan 3, 2013
Merged

Conversation

rgommers
Copy link
Member
@rgommers rgommers commented Jan 2, 2013

@certik commit 9ff6aed should be backported to 1.7.x.

@njsmith
Copy link
Member
njsmith commented Jan 2, 2013

(@certik: just to re-iterate what Ralf said in case you don't look at this closely -- he really does mean only that commit should be backported, the other commit that's included in this PR will break py24 compatibility. This is also why the Travis build failed.)

Am I right that the basic assumption here is that our FP exception checking stuff just doesn't work reliably in general, and so we might as well knownfail the relevant tests whenever this comes up? That's fine if so, but do we have a bug filed anywhere saying this? Seems like something we'd want to figure out how to fix sooner or later and have a central place to accumulate any analysis in the mean time...

Also @rgommers: If you're breaking the test suite on 2.4, can you remove it from .travis.yml as well, so the PR green/grey status remains useful?

@njsmith
Copy link
Member
njsmith commented Jan 2, 2013

Oh yes: actual changes look good to me.

@rgommers
Copy link
Member Author
rgommers commented Jan 3, 2013

There's gh-2350 and gh-593 at least, maybe there are others. Floating point warnings/errors are behaving different on different platforms. @charris expressed the opinion already that we can't fix this in numpy. Some improvement may be possible though.

Done because for 1.8 we're not supporting these Python versions anymore,
and this PR break 2.4 compatibility.
@rgommers
Copy link
Member Author
rgommers commented Jan 3, 2013

OK, Travis build is fixed.

njsmith added a commit that referenced this pull request Jan 3, 2013
@njsmith njsmith merged commit cce1c45 into numpy:master Jan 3, 2013
@certik
Copy link
Contributor
certik commented Jan 4, 2013

Backported in #2883.

@certik
Copy link
Contributor
certik commented Jan 4, 2013

@njsmith let me know if I did this right. I always create a PR for the backport, especially in cases like this, where that particular commit didn't have Travis tests attached to it, and the next commit had failures. All seems to be ok.

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.

3 participants
0