Skip to content

FIX Allow prereleases when evaluating xfail conditions#6722

Merged
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
betatim:fix-version-comparison
May 15, 2025
Merged

FIX Allow prereleases when evaluating xfail conditions#6722
rapids-bot[bot] merged 2 commits intorapidsai:branch-25.06from
betatim:fix-version-comparison

Conversation

@betatim
Copy link
Copy Markdown
Member

@betatim betatim commented May 14, 2025

To test cuml.accel with the scikit-learn test suite we maintain a list of expected failures which can contain conditionals.

This PR makes it so that a condition like scikit-learn >= 1.6 will evaluate to true when scikit-learn 1.7.0rc0 is installed. This is useful when we want to test a upcoming release of scikit-learn (or other packages) for cuml.accel.

@betatim betatim requested a review from a team as a code owner May 14, 2025 14:53
@betatim betatim requested review from csadorf and divyegala May 14, 2025 14:53
@github-actions github-actions Bot added the Cython / Python Cython or Python issue label May 14, 2025
@csadorf csadorf added improvement Improvement / enhancement to an existing function non-breaking Non-breaking change labels May 14, 2025
@csadorf
Copy link
Copy Markdown
Contributor

csadorf commented May 14, 2025

/merge

@rapids-bot rapids-bot Bot merged commit a9fc610 into rapidsai:branch-25.06 May 15, 2025
89 checks passed
@betatim betatim deleted the fix-version-comparison branch May 15, 2025 07:30
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