Skip to content

diabetes instead of Boston dataset, eg feature selection #16305

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.

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

Conversation

maikia
Copy link
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 ogrisel and adrinjalali January 30, 2020 10:45
…into boston_exchange_ex_plot_select_from_model_boston
@adrinjalali
Copy link
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
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
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
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