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.

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

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

Merged
merged 22 commits into from May 24, 2018

Conversation

Projects
None yet
3 participants
@uvchik
Copy link
Member

uvchik commented May 16, 2018

  • Describe your pull request as transparent as possible

    • Fix pep8
    • Add tests
    • Rename attributes with deprecated warning
      ...
  • Follows PR #480

  • Add tests

  • Adapt tests

  • Non-cycled storage

FabianTU and others added some commits May 1, 2018

Merge pull request #480 from FabianTU/dev
Decouple the investment of storage flows and capacity

@uvchik uvchik added this to the v0.2.2 milestone May 16, 2018

@uvchik uvchik requested a review from simnh May 17, 2018

@uvchik uvchik self-assigned this May 17, 2018

@uvchik

This comment has been minimized.

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.

@simnh

simnh approved these changes May 24, 2018

@uvchik uvchik merged commit a9cc262 into dev May 24, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@uvchik uvchik deleted the features/decouple_flows_capacity_investment_storage branch May 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment