-
Notifications
You must be signed in to change notification settings - Fork 542
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
test on UST seems not correct in w3iogomd.F90 #1095
Comments
@mickaelaccensi I agree those variables should not be calculated unless those variables are defined. Do we need the additional if MAPSTA(IY,IX) .EQ. 1 as well as the additional check? I made a branch with your update here: https://github.com/jessicameixner-noaa/ww3/tree/bug/initvalueust and here are the regtest output:
|
Thanks Jessica ! |
We do see a diff in ust:
Basically the same as the diffs that you see here: #1089 (comment) |
Describe the bug
AUX1 and AUX2 should only be written if UST(ISEA) and ASF(ISEA) is not UNDEF
To Reproduce
running regtests ww3_tp2.6, ww3_ufs1.2, mww3_test_04 should show differences
Expected behavior
IX = MAPSF(ISEA,1)
IY = MAPSF(ISEA,2)
IF ( MAPSTA(IY,IX) .EQ. 1 ) THEN
should be replaced by :
IF ( UST(ISEA) .NE. UNDEF .AND. ASF(ISEA) .NE. UNDEF ) THEN
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
The text was updated successfully, but these errors were encountered: