Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

lucyleeow
Copy link
Member

Reference Issues/PRs

Follow on from #30399

What does this implement/fix? Explain your changes.

  • fix copy paste error in name parameter, and clarify when name cannot be list
  • Add from_cv_results to see also entries

Any other comments?

Copy link

github-actions bot commented Sep 21, 2025

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 057ccc2. Link to the linter CI: here

Copy link

@mishra-krishna mishra-krishna left a comment

Choose a reason for hiding this comment

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

The additions to the See Also sections are a great idea for improving discoverability. I noticed a small duplication in the docstring of from_cv_results that you might want to address.

Copy link
Member

@ArturoAmorQ ArturoAmorQ left a comment

Choose a reason for hiding this comment

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

Thanks @lucyleeow!

@ArturoAmorQ ArturoAmorQ merged commit c9a0d2d into scikit-learn:main Sep 22, 2025
36 checks passed
@lucyleeow lucyleeow deleted the doc_fix_roc_disp branch September 22, 2025 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants