Skip to content

Cashflow send excel fixes#936

Merged
cpaulgilman merged 11 commits into
patchfrom
cashflow-send-excel-fixes
Feb 17, 2022
Merged

Cashflow send excel fixes#936
cpaulgilman merged 11 commits into
patchfrom
cashflow-send-excel-fixes

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Fixes #880, #694

O&M Costs

For fuel cell model, disable battery O&M cost inputs when battery not enabled, and display a UI message if those inputs are set to non-zero by a script or opening an old file. This is not enforced in SSC, so SDK users could model battery O&M costs for a system with no batteries.

Set default fuel cell O&M cost by capacity and replacement cost to zero so that battery O&M cost is not included in operating expenses when battery is disabled. This changes the default LCOE for Fuel Cell configurations, so requires running test script.

Fix variable O&M for FOM and BTM with battery configurations.

Cash flow

Fix fuel cell fuel in cash flow: Export SAM cash flow instead of calculating in Excel from heat rate inputs, which works for CSP and other fuel techs, but not for fuel cell.

Show LCOS calculations in cash flow for all battery configs, not just "has_roe()" configs.

Remove has_roe flag from cash flow script because we removed ROE metric in #549

Fix LCOS cash flow equations in Excel.

Fix land lease send to excel for techs with no land area.

Add support for sculpted debt with debt size limit.

Energy Line Items

Add cf_energy_sales to outputs for all financial models.

Change labels on Results page Summary and Losses tab from "energy" to "electricity."

Other

Update urdb_view_rate URL to https://apps.openei.org/IURDB, also fixed in sam-private https://github.com/NREL/SAM-private/commit/af6e99209ef7877496c00a1a39cb786a752b5ab2

UI labels and group names for om_opt_fuel_xxx inputs on Biopower feedstock costs form.

UI group name for Fuel Cell Costs UI form.

Set total_land_area for CSP models for financial model inputs:

  • Empirical Trough, replace ui_total_land_area with total_land_area

  • Tower Solar Field, Linear Fresnel Solar Field, Generic CSP Solar Field, LF DSG Solar Field: Add total_land_area input and UI equation to store value of csp.pt.sf.total_land_area

  • Add total_land_area default for MSPT, DSLF, Generic CSP System, DSGL IPH

@cpaulgilman cpaulgilman added this to the 2021.12.02 Patch 1 milestone Feb 16, 2022
@cpaulgilman cpaulgilman requested a review from sjanzou February 16, 2022 00:36
@cpaulgilman cpaulgilman self-assigned this Feb 16, 2022
@cpaulgilman cpaulgilman merged commit 7ef49b0 into patch Feb 17, 2022
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Feb 21, 2022
@cpaulgilman cpaulgilman deleted the cashflow-send-excel-fixes branch October 20, 2022 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes PR and/or issue has been added to release notes for a public release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Battery operating costs missing from cash flow table

1 participant