Skip to content

[MRG+1] Finishing #5321, Optional verbosity for Pipeline#7163

Closed
jnothman wants to merge 1 commit intoscikit-learn:masterfrom
jnothman:pipeline-verbose
Closed

[MRG+1] Finishing #5321, Optional verbosity for Pipeline#7163
jnothman wants to merge 1 commit intoscikit-learn:masterfrom
jnothman:pipeline-verbose

Conversation

@jnothman
Copy link
Copy Markdown
Member

@jnothman jnothman commented Aug 8, 2016

Just letting CIs run here pending merge in place of #5321. Fixes #5321, #5298.

Adds named verbosity argument in Pipeline constructor.

After each step, verbose pipelines print to standard output lines like:
> [Pipeline] <step name>, <action> (fit or transform), <duration>

- Added :func:`metrics.pairwise.laplacian_kernel`. By `Clyde Fare <https://github.com/Clyde-fare>`_.

<<<<<<< fc66544f931fdde8d4beda323fd5e7b05949fd16
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.

rebase stuff...

@amueller amueller changed the title Finishing #5321, Optional verbosity for Pipeline [MRG+1] Finishing #5321, Optional verbosity for Pipeline Aug 26, 2016
@amueller
Copy link
Copy Markdown
Member

doctest failure

@jnothman jnothman closed this Sep 12, 2016
@amueller
Copy link
Copy Markdown
Member

?

@jnothman
Copy link
Copy Markdown
Member Author

Too much work for a 0.18 rush. Leaving it to #5321

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