Skip to content

Conversation

cpaulgilman
Copy link
Collaborator

Improve error messages and descriptions for :

  • File names with parentheses
  • SAM parameters that are not provided in PAN files -- use current value on Module/Inverter page instead of setting to default value

Files for testing: test-pan-ond-files.zip

Fixes #2005

Checklist

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

Improve error messages and descriptions for :

* File names with parentheses
* SAM parameters that are not provided in PAN files -- use current value on Module/Inverter page instead of setting to default value
@cpaulgilman cpaulgilman added this to the SAM Fall 2025 Release milestone Sep 19, 2025
@cpaulgilman cpaulgilman self-assigned this Sep 19, 2025
@cpaulgilman cpaulgilman added pv pvsam1, pvwatts, hcpv UI User interface issue that applies across performance and financial models labels Sep 19, 2025
@cpaulgilman cpaulgilman merged commit 3acd976 into develop Sep 25, 2025
8 checks passed
@cpaulgilman cpaulgilman deleted the sam-2005-pan-ond-macro-fixes branch September 25, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pv pvsam1, pvwatts, hcpv UI User interface issue that applies across performance and financial models

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve PVsyst PAN to SAM macro to handle NOCT

2 participants