Skip to content

Spectral correction model upgrades#2203

Open
mjprilliman wants to merge 22 commits into
developfrom
spectral-correction-upgrades
Open

Spectral correction model upgrades#2203
mjprilliman wants to merge 22 commits into
developfrom
spectral-correction-upgrades

Conversation

@mjprilliman

@mjprilliman mjprilliman commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Pull Request Template

Description

-Add user interface for Detailed PV Spectral Correction Factors
-Choose between three models
-Help messages to indicate if required weather data, cell technology information is available
-Begin to update help documentation
-Cleanup some formatting issues on PV Resource help page
-Add precipitable water to NSRDB calls (TODO: update solar_resource default files to psm v4, add precipitable water in separate issue)

[ include directions for reviewers if necessary- what or how should they test or review ]

Fixes #NatLabRockies/ssc#24

Corresponding branches and PRs:

[[ which branches of wex, lk, and ssc should be built with this PR ]]
same branch for ssc, rest develop

[ link any corresponding PRs in other repos, i.e. NREL/ssc#x ] (NatLabRockies/ssc#1406)

Unit Test Impact:

[ new tests written? ]

[ expected changes in unit tests or speed of tests? ]

[ expected changes in test_results files? ]

Checklist

  • [x ] requires help revision and I added that label
  • [x ] adds, removes, modifies, or deletes variables in existing compute modules
  • adds a new compute module
  • [x ] changes defaults
  • [x ] I've tagged this PR to a milestone

Reminders- this section can be deleted

[Checking for PySAM Incompatible API Changes]
(https://github.com/NREL/SAM/wiki/PySAM-Incompatible-API-Changes-&-Regenerating-PySAM-Files).

[When do the PySAM files need to be regenerated?]
(https://github.com/NREL/SAM/wiki/PySAM-Incompatible-API-Changes-&-Regenerating-PySAM-Files#when-do-the-pysam-files-need-to-be-regenerated-via-export_config)

@mjprilliman mjprilliman added this to the SAM 2026 Release milestone Jun 3, 2026
@mjprilliman mjprilliman self-assigned this Jun 3, 2026
@mjprilliman mjprilliman added enhancement pv pvsam1, pvwatts, hcpv requires help revision Requires a Help revision before releasing public version labels Jun 3, 2026

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

@mjprilliman This looks good! I made a few tweaks to the UI, fixed the UI message logic (I hope), and fixed the documentation merge conflict.

One question: If I run a simulation with the "Air mass and precipitable water" option with a weather file that has no precipitable water data, the simulation finishes with no messages. What precipitable water data does the model use in that case, or should SSC report an error?

@mjprilliman

Copy link
Copy Markdown
Collaborator Author

@mjprilliman This looks good! I made a few tweaks to the UI, fixed the UI message logic (I hope), and fixed the documentation merge conflict.

One question: If I run a simulation with the "Air mass and precipitable water" option with a weather file that has no precipitable water data, the simulation finishes with no messages. What precipitable water data does the model use in that case, or should SSC report an error?

Good catch, precipitation was defaulting to 0 instead of NaN, which was wrong. Should be working correctly now, and throwing an error message if precipitation data is not present.

@cwhanse

cwhanse commented Jun 16, 2026

Copy link
Copy Markdown

-Choose between three models

Rather than dig through the files to find out: which three models are offered?

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement pv pvsam1, pvwatts, hcpv requires help revision Requires a Help revision before releasing public version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants