Skip to content

Fix cash flow expense in cash flow for solar BTM configurations#909

Merged
cpaulgilman merged 1 commit into
patchfrom
btm_land_lease_error
Jan 31, 2022
Merged

Fix cash flow expense in cash flow for solar BTM configurations#909
cpaulgilman merged 1 commit into
patchfrom
btm_land_lease_error

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

Description

Only display land lease expense cash flow line item for FOM solar configurations.

Fixes #890

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.

  • [x ] My code follows the style guidelines of this project
  • [x ] I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • [x ] My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • [ x] New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • [ x] I have checked my code and corrected any misspellings

@cpaulgilman cpaulgilman added this to the 2021.12.02 Patch 1 milestone Jan 31, 2022
@cpaulgilman cpaulgilman self-assigned this Jan 31, 2022
@cpaulgilman cpaulgilman merged commit b81193d into patch Jan 31, 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 btm_land_lease_error 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.

Land lease variable in cash flowfor PVWatts Commercial case

2 participants