-
Notifications
You must be signed in to change notification settings - Fork 82
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
log dataset and model in new cli #76
Comments
Do you mean just printing out model/dataset/progress percent after each one? |
It looks like the dataset, model, and language (but not progress percent) are already printed out if you use |
opinion on removing verbose option and making it the default ? |
I don't have a strong opinion one way or the other, but if we wanted to make that change, we could replace parser_eval.add_argument('--verbose', default=False, action="store_true", help="verbose mode") with parser_eval.add_argument('--quiet', dest='verbose', action="store_false", help="suppress verbose messages") to just flip the default verbosity (and also rename the cli option name correspondingly). |
yeah #78 |
run.sh had that and it was nice to know progress
The text was updated successfully, but these errors were encountered: