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

Hotfix/hf master (#45) #54

Merged
merged 1 commit into from
Jun 27, 2019
Merged

Hotfix/hf master (#45) #54

merged 1 commit into from
Jun 27, 2019

Conversation

ajhenrique
Copy link
Collaborator

@ajhenrique ajhenrique commented Jun 24, 2019

Hotfix featuring:

In running the full regtests matrix prior to this pull request, there were the expected differences in mww3_test_03 case dN with load balancing options, and in mod_defs for one variation of 3 tests (mww3_test_07, ww3_tp2.7, and ww3_ts4). In my view these results are robust, they identify something that needs to be fine-combed in the generation of mod_defs for these latter tests, but that indicate that all branches are ready for merge.

regtests matrix output run on Theia


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


mww3_test_03/./work_PR3_UQ_MPI_d2 (8 files differ)
mww3_test_03/./work_PR3_UQ_MPI_d2_c (8 files differ)
mww3_test_03/./work_PR2_UQ_MPI_d2 (6 files differ)
mww3_test_03/./work_PR3_UNO_MPI_d2 (3 files differ)
mww3_test_03/./work_PR3_UNO_MPI_d2_c (3 files differ)
mww3_test_03/./work_PR1_MPI_d2 (7 files differ)
mww3_test_03/./work_PR2_UNO_MPI_d2 (4 files differ)
mww3_test_07/./work_PR3_UQ (3 files differ)
ww3_tp2.7/./work_ST0 (1 files differ)
ww3_ts4/./work_ug_MPI (1 files differ)


******************** 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: mww3_test_03; test run: ./work_PR3_UQ_MPI_d2

found 135 files in base directory
found 135 files in compare directory
124 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
8 files differ
out_pnt.low2 (binary) out_grd.low2 (binary) out_grd.low3 (binary) out_grd.low1 (binary) out_pnt.hgh2 (binary) out_grd.hgh2 (binary) out_grd.hgh3 (binary) out_grd.hgh1 (binary)

  • test case: mww3_test_03; test run: ./work_PR3_UQ_MPI_d2_c

found 135 files in base directory
found 135 files in compare directory
124 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
8 files differ
out_pnt.low2 (binary) out_grd.low2 (binary) out_grd.low3 (binary) out_grd.low1 (binary) out_pnt.hgh2 (binary) out_grd.hgh3 (binary) out_grd.hgh2 (binary) out_grd.hgh1 (binary)

  • test case: mww3_test_03; test run: ./work_PR2_UQ_MPI_d2

found 135 files in base directory
found 135 files in compare directory
126 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
6 files differ
out_pnt.low2 (binary) out_grd.low2 (binary) out_grd.low1 (binary) out_grd.hgh2 (binary) out_grd.hgh1 (binary) restart001.low1 (binary)

  • test case: mww3_test_03; test run: ./work_PR3_UNO_MPI_d2

found 135 files in base directory
found 135 files in compare directory
129 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
3 files differ
out_grd.low2 (binary) out_grd.hgh2 (binary) out_grd.hgh1 (binary)

  • test case: mww3_test_03; test run: ./work_PR3_UNO_MPI_d2_c

found 135 files in base directory
found 135 files in compare directory
129 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
3 files differ
out_grd.low2 (binary) out_grd.hgh2 (binary) out_grd.hgh1 (binary)

  • test case: mww3_test_03; test run: ./work_PR1_MPI_d2

found 135 files in base directory
found 135 files in compare directory
125 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
7 files differ
out_grd.low3 (binary) out_grd.low2 (binary) out_grd.low1 (binary) out_grd.hgh3 (binary) out_pnt.hgh2 (binary) out_grd.hgh2 (binary) out_grd.hgh1 (binary)

  • test case: mww3_test_03; test run: ./work_PR2_UNO_MPI_d2

found 135 files in base directory
found 135 files in compare directory
128 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
4 files differ
out_grd.low2 (binary) out_grd.low1 (binary) out_grd.hgh2 (binary) out_grd.hgh1 (binary)

  • test case: mww3_test_07; test run: ./work_PR3_UQ

found 25 files in base directory
found 25 files in compare directory
19 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
3 files differ
mod_def.rect1 (binary) mod_def.points (binary) mod_def.zcmpl (binary)

  • test case: ww3_tp2.7; test run: ./work_ST0

found 78 files in base directory
found 78 files in compare directory
74 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
1 files differ
mod_def.ww3 (binary)

  • test case: ww3_ts4; test run: ./work_ug_MPI

found 13 files in base directory
found 13 files in compare directory
9 files are identical
3 files skipped
0 files in base directory only
0 files in comp directory only
1 files differ
mod_def.ww3 (binary)

Pull request for HF to master featuring:
* bugfix for netcdf file id allocation
* remove extra line due to copy/paste error
* add WCOR to switch list
* add check on array index
* undo modification on ndates
* add feature for fixed filename with ndates=0 for ww3_ounf
Copy link
Collaborator

@JessicaMeixner-NOAA JessicaMeixner-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit looks good. Only comment is this also needs a PR for being merged into develop as well.

@ajhenrique
Copy link
Collaborator Author

Commit looks good. Only comment is this also needs a PR for being merged into develop as well.

@JessicaMeixner-NOAA changes proposed here are also to be merged into develop. We will follow the procedure for reintegrating directly from the Ifremer develop branch in that case. @mickaelaccensi will submit a pull request from the Ifremer develop branch providing output from his regtests matrix run for approval by reviewers.

@mickaelaccensi
Copy link
Collaborator

the only differences are in mww3_test05 due to a different number of time steps. Otherwise all the files are the same bit to bit. see the output of matrix.comp between full matrix from HF_ifremer_pull_20190521 and master branch

NOAA_HF_ifremer_pull_20190521_matrixDiff.txt

@ajhenrique ajhenrique merged commit edb73a3 into master Jun 27, 2019
@ajhenrique ajhenrique mentioned this pull request Jun 27, 2019
@ajhenrique ajhenrique deleted the HF_ifremer_pull_20190521 branch June 27, 2019 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants