oemof / oemof-tabular

Load oemof energy systems from tabular data sources.
https://oemof-tabular.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
7 stars 5 forks source link

Add wrapper to enhance info on failed facade creation #145

Open nailend opened 1 year ago

nailend commented 1 year ago

Description

This wrapper catches errors while creating the facade in the reading module. This way, at least some information on which facade failed and where to look in the data are supplied.

Type of change

Please tick or delete options that are not relevant.

Checklist:

Please tick or delete options that are not relevant.

nailend commented 11 months ago

@henhuy The only way to test this is by providing a false datapackage. I already had this before in #134. Do you think it makes sense to add one to the tests?

henhuy commented 11 months ago

@henhuy The only way to test this is by providing a false datapackage. I already had this before in #134. Do you think it makes sense to add one to the tests?

I think this would be helpful indeed.