-
Notifications
You must be signed in to change notification settings - Fork 69
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
Add learning curve plotting command #396
Conversation
Thank God! This is now ready for review :) |
#!/usr/bin/env python | ||
# License: BSD 3 clause | ||
""" | ||
Little helper script to generate learning plots from the learning curve output TSV file. |
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.
Is it worth adding a sentence about why this would be necessary?
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.
Good call. I will do that.
if not exists(args.output_dir): | ||
makedirs(args.output_dir) | ||
|
||
# check that we have pandas and seaborn available other |
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.
Is this supposed to say "otherwise throw an error" or something along those lines?
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.
I was just supposed to stop at "available" but forgot to delete the last word :) I will fix it in the next PR. Thanks!
plot_learning_curves
that generates learning curve PNG files from learning curve summary TSV files.