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.

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: when scaling gen Pmax, also scale Pmin and cost curve #16

Merged
merged 3 commits into from
Aug 5, 2019

Conversation

ghost
Copy link

@ghost ghost commented Jul 24, 2019

A fix for scaling thermal generation properly. Since we are scaling up and down capacity of existing units rather than adding/subtracting units, we should also be scaling other parameters besides P_max so that the overall generator profile is consistent. I.e. when scaling, P_min:P_max ratio remains the same, marginal cost at P_min is the same, and marginal cost at P_max is the same.

@ghost ghost requested review from kasparm and rouille July 24, 2019 21:48
Copy link
Collaborator

@rouille rouille left a comment

Choose a reason for hiding this comment

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

This makes sense.

@kasparm
Copy link
Contributor

kasparm commented Jul 31, 2019

@danielolsenIV we need to discuss this change in person. This afternoon?

@ghost
Copy link
Author

ghost commented Jul 31, 2019

Scaling cost coefficients has been fixed to be in the right place.

I also added a couple of lines to avoid a potential divide-by-zero error (for example, if we want to shut down all coal plants in Washington state).

@kasparm kasparm force-pushed the gen_scaling_improvement branch from e10544a to 557ea2f Compare August 2, 2019 18:15
@kasparm kasparm force-pushed the gen_scaling_improvement branch from 557ea2f to b1f8970 Compare August 2, 2019 22:25
@kasparm kasparm merged commit b1f8970 into develop Aug 5, 2019
@danielolsen danielolsen deleted the gen_scaling_improvement branch February 26, 2020 00:49
@ahurli ahurli mentioned this pull request Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants