Skip to content

Restore old behavior for threshold in FIL#6922

Merged
rapids-bot[bot] merged 6 commits intorapidsai:branch-25.08from
hcho3:deprecate_fil_threshold
Jun 25, 2025
Merged

Restore old behavior for threshold in FIL#6922
rapids-bot[bot] merged 6 commits intorapidsai:branch-25.08from
hcho3:deprecate_fil_threshold

Conversation

@hcho3
Copy link
Copy Markdown
Contributor

@hcho3 hcho3 commented Jun 19, 2025

In the old FIL, load() accepted the optional threshold parameter for binary classifiers. The new FIL moves the parameter to predict(). To avoid breaking existing user code, we should continue accepting threshold in load(), until 25.10.

@hcho3 hcho3 requested a review from a team as a code owner June 19, 2025 04:01
@hcho3 hcho3 requested review from bdice and csadorf June 19, 2025 04:01
@github-actions github-actions Bot added the Cython / Python Cython or Python issue label Jun 19, 2025
@hcho3 hcho3 added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels Jun 19, 2025
Comment thread python/cuml/cuml/fil/fil.pyx Outdated
Comment thread python/cuml/cuml/fil/fil.pyx
@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented Jun 25, 2025

/merge

@rapids-bot rapids-bot Bot merged commit 2902f2e into rapidsai:branch-25.08 Jun 25, 2025
67 checks passed
@hcho3 hcho3 deleted the deprecate_fil_threshold branch June 25, 2025 17:55
rapids-bot Bot pushed a commit that referenced this pull request Jul 1, 2025
Closes #6825
Requires #6922 to fix the CI

Authors:
  - Philip Hyunsu Cho (https://github.com/hcho3)

Approvers:
  - James Lamb (https://github.com/jameslamb)

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

Labels

Cython / Python Cython or Python issue improvement Improvement / enhancement to an existing function non-breaking Non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants