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

Add multi period deserialization #112

Merged
merged 7 commits into from Jul 24, 2023
Merged

Add multi period deserialization #112

merged 7 commits into from Jul 24, 2023

Conversation

nailend
Copy link
Collaborator

@nailend nailend commented Jul 13, 2023 •
edited

Close #111

  • Needs additional facade parameters from Multi period investment #108
  • Read periods and timeincrement from datapackage
  • Add multi-period dispatch example
  • Adjust infer script
  • Add multi-period investment example
  • Add age and lifetime paramaters

Multi-period parameters will be adressed in #115

@nailend
Read periods and timeincrement from datapackage
8f4a85c 
The multi-period approach is triggered with the creation of the
instanziation of the energy-system. Periods and timeincrement
information, need to be read from datapackage and passed. We
add an additional ressource `periods` to `datapackage.json`
and directory.
@nailend
Add multi-period example
c504f83
@nailend nailend self-assigned this Jul 13, 2023
@nailend nailend linked an issue Jul 13, 2023 that may be closed by this pull request
Add multi-period to deserialization #111
Closed
@nailend nailend requested a review from henhuy July 24, 2023 12:20
henhuy
henhuy approved these changes Jul 24, 2023
View reviewed changes
Copy link
Collaborator

@henhuy henhuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you!

@nailend nailend merged commit 8d26a48 into dev Jul 24, 2023
3 checks passed
@nailend nailend mentioned this pull request Jul 26, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@henhuy henhuy henhuy approved these changes

Assignees

@nailend nailend

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add multi-period to deserialization
2 participants
@nailend @henhuy