Skip to content

IPH models pull request#1900

Merged
sjanzou merged 140 commits into
developfrom
iph_fin_cmods
Nov 21, 2024
Merged

IPH models pull request#1900
sjanzou merged 140 commits into
developfrom
iph_fin_cmods

Conversation

@sjanzou

@sjanzou sjanzou commented Nov 7, 2024

Copy link
Copy Markdown
Collaborator

Please test and resolve issues to merge into for release here!
Goes with NatLabRockies/ssc#1238

tyneises added 30 commits September 29, 2023 11:05
update default costs for mslf iph
move financial model variable definition to callbacks to set on load
@cpaulgilman

Copy link
Copy Markdown
Collaborator

@taylorbrown75 For IPH configurations with the Single Owner financial model, I think the "Electricty Rates" page name in the navigation menu should be "Electricity Purchases" to be consitent with front-of-meter CSP and other configurations where electricity purchases are an operating expense. This is to distinguish from behind-the-meter configurations where "electricity rates" are used to calculate electricity bill savings. (You can see that there are separate Help topics for "Electricity Rates" and "Electricity Purchases" so we have separate places to explain how those inputs work.)

@taylorbrown75

Copy link
Copy Markdown
Collaborator

@taylorbrown75 For IPH configurations with the Single Owner financial model, I think the "Electricty Rates" page name in the navigation menu should be "Electricity Purchases" to be consitent with front-of-meter CSP and other configurations where electricity purchases are an operating expense. This is to distinguish from behind-the-meter configurations where "electricity rates" are used to calculate electricity bill savings. (You can see that there are separate Help topics for "Electricity Rates" and "Electricity Purchases" so we have separate places to explain how those inputs work.)

Fixed with (c06063b)

@sjanzou sjanzou merged commit 1d506fb into develop Nov 21, 2024
@cpaulgilman cpaulgilman added enhancement added to release notes PR and/or issue has been added to release notes for a public release labels Dec 10, 2024
@cpaulgilman cpaulgilman added this to the SAM Fall 2024 Release milestone Dec 10, 2024
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 enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants