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

Fix reporting of ground reflected rear irradiance gains in loss diagram #1125

Merged
merged 2 commits into from Feb 12, 2024

Conversation

mjprilliman
Copy link
Collaborator

-Individual components of bifacial rear irradiance gain in PV loss diagram were reporting wrong values due to clear sky calls of function calculating rear irradiance being made after original call.
-Update variable declarations to not overwrite bifacial irradiance outputs
-No changes to bifacial energy calculations, only loss diagram outputs

@mjprilliman mjprilliman added bug pv photovoltaic, pvsam, pvwatts labels Jan 25, 2024
@mjprilliman mjprilliman added this to the 2023 Release Patch 1 milestone Jan 25, 2024
@mjprilliman mjprilliman self-assigned this Jan 25, 2024
Copy link
Collaborator

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

Some test comparisons based on Detailed PV / No Financial with bifacial module enabled, "Ground reflected to rear" reported on Losses tab:

SAM 2022.11.21: 6.181%
SAM 2023.12.17: 0.142%
SAM Open Source ssc-1124-fix: 6.399%

@mjprilliman mjprilliman merged commit eaa5d02 into patch Feb 12, 2024
4 checks passed
@mjprilliman mjprilliman deleted the ssc-1124-fix branch February 12, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
added to release notes bug pv photovoltaic, pvsam, pvwatts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Issue with reporting of ground reflected rear irradiance in loss diagram
2 participants