-
Notifications
You must be signed in to change notification settings - Fork 250
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
Multi horizon #637
Multi horizon #637
Conversation
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
@FabianHofmann Some points still can be improved for the multi-decade optimisation, but I would be happy to pull a first beta version into the master branch and then work on the single features in separate smaller PR. |
@FabianHofmann CI should run through once PyPSA/PyPSA#721 is in. There is a problem if the optimisation is multi-decade and the list of extendable lines or DC links is empty when selecting the active assets of an empty list. |
@FabianHofmann, what do you think, can we merge? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would be much in favour to merge it. Here some last remaining comments and the CI should run through. Otherwise, it looks great!
Co-authored-by: Fabian Hofmann <fab.hof@gmx.de>
@FabianHofmann thanks for your feedack! |
Nice, let's go for it! |
follow-up to #637: remove duplicate code and update tsam version
Closes # (if applicable).
Changes proposed in this Pull Request
This PR should include perfect foresight for pathway optimisation.
Open ToDos:
Checklist
envs/environment.yaml
.config.default.yaml
.doc/configtables/*.csv
.doc/release_notes.rst
is added.