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

Feature/benchmark test for multiple input flows #852

Closed ciaradunks closed 3 years ago

ciaradunks commented 3 years ago

Changes proposed in this pull request:

The following steps were realized, as well (if applies):

Please mark above checkboxes as following:

:x: Check not applicable to this PR

For more information on how to contribute check the CONTRIBUTING.md.

smartie2076 commented 3 years ago

I think that you, by mistake, added the input files for this tests already in this commit (8890878c941c9cc7f0597d04a74095052b3f2309), and merged it to dev

smartie2076 commented 3 years ago

@ciaradunks can you push your updates on the input files prepared for this test? Then we can merge and close the PR.

Remove Fix #507 and #802 in your first post, so that the issues are not automatically closed when the PR is merged.

ciaradunks commented 3 years ago

@smartie2076 I didn't have time in the end to make conclusive inputs, I only got to the point where I was testing out different ways of defining inputs as a list and what may/may not work. So I think this can simply be closed and the issue will be opened again in the future. I have still removed Fix #507 and #802 though.