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

Startup/shutdown curve #219

Merged
merged 20 commits into from
Apr 22, 2021

Conversation

bknueven
Copy link
Collaborator

@bknueven bknueven commented Apr 6, 2021

Adds the capability to specify start-up and shut-down power curves for thermal generators within the unit commitment models, which specify power output just before turning on and just after turning off. This is especially important for models with short (e.g., 5-minute) time-horizons.

Curves are specified through the startup_curve and shutdown_curve attribute on a thermal generator. These represent, from highest to lowest, the power output of the generator in the time periods just before / just after starting-up or shutting-down. Also adds a future_status parameter for start-ups after the end of the time horizon. An example is provided in the test tiny_uc_11.json.

Also makes some significant changes to the scaling/unscaling functions. Resolves #221 by adding some basic tests for baseMVA scaling/unscaling.

@bknueven bknueven marked this pull request as ready for review April 7, 2021 00:04
@bknueven bknueven requested a review from jeanpaulwatson April 7, 2021 00:04
Copy link
Collaborator

@jeanpaulwatson jeanpaulwatson left a comment

Choose a reason for hiding this comment

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

Nothing really required associated with my comments - all just suggestions for clarity improvement.

egret/model_library/unit_commitment/params.py Show resolved Hide resolved
egret/model_library/unit_commitment/power_vars.py Outdated Show resolved Hide resolved
egret/model_library/unit_commitment/params.py Show resolved Hide resolved
egret/model_library/unit_commitment/power_vars.py Outdated Show resolved Hide resolved
egret/model_library/unit_commitment/power_vars.py Outdated Show resolved Hide resolved
@bknueven bknueven requested a review from jeanpaulwatson April 7, 2021 14:49
@bknueven bknueven marked this pull request as draft April 7, 2021 17:07
@bknueven
Copy link
Collaborator Author

bknueven commented Apr 7, 2021

There's some weirdness in scaling/unscaling I'd like to resolve before this is merged

@bknueven bknueven force-pushed the startup_shutdown_curve branch from c4bb63f to f4ea927 Compare April 19, 2021 18:57
@bknueven bknueven marked this pull request as ready for review April 19, 2021 20:30
@bknueven bknueven requested a review from carldlaird April 19, 2021 20:30
@bknueven bknueven requested a review from jeanpaulwatson April 19, 2021 20:30
@bknueven bknueven merged commit 45e96fe into grid-parity-exchange:main Apr 22, 2021
@bknueven bknueven deleted the startup_shutdown_curve branch April 22, 2021 16:17
@bknueven bknueven mentioned this pull request Sep 14, 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.

Testing for BaseMVA scaling
2 participants