Skip to content

Pin scikit-learn<1.8 in test dependencies#7588

Merged
gforsyth merged 2 commits intorapidsai:release/25.12from
csadorf:pin-sklearn-to-lt-18-in-tests
Dec 10, 2025
Merged

Pin scikit-learn<1.8 in test dependencies#7588
gforsyth merged 2 commits intorapidsai:release/25.12from
csadorf:pin-sklearn-to-lt-18-in-tests

Conversation

@csadorf
Copy link
Copy Markdown
Contributor

@csadorf csadorf commented Dec 10, 2025

While cuML itself is compatible with scikit-learn 1.8, some of our dependencies (umap-learn, hdbscan, xgboost) are not. A very small number of tests is not yet compatible either.

@csadorf csadorf requested a review from a team as a code owner December 10, 2025 15:53
@csadorf csadorf requested a review from msarahan December 10, 2025 15:53
@github-actions github-actions Bot added conda conda issue Cython / Python Cython or Python issue labels Dec 10, 2025
@csadorf csadorf added bug Something isn't working non-breaking Non-breaking change labels Dec 10, 2025
Comment thread dependencies.yaml Outdated
Copy link
Copy Markdown
Member

@jcrist jcrist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: (assuming tests pass)

Comment thread conda/environments/all_cuda-129_arch-aarch64.yaml
@gforsyth gforsyth merged commit 0b41aeb into rapidsai:release/25.12 Dec 10, 2025
105 checks passed
@csadorf csadorf deleted the pin-sklearn-to-lt-18-in-tests branch December 10, 2025 17:52
csadorf added a commit to csadorf/cuml that referenced this pull request Dec 11, 2025
csadorf added a commit to csadorf/cuml that referenced this pull request Dec 12, 2025
csadorf added a commit to csadorf/cuml that referenced this pull request Dec 12, 2025
rapids-bot Bot pushed a commit that referenced this pull request Dec 15, 2025
This PR reverts #7588. and updates the test suite to ensure compatibility with scikit-learn 1.8.

Closes #7599.

Authors:
  - Simon Adorf (https://github.com/csadorf)

Approvers:
  - Jim Crist-Harif (https://github.com/jcrist)
  - Gil Forsyth (https://github.com/gforsyth)
  - Bradley Dice (https://github.com/bdice)

URL: #7589
mani-builds pushed a commit to mani-builds/cuml that referenced this pull request Jan 11, 2026
While cuML itself is compatible with scikit-learn 1.8, some of our
dependencies (umap-learn, hdbscan, xgboost) are not. A very small number
of tests is not yet compatible either.
mani-builds pushed a commit to mani-builds/cuml that referenced this pull request Jan 11, 2026
This PR reverts rapidsai#7588. and updates the test suite to ensure compatibility with scikit-learn 1.8.

Closes rapidsai#7599.

Authors:
  - Simon Adorf (https://github.com/csadorf)

Approvers:
  - Jim Crist-Harif (https://github.com/jcrist)
  - Gil Forsyth (https://github.com/gforsyth)
  - Bradley Dice (https://github.com/bdice)

URL: rapidsai#7589
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working conda conda issue Cython / Python Cython or Python issue non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants