Improve error reporting in PineAPPL

NNPDF / pineappl

PineAPPL is not an extension of APPLgrid

https://nnpdf.github.io/pineappl/

GNU General Public License v3.0

13 stars 3 forks source link

Open cschwan opened 2 years ago

cschwan commented 2 years ago

The quality of error reporting in PineAPPL ranges from very terrible, see for instance https://github.com/N3PDF/pineappl/issues/104#issuecomment-1016658544, to OK-ish.

TODO list:

[x] #198
[ ] https://github.com/N3PDF/pineappl/issues/104#issuecomment-1016658544 is an instance of a completely opaque error
[ ] in general, we should merge most error types into a single one and return it instead of panicing
[ ] use Backtrace?

cschwan commented 2 years ago

A related issue is #92, in which case a None is unwrapped.

cschwan commented 2 years ago

One instance is given in https://github.com/N3PDF/pineappl/issues/101#issuecomment-1012252389, where instead of a panic an error should be returned instead. That could be done at the very start and only once.