pommes-public / pommesinvest

A bottom-up fundamental investment model for the German electricity sector
Other
2 stars 0 forks source link

Bug in time series slicing #64

Closed jokochems closed 1 year ago

jokochems commented 1 year ago

Problem description

For hourly data, it has been decided to only read in a slice if not needing the full data set. The routine used for that purpose has an off by one error. This in turn leads the model to fail by simulating years other than the start year 2020.

Suggested Fix

Introduce proper index slicing.