-
Notifications
You must be signed in to change notification settings - Fork 54
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
Myopic fix #217
Myopic fix #217
Conversation
…ty grid connection costs as well
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.
Thanks @lisazeyen. Looks mostly good to me. Had a few style ideas.
Co-authored-by: Fabian Neumann <fabian.neumann@outlook.de>
Co-authored-by: Fabian Neumann <fabian.neumann@outlook.de>
This PR should resolve the problem with e.g. the biomass/biogas potential described in issue PyPSA/pypsa-eur#504 and closes PR #231 |
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.
Great! It's much clearer now. I only had a few style suggestions.
Hi Fabian, I have included you code style suggestions. Do you think the way the gas network is treated now here is better than before? I initially added the first part to avoid duplicating the capacity of the existing gas network for each investment period. I think with the retrofitting of the gas pipes it does make sense. Without the option to retrofit the gas pipelines their capacity can be expanded further than the current capacity. I guess it is better to keep the gas pipes from the previous investment period with their optimized capacity and with |
The latest commits include a simplification of the retrofitting gas pipe constraint and sets the minimum capacity of AC and DC lines to the optimised capacity of the previous year. This should close PyPSA/pypsa-eur#501 |
this PR should resolve some conflicts with the latest changes if running the model with myopic foresight, namely:
define_spatial
function also for other conventional carriers