10000 [feature request] change verbosity level of _fit_and_score method · Issue #12958 · scikit-learn/scikit-learn · GitHub
[go: up one dir, main page]
Skip to content

[feature request] change verbosity level of _fit_and_score method #12958

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
AlJohri opened this issue Jan 11, 2019 · 4 comments
Closed

[feature request] change verbosity level of _fit_and_score method #12958

AlJohri opened this issue Jan 11, 2019 · 4 comments

Comments

@AlJohri
Copy link
AlJohri commented Jan 11, 2019

If I set verbose=2 within RandomizedSearchCV, I ideally want to see the steps that Parallel takes:

[Parallel(n_jobs=-1)]: Done   2 out of  10 | elapsed:   40.9s remaining:  2.7min

but I don't want to see the intermediate reporting of the CV search since that gets super noisy:

[CV] estimator__alpha=1.356430250055455e-06 ..........................
[CV] estimator__alpha=1.356430250055455e-06 ..........................
[CV] estimator__alpha=8.022557928125194e-06 ..........................
[CV] estimator__alpha=1.1751658491093436e-05 .........................
[CV] estimator__alpha=1.1751658491093436e-05 .........................
[CV] estimator__alpha=1.1751658491093436e-05 .........................
[CV] estimator__alpha=1.356430250055455e-06 ..........................
[CV] estimator__alpha=8.022557928125194e-06 ..........................

if we could change verbose > 1 to verbose > 2 this would solve my usecase.

scikit-learn/sklearn/model_selection/_validation.py

Lines 482 to 488 in acb8106

if verbose > 1:
if parameters is None:
msg = ''
else:
msg = '%s' % (', '.join('%s=%s' % (k, v)
for k, v in parameters.items()))
print("[CV] %s %s" % (msg, (64 - len(msg)) * '.'))

I'd be happy to submit a PR if this makes sense.

Related:
@jnothman
Copy link
Member
jnothman commented Jan 13, 2019 via email

@AlJohri
Copy link
Author
AlJohri commented Jan 13, 2019

@jnothman verbose=1 outputs

[Parallel(n_jobs=-1)]: Using backend LokyBackend with 8 concurrent workers.
[Parallel(n_jobs=-1)]: Done  30 out of  30 | elapsed: 46.5min finished

with nothing in the middle

@AlJohri AlJohri mentioned this issue Feb 4, 2019
Closed
@AlJohri
Copy link
Author
AlJohri commented Feb 4, 2019

@jnothman I filed a PR: #13091

@amueller
Copy link
Member

closing as no reply in the PR. I don't think it's worth doing. Feel free to reply /reopen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

increase verbosity threshold for cross validation
3 participants
@jnothman @amueller @AlJohri
0