Skip to content

Remove bifacial messages for one-axis tracking#1311

Merged
cpaulgilman merged 1 commit into
developfrom
sam-2026-fix-bifacial-messages
Apr 4, 2025
Merged

Remove bifacial messages for one-axis tracking#1311
cpaulgilman merged 1 commit into
developfrom
sam-2026-fix-bifacial-messages

Conversation

@cpaulgilman

Copy link
Copy Markdown
Collaborator

@cpaulgilman cpaulgilman added the pv photovoltaic, pvsam, pvwatts label Apr 4, 2025
@cpaulgilman cpaulgilman added this to the SAM Spring 2025 Release milestone Apr 4, 2025
@cpaulgilman cpaulgilman self-assigned this Apr 4, 2025
@cpaulgilman cpaulgilman requested review from brtietz and sjanzou and removed request for janinefreeman and mjprilliman April 4, 2025 17:03

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

Thanks for removing these!

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 14269408187

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 158 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.002%) to 54.707%

Files with Coverage Reduction New Missed Lines %
ssc/ssc/cmod_pvwattsv8.cpp 51 82.86%
ssc/ssc/cmod_pvsamv1.cpp 107 87.38%
Totals Coverage Status
Change from base Build 14267854697: -0.002%
Covered Lines: 64318
Relevant Lines: 117568

💛 - Coveralls

@cpaulgilman cpaulgilman merged commit 21650f6 into develop Apr 4, 2025
@cpaulgilman cpaulgilman deleted the sam-2026-fix-bifacial-messages branch April 4, 2025 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

added to release notes pv photovoltaic, pvsam, pvwatts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove SSC messages for bifacial modules with tracking, non-horizontal arrays

3 participants