Replace logger with print

[FrickTobias]

See #19

In short, this

• replaces logger.info() with print() and logger.warning() with print("WARNING: ")
• keeps logger.info()/logger.warning() only for log_file writing

[FrickTobias]

Thanks for the feedback, I changed it according to your suggestions, except one case which I thought merited discussion.

In case of a warning, does it make sense to regulate it with the same argument as iteration prints? Maybe adding a silent mode for suppressing warnings (default False)? Or am I just overthinking this?

[fillipe-gsm]

In case of a warning, does it make sense to regulate it with the same argument as iteration prints? Maybe adding a silent mode for suppressing warnings (default False)? Or am I just overthinking this?

That is actually a good question. But in my opinion the gain in flexibility could overcomplicate what should be simple. I think we can consider all prints to be the same level, so we would have the rules:

• If there is a log file, print everything to the file;
• If verbose is True, print everything to the stdout. If False, print nothing, even the warnings.

What do you think?

[FrickTobias]

I agree. Simplicity is probably better here.

[FrickTobias]

Thank you again for sharing your code and actively maintaining it!

I think you'll have to merge though since I don't have write access to the repo!

Screen shot =>

[fillipe-gsm]

Thank you again for sharing your code and actively maintaining it!

I think you'll have to merge though since I don't have write access to the repo!

Screen shot =>

That's odd; I was sure an approval from an admin would be enough :confused: I'll look into that later. For now, I merged the PR.

Thanks again for the work!