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
[MAINT] Test against Numpy 2.0 nightly releases #4130
Conversation
ymzayek
commented
Dec 5, 2023
- Closes Testing against numpy 2.0 #4129
👋 @ymzayek Thanks for creating a PR! Until this PR is ready for review, you can include the [WIP] tag in its title, or leave it as a github draft. Please make sure it is compliant with our contributing guidelines. In particular, be sure it checks the boxes listed below.
For new features:
For bug fixes:
We will review it as quick as possible, feel free to ping us with questions if needed. |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #4130 +/- ##
==========================================
- Coverage 91.84% 91.77% -0.08%
==========================================
Files 144 144
Lines 16224 16241 +17
Branches 3377 3386 +9
==========================================
+ Hits 14901 14905 +4
- Misses 781 791 +10
- Partials 542 545 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
# TODO remove on pull request | ||
pull_request: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove before merging PR
# TODO remove on pull request | |
pull_request: |
The failure still seen here in the numpy2 job: test_decoder_binary_classification_with_masker_object |
Was following the commits of this to see where it was going. Thanks for taking care of that.
Sounds like a sensible approach. You can even mark the test as xfail for this type of nightly release tests until this is fixed. Though maybe not necessary if you plan to do this in afollow up PR right away. |
Actually yes good idea. I'll open up an issue to track the sklearn issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thx
Merging, then. |