The viscosity and diffusivity coefficients that are produced are incorrect when you activate:
ln_traldf_lap = .true.
and use 31 for nn_aht_ijk_t
ln_dynldf_lap = .true.
and use 31 for nn_ahm_ijk_t
The coefficients should be true to what is written in the commentary, namely it should be ahm = 1/2 Uv*Lv
instead of ahm = 1/12 Uv*Lv
which is what is computed for momentum. Similarly, for tracers, aht
should be 1/2 Ud*Ld
instead of 1/12 Ud*Ld
.
The laplacian coefficient will be 6 times smaller than expected. This will result in excessive diffusion of tracers and/or momentum in your configuration.
Replace 1/12 by 1/2 for the case "31" in ldftra.F90 and ldfdyn.F90
The viscosity and diffusivity coefficients that are produced are incorrect when you activate:
ln_traldf_lap = .true.
and use 31 for nn_aht_ijk_t
ln_dynldf_lap = .true.
and use 31 for nn_ahm_ijk_t
The coefficients should be true to what is written in the commentary, namely it should be ahm = 1/2 Uv*Lv
instead of ahm = 1/12 Uv*Lv
which is what is computed for momentum. Similarly, for tracers, aht
should be 1/2 Ud*Ld
instead of 1/12 Ud*Ld
.
The laplacian coefficient will be 6 times smaller than expected. This will result in excessive diffusion of tracers and/or momentum in your configuration.
Replace 1/12 by 1/2 for the case "31" in ldftra.F90 and ldfdyn.F90
This is a trial run to check if an external user can create a bug report on the NEMO Gitlab.
Katherine
Closes #71
Once the development is done, the PI should complete the tests section below and after ask the reviewers to start their review.
This part should contain the detailed results of SETTE tests (restartability and reproducibility for each of the reference configuration) and detailed results of restartability and reproducibility when the option is activated on specified configurations used for this test.
Regular checks
A successful review is needed to schedule the merge of this development into the future NEMO release during next Merge Party (usually in November).
Assessments
Katherine Hutchinson (7808b142) at 01 Jul 13:07
The viscosity and diffusivity coefficients that are produced are incorrect when you activate:
ln_traldf_lap = .true.
and use 31 for nn_aht_ijk_t
ln_dynldf_lap = .true.
and use 31 for nn_ahm_ijk_t
The coefficients should be true to what is written in the commentary, namely it should be ahm = 1/2 Uv*Lv
instead of ahm = 1/12 Uv*Lv
which is what is computed for momentum. Similarly, for tracers, aht
should be 1/2 Ud*Ld
instead of 1/12 Ud*Ld
.
The laplacian coefficient will be 6 times smaller than expected. This will result in excessive diffusion of tracers and/or momentum in your configuration.
Replace 1/12 by 1/2 for the case "31" in ldftra.F90 and ldfdyn.F90
Add a .gitlab-ci.yml to set up the ci of NEMO (need to be tuned once discussion on host machine for runner finalised)