Describe the bug
The solar resource file converter macro pulls minute values from the SolarAnywhere weather file (e.g., minute value of "00" for a row with original timestamp "01/01/2021 11:00") and uses that to populate the resulting SAM weather file.
For 60 minute interval SolarAnywhere data, this results in a SAM-formatted weather file with a minute column full of zeros. My understanding of the solar position calculations in SAM, and what is represented in SolarAnywhere data [1], is that it would be better to either leave the minutes column out for 60 minute interval data, or to calculate the minute that is the midpoint of the interval (e.g., 30 for 60 minute interval data).
This same issue should apply to sub-hourly SolarAnywhere as well. Calculating the midpoint minute should be a more robust solution.
Tagging @KWagnerCPR, as they submitted a recent issue related to Clean Power Research SolarAnywhere data.
Here is the relevant code, I think:
https://github.com/NREL/SAM/blob/062a09e263f34b32846c5d801ae23619f60092d9/deploy/runtime/macros/Solar%20Resource%20File%20Converter.lk#L523-L550
Additional context
[1] https://www.solaranywhere.com/support/data-fields/resolution/#data-averaging
Describe the bug
The solar resource file converter macro pulls minute values from the SolarAnywhere weather file (e.g., minute value of "00" for a row with original timestamp "01/01/2021 11:00") and uses that to populate the resulting SAM weather file.
For 60 minute interval SolarAnywhere data, this results in a SAM-formatted weather file with a minute column full of zeros. My understanding of the solar position calculations in SAM, and what is represented in SolarAnywhere data [1], is that it would be better to either leave the minutes column out for 60 minute interval data, or to calculate the minute that is the midpoint of the interval (e.g., 30 for 60 minute interval data).
This same issue should apply to sub-hourly SolarAnywhere as well. Calculating the midpoint minute should be a more robust solution.
Tagging @KWagnerCPR, as they submitted a recent issue related to Clean Power Research SolarAnywhere data.
Here is the relevant code, I think:
https://github.com/NREL/SAM/blob/062a09e263f34b32846c5d801ae23619f60092d9/deploy/runtime/macros/Solar%20Resource%20File%20Converter.lk#L523-L550
Additional context
[1] https://www.solaranywhere.com/support/data-fields/resolution/#data-averaging