CI Update xfail list

[betatim]

I think what happened is that #7751 fixed something that used to lead to the check_do_not_raise_errors_in_init_or_set_params check failing. The reason we ended up seeing it in #7632 is that #7751 was merged before #7753 (and we didnt rerun the CI for that PR).

[coderabbitai]

📝 Walkthrough

Summary by CodeRabbit

• Tests
  • Updated sklearn compatibility test expectations for multiple estimators (including KMeans, LogisticRegression, SVC, PCA, TSNE, and UMAP) by removing expected failure checks for initialization and parameter setting operations, reflecting improved behavioral alignment with sklearn standards.

Walkthrough

Removes xfail expectations for "check_do_not_raise_errors_in_init_or_set_params" from multiple estimators in the sklearn compatibility test suite, broadening which checks are no longer expected to fail during initialization and parameter setting.

Changes

Cohort / File(s)	Summary
Test Compatibility Configuration python/cuml/tests/test_sklearn_compatibility.py	Removed xfail mappings for "check_do_not_raise_errors_in_init_or_set_params" from PER_ESTIMATOR_XFAIL_CHECKS for ~26 estimators including KMeans, KernelRidge, LogisticRegression, LinearRegression, Ridge, NearestNeighbors, SVC/SVR variants, PCA variants, TSNE, UMAP, Lasso, ElasticNet, clustering and Naive Bayes estimators. RandomForestRegressor entry retained.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• CI Switch to strict xfail policy for common estimator checks #7753: Also modifies xfail expectations in test_sklearn_compatibility.py by removing expected-failure entries for estimator checks.

Suggested labels

improvement, non-breaking

Suggested reviewers

• csadorf
• viclafargue

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'CI Update xfail list' clearly and concisely describes the main change: removing outdated xfail entries from the test compatibility checks.
Description check	✅ Passed	The description explains why the xfail entries are being removed, referencing specific PRs that fixed the underlying issues.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[jcrist]

[betatim]

/merge