Skip to content

DOC Add links, update plot_all_scaling#17340

Merged
glemaitre merged 2 commits intoscikit-learn:masterfrom
lucyleeow:plot_allscale
May 26, 2020
Merged

DOC Add links, update plot_all_scaling#17340
glemaitre merged 2 commits intoscikit-learn:masterfrom
lucyleeow:plot_allscale

Conversation

@lucyleeow
Copy link
Copy Markdown
Member

@lucyleeow lucyleeow commented May 25, 2020

Reference Issues/PRs

What does this implement/fix? Explain your changes.

  • Link to internal California dataset description
  • Link to class documentation in text
  • Switch QuantileTransformer (Gaussian output) and QuantileTransformer (uniform output) as it seems that QuantileTransformer (uniform output) should go first, as it explains the transformer.
  • some wording changes

Any other comments?

@lucyleeow lucyleeow changed the title WIP DOC Add links, format plot_all_scaling DOC Add links, format plot_all_scaling May 25, 2020
@lucyleeow lucyleeow changed the title DOC Add links, format plot_all_scaling DOC Add links, update plot_all_scaling May 25, 2020
Copy link
Copy Markdown
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

thanks @lucyleeow Looks good

@glemaitre glemaitre merged commit 75c76ae into scikit-learn:master May 26, 2020
@glemaitre
Copy link
Copy Markdown
Member

Thanks @lucyleeow

@lucyleeow lucyleeow deleted the plot_allscale branch May 26, 2020 10:38
viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
jayzed82 pushed a commit to jayzed82/scikit-learn that referenced this pull request Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants