-
Notifications
You must be signed in to change notification settings - Fork 45
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
Bug Fix Load Following Dispatch Strategy #341
Conversation
HOPP v2.2.0
…to calculate lifetime schedule used for dispatch
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.
Looks good, just might want to touch base on what we want the default of this parameter to be
hopp/simulation/technologies/grid.py
Outdated
zip(lifetime_schedule, self.generation_profile)]) | ||
self.missed_load_percentage = sum(self.missed_load)/sum(lifetime_schedule) | ||
|
||
if self.site.curtailment_value_type == "grid": |
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.
why is this type called "grid"?
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.
Let's change this to interconnect_kw
self.missed_load_percentage = sum(self.missed_load)/sum(lifetime_schedule) | ||
|
||
if self.site.curtailment_value_type == "grid": | ||
lifetime_schedule: NDArrayFloat = np.tile([self.interconnect_kw], |
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.
Why call this "lifetime_schedule"? Isn't this the maximum allowed generation?
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 guess it is legacy
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 for the correction. I think this is much more clear.
self.missed_load_percentage = sum(self.missed_load)/sum(lifetime_schedule) | ||
|
||
if self.site.curtailment_value_type == "grid": | ||
lifetime_schedule: NDArrayFloat = np.tile([self.interconnect_kw], |
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 for the correction. I think this is much more clear.
Bug Fix Load Following Dispatch Strategy
grid.py
to calculate missed load, missed load percentage and load met using thedesired_schedule
even when curtailing at grid interconnection limit.04-load-following-battery
to usecurtailement_value_type = "grid"
and updated explanation and outputs.Related issue
Impacted areas of the software
Additional supporting information
Test results, if applicable
test_hybrid.py
andtest_detailed_pv_plant.py
updated with cherry-picked commit 7cddc6a for CI integration tests.