-
Notifications
You must be signed in to change notification settings - Fork 35
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
Verbose flag #53
base: master
Are you sure you want to change the base?
Verbose flag #53
Conversation
I think, I fixed this. Looks a bit weird, though. |
Thanks @syxolk, sorry I haven't had a chance to review yet. I agree the multiprocessing stuff looks a bit weird but on first glance it seems correct. |
@syxolk Apologies for the delay, are you able to rebase this? |
Yep, will do so tomorrow. |
Rebased branch and reformatted code. My biggest problem was the move of the test files to another location but it should work now again. |
Fix #50, where all exceptions were collected in the
failures
dictionary.Added a new option
verbose
toVlad
that enables/disables the use of thefailures
dictionary and_log_debug_failures
.Added new command line option
--verbose
that enables/disables verbose on allVlad
subclasses that are instantiated. Currently this is not implemented for multiprocessing. Can you help me here?