Skip to content

MNT: scikit-learn compat: remove criterion param from boosting#582

Draft
cakedev0 wants to merge 1 commit into
sebp:mainfrom
cakedev0:mnt/sklean19/friedman_mse
Draft

MNT: scikit-learn compat: remove criterion param from boosting#582
cakedev0 wants to merge 1 commit into
sebp:mainfrom
cakedev0:mnt/sklean19/friedman_mse

Conversation

@cakedev0
Copy link
Copy Markdown
Contributor

@cakedev0 cakedev0 commented Feb 15, 2026

Hi, I'm a scikit-learn contributor and I was looking at your repo because I was investigating scikit-learn/scikit-learn#30554 that you opened. On the way, I stumbled on a few things that will crash with scikit-learn 1.9

Checklist

  • pytest passes => no, not compatible with scikit-learn 1.8
  • code is well formatted

What does this implement/fix? Explain your changes

See: scikit-learn/scikit-learn#32708

Note: this change is NOT compatible with scikit-learn 1.8.

@cakedev0 cakedev0 requested a review from sebp as a code owner February 15, 2026 21:54
@cakedev0 cakedev0 marked this pull request as draft February 15, 2026 21:54
@cakedev0 cakedev0 changed the title Remove criterion param from boosting MNT: scikit-learn compat: remove criterion param from boosting Feb 15, 2026
@cakedev0
Copy link
Copy Markdown
Contributor Author

JFYI @sebp https://pypi.org/project/scikit-learn/1.9.0rc1/ is out, scikit-learn 1.9 will be out soon, let me know when/if you want to bump the scikit-learn dependency, I should be able to help 👍

@sebp
Copy link
Copy Markdown
Owner

sebp commented May 25, 2026

@cakedev0 Thanks for the update. Please go ahead an bump scikit-learn. Let's see if it require additional changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants