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

Billing demand lookback with TOU "Included in Billing Demand" of zero causes unexpected results #1985

Open
cpaulgilman opened this issue Dec 20, 2024 · 2 comments
Assignees
Labels

Comments

@cpaulgilman
Copy link
Collaborator

Describe the bug

See https://sam.nrel.gov/forum/forum-general/4849-billing-demand-potential-discrepancy.html

billing-demand-lookback-issue.zip

The attached default PV / Commercial is modified to enable billing demand lookback and set "Included in Billing Demand" in the Billing Demand by Time-of-use Period for Demand Charges table to zero:

image

This causes unexpected billing demand with system results.

Consider adding SSC error check to display error when Billing Demand by TOU table has no TOU periods included in billing demand. Suggested message:

"The ur_dc_billing_demand_periods matrix must include at least one TOU demand period from the ur_dc_tou_mat matrix."

Note that this is required even when Charge ($/kW) in the TOU Demand Charge table is zero for all periods.

To Reproduce

  1. Open attached .sam file (see above).
  2. Run simulation.
  3. On Data Tables tab, under "Electricity Rate Data by Year," click Billing demand with system (kW) and note billing demand in Years 2 and later is set to minimum billing demand of 100 kW.

Operating System and Version

Tested in SAM 2024.12.12 and SAM 2023.12.17 r2

Windows 11

@cpaulgilman
Copy link
Collaborator Author

cpaulgilman commented Dec 20, 2024

Help revision for SAM 2024.12.12 r1 with note to explain requirement:

image

@brtietz brtietz added this to the 2024 Release Patch 1 milestone Dec 20, 2024
@brtietz
Copy link
Collaborator

brtietz commented Dec 20, 2024

Help revision for SAM 2024.12.12 r1 with note to explain requirement:

That note text looks great, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants