Skip to content

Add land lease calculations to fix #719#779

Merged
cpaulgilman merged 7 commits into
developfrom
sam-719-land-lease
Nov 18, 2021
Merged

Add land lease calculations to fix #719#779
cpaulgilman merged 7 commits into
developfrom
sam-719-land-lease

Conversation

@cpaulgilman

@cpaulgilman cpaulgilman commented Nov 17, 2021

Copy link
Copy Markdown
Collaborator

Goes with NatLabRockies/ssc#663
Fixes #719

Add Operating Costs Land Lease UI form to configs for front-of-meter financial models

  • PVWatts (plus Community Solar)
  • Flat Plate PV
  • CSP
  • Fuel Cell
  • PV Battery

Get land area value from CSP solar field pages.

Update PV Land Area UI form to work for both PVWatts and Flat Plate PV.

Add land lease to cashflow.lk.

This LK script tests land lease calculations: land-lease-test-script.zip

Add operating costs land lease UI form to solar FOM configs

Add land lease to cash flow (need to update send-to-Excel with equations)

Get land area value for all solar techs (PV, CSP, HCPV)

Adjust PVWatts forms to have same width as PV so PV Land Area works for both PV and PVWatts

@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.

ran ssc tests SAM defaults and land lease test scripts with results in attached word document. Everything working..
GitHubPullRequestsSAM_779.docx
.

@janinefreeman janinefreeman removed their request for review November 18, 2021 19:30
@cpaulgilman cpaulgilman merged commit 64895f0 into develop Nov 18, 2021
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Dec 2, 2021
@cpaulgilman cpaulgilman deleted the sam-719-land-lease branch December 7, 2021 23:03
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.

Add land lease option from community solar to other financial models

2 participants