Wave File Reader error handling, header reading#1701
Conversation
|
@mjprilliman Is there supposed to be an option to choose time series or JPD for the ME Wave model? Help describes the option, but I don't see it on the Wave Resource page: Also, when I click Create JPD from time series weather file(s) for a folder with 1979-2010 time series data that I just downloaded, I get a UI callback error: |
Paul, no for the Single Owner model there is only the time series option due to the requirement for a time series in the cash flow. This PR is to fix some issues with JPD file generation and to make the header data for wave and tidal resource data more dynamic. I will look into the bug you've posted and schedule a meeting to discuss the weather file handling. |
cpaulgilman
left a comment
There was a problem hiding this comment.
JPD file generation works. Other issues to be fixed separately.
|
@mjprilliman I'm testing this with the hourly file you fixed from this discussion on the SAM Forum. I put the file in my SAM Downloaded Weather files folder, but am not seeing it in the library, even after clicking Refresh library and reloading the input page. The file does appear in the library in SAM 2023.12.17, but not in my build from this me-wave-file-reader branch. |
cpaulgilman
left a comment
There was a problem hiding this comment.
See my previous comment about the library.
Did you have the ssc branch checked out as well? |
Yes, this is for a clean build from SAM/me-wave-file-reader and ssc/me-wave-file-reader. |
Can you pull ssc for the most recent commit and try again? I downloaded the file directly from email and was able to get it into the library locally just now. |
|
In the UI callback for the Marine Wave Resource Time Series form, I'm seeing Also, in this snippet for the Refresh library button there is |
|
@mjprilliman Here's some additional information: I have several valid wave resource files in my However, if I add The library should populate from the folder shown in "Folder for Downloaded Wave Data Files" and the folders listed under "Wave Data File Folders" in the Wave Resource Data Settings widget. |
cpaulgilman
left a comment
There was a problem hiding this comment.
Hourly ME Wave files load into Wave Resource library correctly and simulations run.







Pull Request Template
Description
-Update file reader to catch missing data, throw error in JPD generation process
-Update file reader to be more flexible with header data in first two rows
Fixes #1673
Type of change
Please delete options that are not relevant.
Checklist:
If you have added a new compute module in a SSC pull request related to this one, be sure to check the Process Requirements.