NOAA-EMC / WW3

WAVEWATCH III
Other
262 stars 530 forks source link

Update input definitions for ww3_uprstr modules #255

Closed ukmo-ansaulter closed 3 years ago

ukmo-ansaulter commented 4 years ago

This enhancement is a minor technical change to improve the way in which input data is read in and logged by the ww3_uprstr program.

The changes should:

ukmo-ansaulter commented 4 years ago

@mickaelaccensi , I'm doing this work in preparation for the enhancement you are making with the use of .nml files for all modules/regtests. The work won't take long, will this time in with you OK?

There is an option for me to try and code the namelist input for ww3_uprstr up if that would be helpful? At the moment I have concentrated on .inp as there is no namelist option currently

ukmo-ansaulter commented 4 years ago

Completed initial regtests on the updated code/input files. These will write a more verbose output to ww3_uprstr.out and have also bugfixed incorrectly set cap values in tests UPD2_U_CAP and UPD3_U_CAP

As a result, new regtest outputs will be revised compared to previous branches; as per the summary below (NB the critical unchanged files for the regtest are the restart files in all tests except UPD2_U_CAP and UPD_3_CAP; the changed out_grd.ww3 is a known issue and does not affect these tests for ww3_uprstr.ftn):

   ******************************************************
 ***  compare WAVEWATCH III matrix of regression tests  ***
   ******************************************************

base directory : /home/d01/frxs/Git_WW3/working/WW3/regtests comp directory : /home/d02/frey/WW3/REGTESTS_GNU/regtests_96e3f3a_20200923 test(s) : ww3_ta1


***** non-identical cases ****


ww3_ta1/./work_UPD0F_O (2 files differ) ww3_ta1/./work_UPD5_U (2 files differ) ww3_ta1/./work_UPD3_U (2 files differ) ww3_ta1/./work_UPD3_O (2 files differ) ww3_ta1/./work_UPD2_O (2 files differ) ww3_ta1/./work_UPD0F_U (2 files differ) ww3_ta1/./work_UPD5_O (2 files differ) ww3_ta1/./work_UPD2_U_cap (3 files differ) ww3_ta1/./work_UPD5_U_cap (2 files differ) ww3_ta1/./work_UPD6_U_cap (2 files differ) ww3_ta1/./work_UPD3_U_cap (3 files differ) ww3_ta1/./work_UPD6_U (2 files differ) ww3_ta1/./work_UPD2_U (2 files differ) ww3_ta1/./work_UPD6_O (2 files differ)


**** identical cases *****



**** summary of comparison *** ** only results of non-identical cases are listed **** ** if less than 10 files differ for a case, they are listed **


mickaelaccensi commented 4 years ago

I can give you some help on namelist if needed. Just add me on your development branch

ukmo-ansaulter commented 4 years ago

Thanks @mickaelaccensi . I think I'm almost there, so will add you as a reviewer for the PR :-)

ukmo-ansaulter commented 4 years ago

Updates now have the namelist option available and this has been used in 5 regtests:

The differences in the regtests are an expected outcome of the changes. Details of the differences are attached.

matrixDiff.out.TXT