update attributes for psm3-tmy to match the lk script#1023
Conversation
|
This fix is consistent with the NSRDB API documentation for the psm3-tmy endpoint showing that clearsky attributes are not available: https://developer.nrel.gov/docs/solar/nsrdb/psm3-tmy-download/ Other endpoints, such as psm3 have clearsky attributes that we want to include when available to address this SSC issue: NatLabRockies/ssc#151 |
This ensures that time zone is local
|
When a call to the NSRDB fails, SAM tries a second call with no attributes in case the attribute list was incorrect. The reason the psm3-tmy call with incorrect attributes returned a file with UTC time zone is because the subsequent call with no attributes did not include the UTC parameter. That is fixed in 8593030. |
cpaulgilman
left a comment
There was a problem hiding this comment.
This should be good after my commit adding the UTC attribute to the second API call.
Pull Request Template
Description
Queries to psm3-tmy were coming back with no timezone information. When I tried the query in a browser, I got the following error:
Remove the problematic attrs, which now matches the lk script for the quick download.
Fixes #1022
To test:
Type of change
Please delete options that are not relevant.