Skip to content
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

Closed
wants to merge 4 commits into from

Conversation

jamesmyatt
Copy link

@jamesmyatt jamesmyatt commented Dec 7, 2023

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.

  • Improvement (non-breaking)
  • Bug fix (non-breaking)
  • New feature (non-breaking)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Testing

Please describe the tests that you ran to verify your changes. Consider listing any relevant details of your test configuration.

  • CI

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • My changes have been tested using both CPU and GPU devices

@jamesmyatt jamesmyatt changed the title Update scikit-learn requirement >=0.22.2,<1.4.0 Update scikit-learn requirement to >=0.22.2,<1.4.0 Dec 7, 2023
@beat-buesser
Copy link
Collaborator

Hi @jamesmyatt Thank you very much! I have started the unit-test workflows for this PR.

@beat-buesser beat-buesser self-requested a review December 13, 2023 13:49
@beat-buesser beat-buesser self-assigned this Dec 13, 2023
@beat-buesser beat-buesser added the dependencies Pull requests that update a dependency file label Dec 13, 2023
@jamesmyatt
Copy link
Author

@beat-buesser There seems to be a problem with Codecov:

[2023-12-13T14:18:25.530Z] ['error'] There was an error running the uploader: Error uploading to [https://codecov.io:](https://codecov.io/) Error: There was an error fetching the storage URL during POST: 404 - {'detail': ErrorDetail(string='Unable to locate build via Github Actions API. Please upload with the Codecov repository upload token to resolve issue.', code='not_found')}

@jamesmyatt jamesmyatt force-pushed the sklearn branch 2 times, most recently from d3ce0a5 to e9a03f6 Compare December 13, 2023 14:25
@codecov-commenter
Copy link

codecov-commenter commented Dec 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d7e91dd) 84.02% compared to head (63339b2) 21.24%.
Report is 10 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@             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     

see 262 files with indirect coverage changes

@beat-buesser
Copy link
Collaborator

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.

@jamesmyatt
Copy link
Author

jamesmyatt commented Dec 19, 2023

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.

@beat-buesser
Copy link
Collaborator

You're right. Can you try if it is possible to upgrade numba too? If I remember correctly it depends on the numpy version.

Signed-off-by: James Myatt <[email protected]>
Scikit-learn 1.0.0 was released Sep-2021

Signed-off-by: James Myatt <[email protected]>
@jamesmyatt
Copy link
Author

@beat-buesser Can you approve the workflows, please?

@beat-buesser
Copy link
Collaborator

Fixed by #2372

@jamesmyatt jamesmyatt deleted the sklearn branch January 15, 2024 13:25
@jamesmyatt jamesmyatt restored the sklearn branch January 15, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update version of scikit-learn in setup.py and CI yml files
3 participants