-
Notifications
You must be signed in to change notification settings - Fork 2
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
fix: replace capacity limit values for existing generators with '.' #82
Conversation
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 followed the conversation on Slack and it does exactly what we intend to.
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.
if it runs, I am happy
It runs... sort of:
We do have the results.pickle file in the expected location, and it is 132 MB (i.e. not empty), so maybe we have everything we need... |
Pull Request doc
Purpose
Avoid generating an infeasible model
What the code is doing
We change the code so that entries in the
gen_capacity_limit_mw
column of the generation_projects_info.csv file for existing generators are filled with"."
, rather than the default capacity limits from const.py. This avoids generating an infeasible problem, I think because we do not constrain the existing coal plants to be zero capacity, which conflicts with their non-zero capacities from the gen_build_predetermined.csv file. The original issue for generating this file (#1) was ambiguous as to what should be in these entries, and we made a mistake in porting the logic from the original script in #35.Testing
Tested manually. Before: the solver returns Infeasible in 1-2 minutes. After: the solver keeps running much longer than that.
Time estimate
5-10 minutes.