pySIPFENN icon indicating copy to clipboard operation
pySIPFENN copied to clipboard

Suggestion: `Calculator(verbose=False)` should not print (or tqdm) any output

Open bocklund opened this issue 1 year ago • 2 comments
trafficstars

The verbose output is helpful to see what's going on and get realtime feedback, but sometimes it's hard to find my output among all of pysipfenn's output. I'd like to be able to hide all the output (at least on the happy path, if stuff is breaking or not right, then being noisy is the right thing to do IMO).

bocklund avatar Aug 13 '24 22:08 bocklund

Thanks for the feedback @bocklund ! As always, it is very much appreciated!

The idea behind this switch was to make printouts verbose vs. concise rather than on vs. off, but you are right that a no-printout option would be nice to have. I was actually thinking about it like two months ago - whether or not to turn it into a "verbosityLevel" switch with Flase, 0, 1, ... options.

Another option I like, especially in the context of debugging, would be to have both verbose and printout switches, where Calculator with the default printout=True acts just like now, while printout=False will direct messages, verbose or concise, into an internal logger variable that could be accessed when needed. What are your thoughts on that?

amkrajewski avatar Aug 22 '24 22:08 amkrajewski

I created a new branch, improve_CalculatorPrintout, for this purpose.

amkrajewski avatar Aug 23 '24 19:08 amkrajewski