Skip to content

Add a demo of the HTML repr to "Compact estimator representations" example #21289

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
Closed
GaelVaroquaux opened this issue Oct 8, 2021 · 2 comments · Fixed by #24439
Closed

Add a demo of the HTML repr to "Compact estimator representations" example #21289

GaelVaroquaux opened this issue Oct 8, 2021 · 2 comments · Fixed by #24439
Labels
Documentation good first issue Easy with clear instructions to resolve

Comments

@GaelVaroquaux
Copy link
Member

Describe the issue linked to the documentation

The HTML repr is demoed in very few examples, and people might not discover it.

Suggest a potential alternative/fix

Evolve the example:
https://scikit-learn.org/stable/auto_examples/miscellaneous/plot_changed_only_pprint_parameter.html
to one with two sections: one on the compact representation and one demoing the HTML repr as on:
https://scikit-learn.org/stable/auto_examples/release_highlights/plot_release_highlights_0_23_0.html

The reason why we should modify the existing example and not create a new one is that too many examples make things hard to discover.
@reshamas
Copy link
Member

reshamas commented Oct 8, 2021

Referencing issues

GaelVaroquaux referenced this issue Oct 13, 2021
@reshamas @NicolasHug
@thomasjpfan @ogrisel
DOC Display diagram to pipeline example (#18758)
7601f87 
Co-authored-by: Nicolas Hug <contact@nicolas-hug.com>
Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
Co-authored-by: Olivier Grisel <olivier.grisel@ensta.org>
@cmarmo cmarmo added good first issue Easy with clear instructions to resolve help wanted labels Sep 14, 2022
@betatim
Copy link
Member

betatim commented Sep 14, 2022

I'll take a look at this.

@betatim betatim mentioned this issue Sep 14, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Documentation good first issue Easy with clear instructions to resolve
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

DOC Update estimator representation example betatim/scikit-learn
4 participants
@GaelVaroquaux @betatim @cmarmo @reshamas