search

oemof / oemof-solph

A model generator for energy system modelling and optimisation (LP/MILP).
https://oemof.org
MIT License
297 stars 126 forks source link

Adjust indexing for content of InvestStorageBlock #1039

Closed p-snft closed 1 month ago

p-snft commented 8 months ago

Fix #1030

The file has a lot of duplication and should be reworked. I try to provide a quick fix without changing too much but I failed once already.

Additional todos I think that are needed because of my experience in #1038.

pep8speaks commented 1 month ago

Hello @p-snft! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2024-08-14 11:10:55 UTC
p-snft commented 1 month ago

I pulled this into #1080 to fix it without having to rewrite lp files.