Hi @mheberger,
Before I open another PR, I wanted to propose a change: What to you think of replacing the print statements with a logging.Logger?
The VERBOSE setting could then be used to pre-configure the level of the logger, so if the user sets VERBOSE to false nothing would be printed. The huge advantage would be that the output could be more fine-grained using debug, info and warning log messages. Secondly, and for me more importantly, I could easily replace the default logger with another instance that formats log messages as I need them.
I would be willing to do the implementation, but wanted to make sure that you are interested and like it, because it's more work than my last PR.
Best,
Mirko
Hi @mheberger,
Before I open another PR, I wanted to propose a change: What to you think of replacing the
printstatements with alogging.Logger?The
VERBOSEsetting could then be used to pre-configure the level of the logger, so if the user sets VERBOSE to false nothing would be printed. The huge advantage would be that the output could be more fine-grained using debug, info and warning log messages. Secondly, and for me more importantly, I could easily replace the default logger with another instance that formats log messages as I need them.I would be willing to do the implementation, but wanted to make sure that you are interested and like it, because it's more work than my last PR.
Best,
Mirko