search

OpenSTEF / openstef-dbc

Provides (company specific) database connector for the OpenSTEF package
Mozilla Public License 2.0
1 stars 7 forks source link

Update data_interface.py #127

Closed FrankKr closed 6 months ago

FrankKr commented 6 months ago

Should fix: image

Replacing None with empty dict is not needed and causes issues, pd.read_sql can deal with params=None.

I did not test if this actually resolves the issue. Maybe we can make a pre-release, include that in a fix-branch of icarus-forecasts and see if that resolves the issue?

Update; the pre-release works: image Bumping it to a proper version an releasing.

bartpleiter commented 6 months ago

I also do not know if this will fix the issue. However, your suggstion to test using a pre-release is to me a very good idea!