Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[MRG] EXA improve examples included #8866

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

Merged
merged 1 commit into from
May 12, 2017

Conversation

glemaitre
Copy link
Member

Reference Issue

Follow of #8851

What does this implement/fix? Explain your changes.

This PR attends to:

  • Remove the "Outpout:: ..." of the docstring since the example is now executed.
  • Improve the Pipeline example by adding some common split train/test and classification report to have an output.

Any other comments?

@glemaitre
Copy link
Member Author

@lesteve Open to any suggestion

@lesteve
Copy link
Member

lesteve commented May 12, 2017

LGTM, merging.

To be honest, moving print(__doc__) is not mandatory since we ignore flake8 warnings about imports not at the top of file in flake8_diff.sh.

@lesteve lesteve merged commit 3de7da3 into scikit-learn:master May 12, 2017
Sundrique pushed a commit to Sundrique/scikit-learn that referenced this pull request Jun 14, 2017
dmohns pushed a commit to dmohns/scikit-learn that referenced this pull request Aug 7, 2017
dmohns pushed a commit to dmohns/scikit-learn that referenced this pull request Aug 7, 2017
NelleV pushed a commit to NelleV/scikit-learn that referenced this pull request Aug 11, 2017
paulha pushed a commit to paulha/scikit-learn that referenced this pull request Aug 19, 2017
AishwaryaRK pushed a commit to AishwaryaRK/scikit-learn that referenced this pull request Aug 29, 2017
maskani-moh pushed a commit to maskani-moh/scikit-learn that referenced this pull request Nov 15, 2017
jwjohnson314 pushed a commit to jwjohnson314/scikit-learn that referenced this pull request Dec 18, 2017
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.

2 participants