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

Rename gamma r #1231

Merged
merged 4 commits into from
Nov 4, 2024
Merged

Rename gamma r #1231

merged 4 commits into from
Nov 4, 2024

Conversation

janinefreeman
Copy link
Collaborator

@janinefreeman janinefreeman commented Oct 28, 2024

Renames "gamma r" to "gamma pmp", and also removes it from two compute modules that had it as an unused input (pv6parmod and pvsamv1). Closes NREL/SAM#210 and goes with NREL/SAM#1898

@janinefreeman janinefreeman merged commit 184e156 into develop Nov 4, 2024
8 checks passed
@janinefreeman janinefreeman deleted the renameGammaR branch November 4, 2024 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pv photovoltaic, pvsam, pvwatts PySAM/SDK Breaking Change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CEC Single Diode model: gamma_r should be renamed gamma_pmp
3 participants