Updates to nrelsam2013 -> PySAM Transition#466
Merged
dpinney-nreca merged 11 commits intonreca-bts:masterfrom Jan 28, 2026
Merged
Updates to nrelsam2013 -> PySAM Transition#466dpinney-nreca merged 11 commits intonreca-bts:masterfrom
dpinney-nreca merged 11 commits intonreca-bts:masterfrom
Conversation
…ith PySAM Pvwattsv8
This reverts commit 92b5060.
dpinney-nreca
approved these changes
Jan 28, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
API Request moved to weather.py for universal access and re usability,
Modified all the models that require new PySAM Code,
-> Models modified: solarSunda.py, solarFinancial.py, pvWatts.py, solarConsumer.py, solarCashFlow.py
-> Note: solarFinancial newly modified. There are some issues with non-default values for future updates
-> Temporary fixes to timestamp issues - may require future modifications with better ones.
Fixed latitude/longitude issue ( hopefully ),
-> Testing manually
Modifications to hostingExpansion.py/html to correspond with updates to DECAF requirements and specifications,
-> Includes modifications/additions to test files
removed an unused import in anomolyDetector.py,
Transition iso3166 import from requirements INTO omf solvers to remove one extra import.