Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Decouple the investment of storage flows and capacity - Part 2 #491

Merged
merged 22 commits into from
May 24, 2018
Merged

Decouple the investment of storage flows and capacity - Part 2 #491

merged 22 commits into from
May 24, 2018

Conversation

uvchik
Copy link
Member

@uvchik uvchik commented May 16, 2018 •
edited
Loading

FabianTU and others added 13 commits May 1, 2018 11:57
@uvchik uvchik added this to the v0.2.2 milestone May 16, 2018
@uvchik uvchik requested a review from simnh May 17, 2018 08:57
@uvchik uvchik self-assigned this May 17, 2018
@uvchik
Copy link
Member Author

uvchik commented May 17, 2018

Okay, I finished this feature but removed the non-cycled option as it did not work. I will create a new PR for this feature after this PR is merged.

@uvchik uvchik mentioned this pull request May 17, 2018
Merged
@uvchik uvchik merged commit a9cc262 into dev May 24, 2018
@uvchik uvchik deleted the features/decouple_flows_capacity_investment_storage branch May 24, 2018 09:47
@uvchik uvchik mentioned this pull request May 24, 2018
Merged
6 tasks
p-snft added a commit that referenced this pull request Feb 19, 2019
@p-snft
Adjust InvestmentStorage unit test for (un)balanced storage
6df4719 
lp file contains new variables due to feature request in #491.
A check for correctly considered n.investment.minimum that was
added in parallel (#561) did not have these, yet.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@simnh simnh simnh approved these changes

Assignees

@uvchik uvchik

Labels
None yet
Projects
None yet
Milestone
v0.2.2
Development

Successfully merging this pull request may close these issues.
3 participants
@uvchik @simnh @FabianTU