Skip to content

Increase pvwatts-battery o&m to account for battery o&m#1326

Merged
brtietz merged 1 commit into
patchfrom
sam_1298_update_pvwatts_om
Jan 26, 2023
Merged

Increase pvwatts-battery o&m to account for battery o&m#1326
brtietz merged 1 commit into
patchfrom
sam_1298_update_pvwatts_om

Conversation

@brtietz

@brtietz brtietz commented Jan 24, 2023

Copy link
Copy Markdown
Collaborator

Pull Request Template

Description

Add battery O&M from detailed cases to PVWatts-Battery single-line O&M number. Since detailed number is $/kWh, multiply by number of hours of storage to convert to $/kW.

Fixes #1298

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)

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.

  • My code follows the style guidelines of this project
  • 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
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings

@brtietz brtietz added this to the 2022.11.21 Patch 1 milestone Jan 24, 2023
@brtietz brtietz requested a review from cpaulgilman January 24, 2023 18:50
@brtietz brtietz self-assigned this Jan 24, 2023

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

Looks good.

@brtietz brtietz merged commit 2e441ae into patch Jan 26, 2023
@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, 2023
@brtietz brtietz deleted the sam_1298_update_pvwatts_om branch December 13, 2024 19:22
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.

Update PVWatts-Battery O&M numbers to account for battery

2 participants