Skip to content

diabetes instead of Boston dataset, eg feature selection#16305

Merged
rth merged 12 commits intoscikit-learn:masterfrom
maikia:boston_exchange_ex_plot_select_from_model_boston
Feb 5, 2020
Merged

diabetes instead of Boston dataset, eg feature selection#16305
rth merged 12 commits intoscikit-learn:masterfrom
maikia:boston_exchange_ex_plot_select_from_model_boston

Conversation

@maikia
Copy link
Copy Markdown
Contributor

@maikia maikia commented Jan 30, 2020

Reference Issues/PRs

Towards #16155

Removed:
examples/feature_selection/plot_select_from_model_boston.py
Introduced instead using diabetes dataset:
examples/feature_selection/plot_select_from_model_diabetes.py

Old final figure:
before

New final figure:
after

What does this implement/fix? Explain your changes.

Any other comments?

@maikia maikia changed the title [wip] diabetes instead of Boston dataset, ex feature selection diabetes instead of Boston dataset, eg feature selection Jan 30, 2020
@maikia maikia requested review from adrinjalali and ogrisel January 30, 2020 10:45
…into boston_exchange_ex_plot_select_from_model_boston
@adrinjalali
Copy link
Copy Markdown
Member

This is how this example is rendered now: https://91364-843222-gh.circle-artifacts.com/0/doc/auto_examples/feature_selection/plot_select_from_model_diabetes.html

And recently we have been using some nice features which make an example look like this one: https://scikit-learn.org/dev/auto_examples/inspection/plot_partial_dependence.html#sphx-glr-auto-examples-inspection-plot-partial-dependence-py

I'm happy with what you have here, but if you have the time and feel like improving the way the example looks on the website, it'd be awesome :)

Copy link
Copy Markdown
Member

@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

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

thanks @maikia

the baseline.

Authors: Manoj Kumar <mks542@nyu.edu>
Maria Telenczuk
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

you could also point to your github user account page if you want, instead of email. But leaving it as is is also fine.

@maikia maikia requested a review from rth February 5, 2020 09:01
…into boston_exchange_ex_plot_select_from_model_boston
Copy link
Copy Markdown
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

Very nice improvement of the example, thanks @maikia !

@rth rth merged commit 6324e40 into scikit-learn:master Feb 5, 2020
thomasjpfan pushed a commit to thomasjpfan/scikit-learn that referenced this pull request Feb 22, 2020
panpiort8 pushed a commit to panpiort8/scikit-learn that referenced this pull request Mar 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants