-
Notifications
You must be signed in to change notification settings - Fork 26
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
example_1 does not complete - due to some issue in tide_faq? #165
Comments
@WPringle can you please take a look? I'm not very familiar with this territory of the code. |
@shinbunya @SorooshMani-NOAA This is probably due to bad interpolation which I told Jaime about a long time ago, that you need to use complex numbers to get the phase correct. I'll take a look and see if I can come up with fix. |
Actually, the problem is not with complex interpolation. The problem is that the interpolation picks up the near shore values of the TPXO which tend to not be very accurate. There are 2 potential fixes for this. One is to redefine the open boundary to only use nodes that are in deeper water, the other is to set a depth threshold value for the interpolation to discard the nearshore nodes and simply pad them with the nearest. |
Also, in order to be able to compare apples to apples, make sure that you are setting up the same start date in adcircpy as the start date in the Shinnecock example from the ADCIRC website. Thanks. |
I generated a set of input files by executing sample_1.py. I then ran adcirc on it. The run stopped before completion. As I looked into it, I found that the nodal phases of tidal constituents in fort.15 looked odd. The beginning of it goes like this:
The phase at the first node (= 2.20112655e+02) is different by about 90 degrees from the one at the second line (= 3.11130902e+02), which seems to be too big. The phases at the following nodes also vary to a larger extent than expected. I replaced these nodal constituents with ones in fort.15 from the ADCIRC website's Shinnecock inlet case. Then, an adcirc run completed with the replaced nodal constituents. I believe there should be something wrong in these nodal constituent values.
The text was updated successfully, but these errors were encountered: