-
Notifications
You must be signed in to change notification settings - Fork 86
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
Shorten results output in docs #3328
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3328 +/- ##
=======================================
- Coverage 99.7% 99.0% -0.6%
=======================================
Files 327 327
Lines 31840 31840
=======================================
- Hits 31715 31494 -221
- Misses 125 346 +221
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bchen1116 ! This looks good to me. If we decide we need a pprint call in more places in the docs we can invest in trying to find a way to apply the pretty-printing automatically.
@@ -657,7 +657,10 @@ | |||
"metadata": {}, | |||
"outputs": [], | |||
"source": [ | |||
"automl.results" | |||
"import pprint\n", | |||
"pp = pprint.PrettyPrinter(indent=0, width=100, depth=3, compact=True, sort_dicts=False)\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could we set depth=2
and then showcase just one pipelines results with:
automl.results['pipeline_results'][0]
? I still think the output is rather long and takes multiple scrolls to get through.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah true that doesn't look the best.. lmk if adding more information will make it better but def not blocking!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bchen1116! Agreed with @freddyaboulton that it could be a good idea to do this automagically if we run into it more in the future but this is great 😁
fix #3032
Rather than editing the output of the results themselves, I figured the better approach would be to truncate the cell output for this.
https://feature-labs-inc-evalml--3328.com.readthedocs.build/en/3328/user_guide/automl.html#Access-raw-results