Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update default utility rates for hybrids and test script #1958

Merged
merged 4 commits into from
Dec 9, 2024

Conversation

brtietz
Copy link
Collaborator

@brtietz brtietz commented Dec 6, 2024

Pairs with SSC PR: NREL/ssc#1258

Fixes #1949

Test script updates reflect ssc changes.

…r (demand charges). Re-run test script for ssc changes (hybrids and me wave battery
@brtietz brtietz added the dev bug Internal bug that should not be included in release notes label Dec 6, 2024
@brtietz brtietz added this to the SAM Fall 2024 Release milestone Dec 6, 2024
@brtietz brtietz requested a review from mjprilliman December 6, 2024 21:05
@brtietz
Copy link
Collaborator Author

brtietz commented Dec 9, 2024

Test failures appear to be due to branch naming. Testing again with merge from develop and NREL/ssc#1258 merged in on the SSC side.

@brtietz brtietz merged commit 26e1192 into develop Dec 9, 2024
8 checks passed
@cpaulgilman cpaulgilman added the added to release notes PR and/or issue has been added to release notes for a public release label Dec 10, 2024
@brtietz brtietz deleted the sam_1949_ssc_1256 branch December 13, 2024 19:21
@cpaulgilman cpaulgilman removed the added to release notes PR and/or issue has been added to release notes for a public release label Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev bug Internal bug that should not be included in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Monthly Utility Bill summary chart doesn't work with hybrids
3 participants