-
Notifications
You must be signed in to change notification settings - Fork 21
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
Limit battery storage duration #332
Conversation
6073760
to
d37dafb
Compare
d37dafb
to
adcfdc6
Compare
@atpham88 I changed the new input name from Could you:
|
otherwise, if min is > max, size will just be set to zero
@atpham88 I pushed a few minor changes. Before I approve, could you please:
|
…opt.jl into limit_battery_energy_cap
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.
@atpham88 Looks good! Feel free to merge into develop
Summary
This PR adds user-input option to limit the hour duration for battery storage.
Implementation notes
Changes in this PR to add this option:
1. electric_storage.jl: added a
max_duration_hours
input, default as a big number - 100,000 (hours), andmin_duration_hours
, default is 0 (hour). This defaults can be changed in["ElectricStorage"]["max_duration_hours"]
and["ElectricStorage"]["min_duration_hours"]
.2. storage_constraints.jl: Add a constraint to limit battery energy capacity <= battery power rating * max_duration_hours and battery energy capacity >= battery power rating * min_duration_hours.
3. runtests.jl: Add "Storage Duration" test.