NNPDF / pineappl

PineAPPL is not an extension of APPLgrid
https://nnpdf.github.io/pineappl/
GNU General Public License v3.0
12 stars 3 forks source link

Add a logging system that records and prints warnings #254

Closed cschwan closed 1 month ago

cschwan commented 6 months ago

Related Issues:

alecandido commented 6 months ago

Not exactly logging, but you could consider tracing (tracing is a bit more than logging, but it might be useful in PineAPPL, e.g. to debug and profile new interfaces)

cschwan commented 1 month ago

In the meantime I think warnings aren't a good idea anymore. An operation should either succeed or fail.

alecandido commented 1 month ago

I arrived more or less to the same conclusion: warnings are mostly cluttering the output and ignored. Instead, if you want to pass some further information down to your user, you should add that to the function output, not printing it on stdout.