-
Notifications
You must be signed in to change notification settings - Fork 101
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
[BUG] NumPy 1.24: test regression ValueError: cannot convert float NaN to integer
#423
Comments
https://build.opensuse.org/request/show/1045128 by user mcepl + dimstar_suse Forwarded request #1045057 from bnavigator - Skip a failing test -- gh#pydata/bottleneck#423 - Add rpmlintrc
This is now reported also as a bug in corresponding debian package which need to be addressed soonish. A hacky way of getting this moving would be to create the
I know this looks ugly, but just a suggestion which kind of works for a few combinations that I tried. |
@nileshpatra Can you just check the performance impact of your fix? You can run the fix with and without the fix on the previous numpy version and see how that impacts. if it's negligible, I think we can go on with the PR. |
Hi @rdbisme I ran the benchmark with the current HEAD of bottleneck (master branch) and numpy 1.23.5 (wherein the code works fine). As it seems, the benchmark numbers are not consistent across runs (there are deltas of +/- 15% and I do not have the time/energy to do more runs/analysis on this), but the runs with and without patches do not differ much. Let me know what you think about this. Without the patch:
With the patch:
|
@rdbisme We'd need to fix this in debian package soonish since it affects some key packages. And hence, could you please comment if the above report looks OK to you or you'd like something more? |
@nileshpatra Sounds good. Feel free to open a PR and we'll get it merged fast! :) |
@rdbisme done, please consider to check. |
Describe the bug
When NumPy is upgrade to 1.24, the following test fails:
Downgrading to 1.23.5 makes tests pass.
To Reproduce
To assist in reproducing the bug, please include the following:
git clone https://github.com/pydata/bottleneck
pip install bottleneck pytest
python -c "import bottleneck;bottleneck.test()"
Expected behavior
Tests passing ;-).
Additional context
Tested on a825e20.
The text was updated successfully, but these errors were encountered: