Skip to content

Send-to-excel Fixes#2169

Merged
cpaulgilman merged 1 commit intodevelopfrom
sam-2108-send-excel-spreadsheet
Apr 13, 2026
Merged

Send-to-excel Fixes#2169
cpaulgilman merged 1 commit intodevelopfrom
sam-2108-send-excel-spreadsheet

Conversation

@cpaulgilman
Copy link
Copy Markdown
Collaborator

Description

Remove data validation in Inputs!C34.

Multiply major equipment reserve account by 1000 W/kW in Inputs!D96, D97, D98. Also fix in host developer, merchant plant, partnership flip, sale leaseback, and single owner spreadsheets. Note this only affects value on Inputs sheet. Cash flow value is correctly calculated on Cash Flow sheet.

Fix unsubscribed share Inputs!C37: "=100-SUM(subscriber1_share,subscriber2_share,subscriber3_share,subscriber4_share)".

Fixes #2108

Corresponding branches and PRs:

None

Unit Test Impact:

None

Checklist

  • requires help revision and I added that label
  • adds, removes, modifies, or deletes variables in existing compute modules
  • adds a new compute module
  • changes defaults
  • I've tagged this PR to a milestone

Remove data validation in Inputs!C34.

Multiply major equipment reserve account by 1000 W/kW in Inputs!D96, D97, D98. Also fix in host developer, merchant plant, partnership flip, sale leaseback, and single owner spreadsheets. Note this only affects value on Inputs sheet.  Cash flow value is correctly calculated on Cash Flow sheet.

Fix unsubscribed share Inputs!C37: "=100-SUM(subscriber1_share,subscriber2_share,subscriber3_share,subscriber4_share)".
@cpaulgilman cpaulgilman added this to the SAM 2026 Release milestone Apr 9, 2026
@cpaulgilman cpaulgilman requested a review from brtietz April 9, 2026 23:27
@cpaulgilman cpaulgilman self-assigned this Apr 9, 2026
@cpaulgilman cpaulgilman added bug UI User interface issue that applies across performance and financial models labels Apr 9, 2026
Copy link
Copy Markdown
Collaborator

@brtietz brtietz left a comment

Choose a reason for hiding this comment

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

Thanks for fixing!

@cpaulgilman cpaulgilman merged commit 59a8997 into develop Apr 13, 2026
6 checks passed
@cpaulgilman cpaulgilman deleted the sam-2108-send-excel-spreadsheet branch April 13, 2026 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug UI User interface issue that applies across performance and financial models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Errors in Community Solar Send to Excel spreadsheet

2 participants