Skip to content

839 forecast for generic battery#992

Merged
brtietz merged 7 commits into
patchfrom
839_forecast_for_generic_battery
Mar 31, 2022
Merged

839 forecast for generic battery#992
brtietz merged 7 commits into
patchfrom
839_forecast_for_generic_battery

Conversation

@brtietz

@brtietz brtietz commented Mar 29, 2022

Copy link
Copy Markdown
Collaborator

Pull Request Template

Description

Add a data array for a custom production forecast for generic system. This array is run through generic system as a custom profile to apply availability losses and degradation. Update the lk scrips to accommodate this change in function. Will be demoed at the SAM meeting today.

Fixes # (839)

Type of change

  • [x ] New feature (non-breaking change which adds functionality)

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
  • [ x] 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

@brtietz brtietz requested a review from cpaulgilman March 29, 2022 14:51
@brtietz brtietz added this to the 2021.12.02 Patch 2 milestone Mar 29, 2022
@brtietz brtietz linked an issue Mar 29, 2022 that may be closed by this pull request
@brtietz brtietz merged commit 214470e into patch Mar 31, 2022
@brtietz brtietz deleted the 839_forecast_for_generic_battery branch April 29, 2022 16:04
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Jun 6, 2022
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 battery enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow custom forecasts for generic system

2 participants