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.

Sign up for GitHub

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

Jump to bottom

New Hybrid Configurations #164

Merged
merged 8 commits into from
Sep 18, 2023
Merged

New Hybrid Configurations #164

merged 8 commits into from
Sep 18, 2023

Conversation

cpaulgilman
Copy link
Collaborator

Defaults and UI for new hybrid configurations for Single Owner and Host Developer financial models:

PVWatts Wind Battery Hybrid

PVWatts Wind FuelCell Battery Hybrid

Photovoltaic Wind Battery Hybrid

Generic PVWatts Wind FuelCell Battery Hybrid

Goes with:

@cpaulgilman cpaulgilman added this to the SAM Fall 2023 Release milestone Sep 14, 2023
This was referenced Sep 14, 2023
Merged
Merged
brtietz
brtietz approved these changes Sep 14, 2023
View reviewed changes
Copy link
Collaborator

@brtietz brtietz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. I'm assuming re-running on the PR test fixed whatever fluke came up in the push test.

@sjanzou sjanzou merged commit d28005f into develop Sep 18, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sjanzou sjanzou sjanzou approved these changes

@brtietz brtietz brtietz approved these changes

@janinefreeman janinefreeman Awaiting requested review from janinefreeman

Assignees

@sjanzou sjanzou

Labels
None yet
Projects
None yet
Milestone
SAM Fall 2023 Release
Development

Successfully merging this pull request may close these issues.
4 participants
@cpaulgilman @brtietz @sjanzou @mjprilliman