Skip to content
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

Update input definitions for ww3_uprstr modules #255

Closed
ukmo-ansaulter opened this issue Oct 5, 2020 · 5 comments
Closed

Update input definitions for ww3_uprstr modules #255

ukmo-ansaulter opened this issue Oct 5, 2020 · 5 comments
Assignees
Labels
enhancement New feature or request

Comments

@ukmo-ansaulter
Copy link
Collaborator

ukmo-ansaulter commented Oct 5, 2020

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:

  • read in variables specific to the update process selected
  • output the values provided in the ww3_uprstr.out log file
  • update the .inp template file and regtests to improve clarity and work with the changes
  • add capability to read inputs from a namelist (ww3_uprstr.nml) file
@ukmo-ansaulter ukmo-ansaulter self-assigned this Oct 5, 2020
@ukmo-ansaulter ukmo-ansaulter added the enhancement New feature or request label Oct 5, 2020
@ukmo-ansaulter
Copy link
Collaborator Author

@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
Copy link
Collaborator Author

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 ******


  • test case: ww3_ta1; test run: ./work_UPD0F_O

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD5_U

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD3_U

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD3_O

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD2_O

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD0F_U

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD5_O

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD2_U_cap

found 15 files in base directory
found 15 files in compare directory
10 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
3 files differ
ww3_uprstr.out restart001.ww3 (binary) out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD5_U_cap

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD6_U_cap

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD3_U_cap

found 15 files in base directory
found 15 files in compare directory
10 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
3 files differ
ww3_uprstr.out restart001.ww3 (binary) out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD6_U

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD2_U

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

  • test case: ww3_ta1; test run: ./work_UPD6_O

found 15 files in base directory
found 15 files in compare directory
11 files are identical
2 files skipped
0 files in base directory only
0 files in comp directory only
2 files differ
ww3_uprstr.out out_grd.ww3 (binary)

@mickaelaccensi
Copy link
Collaborator

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

@ukmo-ansaulter
Copy link
Collaborator Author

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

@ukmo-ansaulter
Copy link
Collaborator Author

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

  • ww3_ta1/./work_UPD0F_O (3 files differ)
  • ww3_ta1/./work_UPD5_U (3 files differ)
  • ww3_ta1/./work_UPD2_U (3 files differ)
  • ww3_ta1/./work_UPD6_O (3 files differ)
  • ww3_ta1/./work_UPD3_U_cap (4 files differ)

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

matrixDiff.out.TXT

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants