Skip to content

745 zero load during grid outage#741

Merged
brtietz merged 10 commits into
patchfrom
745_zero_load_during_grid_outage
Feb 11, 2022
Merged

745 zero load during grid outage#741
brtietz merged 10 commits into
patchfrom
745_zero_load_during_grid_outage

Conversation

@brtietz

@brtietz brtietz commented Jan 28, 2022

Copy link
Copy Markdown
Collaborator
  • cmod_grid now sets "load" to zero during grid outage steps
  • To report inputs in the data tables, full_load now has the load without the outage steps
  • Fixes a bug with the shared inverter losses in the dc connected systems
  • Increases the tolerance for when a system is considered offline to cover edge cases with degradation

This means if the outage is a full year, the utility bill without system and the utility bill with system will just consist of fixed costs. This is a big change, so I'm happy to wait to merge the PR until we've had a chance to talk through the downstream effects at a meeting.

Closes NatLabRockies/SAM#745

Brian Mirletz added 3 commits January 27, 2022 16:39
… intended, but some annoying side effects from losses
…error where DC connected battery was reporting incorrect loss values
@brtietz brtietz added this to the 2021.12.02 Patch 1 milestone Jan 28, 2022
@sjanzou

sjanzou commented Jan 29, 2022

Copy link
Copy Markdown
Collaborator

Hmmm... maybe further discussion is necessary. In using the attached project file,
test_file.zip

the dc connected electricity bill savings are:
image

and ac connected (same configuration) are: (varying seems okay based on differences in annual energy and battery sizing - however, I would not like to explain the differences!)
image

@brtietz

brtietz commented Jan 31, 2022

Copy link
Copy Markdown
Collaborator Author

@sjanzou Ah, thanks for testing those additional cases. There were some situations where unmet losses were slipping through and pushing negative numbers into gen - which counted as small demand charges. I've cleaned that up in the latest push, and now see zero for all "electricity bill savings" locally.

@sjanzou sjanzou left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good - should probably confirm this is the desired behavior at the Puerto Rico meeting...
image

@brtietz

brtietz commented Feb 8, 2022

Copy link
Copy Markdown
Collaborator Author

Grid outage added as an hourly output, as per discussion at the SAM team meeting.

@cpaulgilman Does the label still make sense given that it's an output?

image

Perhaps it should be "grid outage in this timestep"?

@cpaulgilman

Copy link
Copy Markdown
Collaborator

"Grid outage in this time step" is good.

@brtietz brtietz merged commit 254b0ca into patch Feb 11, 2022
@brtietz brtietz deleted the 745_zero_load_during_grid_outage branch April 29, 2022 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ensure that the load passed to the utility bill calculations align with the grid outage

3 participants