search

rl-institut / multi-vector-simulator

Multi-vector Simulation Tool assessing and optimizing Local Energy Systems (LES) for the E-LAND project
GNU General Public License v2.0
21 stars 10 forks source link

Raise errors instead of logging.error messages #858

Open smartie2076 opened 3 years ago

smartie2076 commented 3 years ago

In #848 we discussed whether we should rather use logging.Error messages or raise Error. raise Error is the correct way to do it in the MVS, as this also will make sure that the issue is parsed back to the EPA. We should check all logging.Error messages generated within the MVS and make sure they are intentional, or else translate them to raised Errors.

smartie2076 commented 3 years ago

There are approx. 10 uses of logging.error that need to be checked. I already saw that some are really intended to be displayed to the user, and should not result in terminations.