-
-
Notifications
You must be signed in to change notification settings - Fork 11k
BUG: AVX-512 path for ufunc.accumulate
is broken
#15597
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
Comments
here's a maximum example from the unit tests
the original pvals in the test is a view, but pvals_corrected_raw shouldn't be (Pdb) pval.flags |
@r-devulap can you ave a look? |
yup, taking a look .. |
Found the bug, |
I brought up this question when I submitted the patch (see comments #14867 (comment)). There are no tests covering |
…m and np.minimum Fixes bug in np.maximum.accumulate and np.minimum.accumulate See numpy#15597
We use
np.minimum.accumulate
in statsmodels. Recent pre-release tests have started failing on after calls tonp.minimum.accumulate
.This code only fails on systems with AVX-512. Passes on systems with AVX and AVX2.
Reproducing code example:
Failing job:
https://travis-ci.org/statsmodels/statsmodels/jobs/651893167
Error message:
No error, but wrong results. Likely strides are off since we often seem empty-like values in places.
Numpy/Python version information:
The text was updated successfully, but these errors were encountered: