Skip to content

Fixes #1027#1028

Merged
sjanzou merged 6 commits into
patchfrom
ssc_1027
May 25, 2023
Merged

Fixes #1027#1028
sjanzou merged 6 commits into
patchfrom
ssc_1027

Conversation

@sjanzou

@sjanzou sjanzou commented May 25, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

@sjanzou sjanzou self-assigned this May 25, 2023
@sjanzou sjanzou requested review from cpaulgilman and tyneises May 25, 2023 06:58
@sjanzou sjanzou linked an issue May 25, 2023 that may be closed by this pull request
@sjanzou sjanzou added this to the 2022.11.21 Patch 2 milestone May 25, 2023
@cpaulgilman

Copy link
Copy Markdown
Collaborator

Fixes NatLabRockies/SAM#1401

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

This seems to fix the problem of heliostat availability (sf_adjust) not being applied correctly for the Power Tower model in either time series or custom periods mode. Would be good for @tyneises to confirm.

@sjanzou sjanzou merged commit 06c4c33 into patch May 25, 2023
@sjanzou sjanzou deleted the ssc_1027 branch May 25, 2023 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CSP models use different class for adjustment factors

3 participants