Skip to content

DOC remove redundant example on MDI feature importance#29965

Merged
adrinjalali merged 2 commits intoscikit-learn:mainfrom
glemaitre:remove_plot_forest_importances_faces
Oct 15, 2024
Merged

DOC remove redundant example on MDI feature importance#29965
adrinjalali merged 2 commits intoscikit-learn:mainfrom
glemaitre:remove_plot_forest_importances_faces

Conversation

@glemaitre
Copy link
Copy Markdown
Member

Partially addressing #27151

Removing one of the example on MDI feature importance since we have two of them + an additional one discussing the cons and comparing it to permutation importance.

@github-actions
Copy link
Copy Markdown

✔️ Linting Passed

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

Generated for commit: 83d9729. Link to the linter CI: here

Copy link
Copy Markdown
Member

@virchan virchan left a comment

Choose a reason for hiding this comment

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

LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants