-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update scikit-learn requirement to >=0.22.2,<1.4.0 #2346
Conversation
Hi @jamesmyatt Thank you very much! I have started the unit-test workflows for this PR. |
@beat-buesser There seems to be a problem with Codecov:
|
d3ce0a5
to
e9a03f6
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #2346 +/- ##
===========================================
- Coverage 84.02% 21.24% -62.79%
===========================================
Files 324 324
Lines 29988 29331 -657
Branches 5539 5028 -511
===========================================
- Hits 25198 6230 -18968
- Misses 3374 22982 +19608
+ Partials 1416 119 -1297 |
Hi @jamesmyatt Could you please take a look at one of the new runs? It seems that Python 3.11 is not yet available. I suggest we remove the second new run for Python 3.11 and also remove the oldest run for 0.24.2. |
Removing 0.24.2 makes sense. Looks like the issue for python 3.11 is the version of numba that's pinned for the tests. That probably needs updating too. |
You're right. Can you try if it is possible to upgrade |
Signed-off-by: James Myatt <[email protected]>
Signed-off-by: James Myatt <[email protected]>
Signed-off-by: James Myatt <[email protected]>
Scikit-learn 1.0.0 was released Sep-2021 Signed-off-by: James Myatt <[email protected]>
@beat-buesser Can you approve the workflows, please? |
Fixed by #2372 |
Description
Scikit-learn 1.1.x is over 1 year old. Also Python 3.10 is over 2 years old.
Fixes #1953,
Replaces #1964
Type of change
Please check all relevant options.
Testing
Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.
Checklist