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

Skip to content

Conversation

GaelVaroquaux
Copy link
Member

  • Remove "advanced" so that everybody reads it, including beginners
  • Make the title shorter

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Any other comments?

- Remove "advanced" so that everybody reads it, including beginners
- Make the title shorter
Copy link

✔️ Linting Passed

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

Generated for commit: 3c1ca25. Link to the linter CI: here

Copy link
Member

@thomasjpfan thomasjpfan left a comment

Choose a reason for hiding this comment

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

LGTM

@thomasjpfan thomasjpfan added Quick Review For PRs that are quick to review Waiting for Second Reviewer First reviewer is done, need a second one! labels Feb 22, 2024
@lucyleeow lucyleeow merged commit d1e7caf into scikit-learn:main Feb 23, 2024
@lucyleeow
Copy link
Member

Thanks @GaelVaroquaux !

@reshamas reshamas removed the Waiting for Second Reviewer First reviewer is done, need a second one! label Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Quick Review For PRs that are quick to review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants