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

Skip to content

DOC remove basic iris example and merge with iris with PCA example #29964

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 15, 2024
Merged

DOC remove basic iris example and merge with iris with PCA example #29964

merged 1 commit into from
Oct 15, 2024

Conversation

glemaitre
Copy link
Member

Partially addressing #27151

We have two really redundant examples:

  • One presenting the iris dataset and apply a PCA
  • One presenting PCA by applying it on the iris dataset

I therefore rework to keep a single example such that we have a good enough narration and remove one of the example.

@github-actions GitHub Actions
Copy link

✔️ Linting Passed

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

Generated for commit: 65eb7fa. Link to the linter CI: here

@adrinjalali adrinjalali merged commit 019e953 into scikit-learn:main Oct 15, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adrinjalali adrinjalali adrinjalali approved these changes

Assignees
No one assigned
Labels
Documentation module:datasets
Projects
Documentation improvement
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@glemaitre @adrinjalali