-
Notifications
You must be signed in to change notification settings - Fork 183
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
Solar Resource File Converter Macro for SolarAnywhere V3.4 Files #491
Comments
@KWagnerCPR, would you mind attaching an example of a file in the both the TMY3 and SA format generated by SolarAnywhere V3.4 that we can use as a basis for updating the macro? |
@cpaulgilman Sure thing! I'm going to email you examples of a time series, TGY, and TDY files in both TMY3 and SolarAnywhere output formats. We also provide probability of exceedance files in both formats. Do you need examples of those? The header information in cell H1 would be slightly different. |
Sample files for testing: |
* Replace SolarAnywhere format check for number of columns by check for "0" in first column of first row. * Handle single-year files with less than one-year of data. For example, files for current year only have data up to current date. * Clean up Solar Resource File Checker error mssage. * Solar Resource UI, error message, explain to remove file from library when file with partial year data is added. I decided not to remove files with less than one year of data from library in preparation for SAM being able to run part year simulations in the future.
…mats (#542) * Fix #491 Solar Resource Converter macro for new SolarAnywhere format * Replace SolarAnywhere format check for number of columns by check for "0" in first column of first row. * Handle single-year files with less than one-year of data. For example, files for current year only have data up to current date. * Clean up Solar Resource File Checker error mssage. * Solar Resource UI, error message, explain to remove file from library when file with partial year data is added. I decided not to remove files with less than one year of data from library in preparation for SAM being able to run part year simulations in the future. * Resolve Solar Resource Data.txt merge conflict with patch * Solar Resource UI message when weather file reader fails Show name of offending file with suggestion to remove it from weather file folders.
Describe the bug
SolarAnywhere V3.4 file downloads have 9 columns in the first row. This is the case when the metadata in cell H1 spills over into cell I1. The SAM solar resource file converter macro is currently designed to only accept files with 8 columns in row 1 so attempting to convert a SolarAnywhere V3.4 file breaks the macro.
To Reproduce
Steps to reproduce the behavior:
Weather file format drop down in the top right corner.
Expected behavior
I would expect the converter to successfully convert the SolarAnywhere format file to a samcsv format file as it has in the past when converting SolarAnywhere V3.3 files. But instead you receive an error saying the file does not appear to be in SolarAnywhere format.
Screenshots
Operating System and Version
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: