Changes between Version 1 and Version 2 of ticket/1620_MERCATOR_SOLVER_SIMPLIFICATION

Timestamp:

2015-11-09T14:34:42+01:00 (8 years ago)

Author:

jchanut

Comment:

--

ticket/1620_MERCATOR_SOLVER_SIMPLIFICATION

@@ -20 +20 @@
 NB: one should change "solver.stat" file and numsol names in stp_ctl and in_out_manager... keep it as is for the time being[[BR]]
 
-==> '''Commit changes to revision:5868'''
-[https://forge.ipsl.jussieu.fr/nemo/changeset?old_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&old=5867&new_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&new=5868]
+==> '''Commit changes to revision 5868'''
+[https://forge.ipsl.jussieu.fr/nemo/changeset?old_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&old=5867&new_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&new=5868] 
+
+GYRE        restartability  passed
+ORCA2LIMPIS restartability  passed
+ORCA2OFFPIS restartability  passed
+AMM12       restartability  passed
+ORCA2LIM3   restartability  passed
+SAS         restartability  inconclusive (  solver.stat  not found )
+GYRE        reproducibility passed
+ORCA2LIMPIS reproducibility passed
+ORCA2OFFPIS reproducibility passed
+AMM12       reproducibility passed
+ORCA2LIM3   reproducibility passed
+SAS         reproducibility inconclusive (  solver.stat  not found )
+ORCA2_LIM_AGRIF runability passed
+ORCA2_LIM_AGRIF code corruption test passed
 
 ==> Pass Sette tests:YES
@@ -30 +45 @@
 ln_hpg_imp namelist variable suppressed in reference namelist.[[BR]]
 
-==> Commit changes to revision: ???[[BR]]
-==> Pass Sette tests:???
+==> '''Commit changes to revision 5869'''[ https://forge.ipsl.jussieu.fr/nemo/changeset?old_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&old=5868&new_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&new=5869] [[BR]]
+
+==> Pass Sette tests: OK
 
 '''Step 3: Time Stepping readibility '''[[BR]]
 
 1)''' dynzdf'''[[BR]]
-Make time stepping identical in explicit or split-explicit cases => assume that "after" velocities not tendencies are available after vertical diffusion of momentum (as for tracers) => modify dynzdf routines
+Make time stepping identical in explicit or split-explicit cases => assume that "after" velocities not tendencies are available after vertical diffusion of momentum (as for tracers) => modify dynzdf/dynnxt routines
 
 2) '''step'''[[BR]]
-- Remove useless conditions depending on ln_dynspg_ts in step. Group all dynamic trends at the same place (at the beginning of the time stepping). Still keep dynnxt after tracer time stepping at the end.[[BR]]
+- Group all dynamical trends at the same place (at the beginning of the time stepping). Still keep dynnxt after tracer time stepping at the end.[[BR]]
 - Assuming ua/va are not used anymore during tracer computation as working arrays, remove temporary 3d arrays ua_sv, va_sv
 
@@ -45 +61 @@
 ==> Pass Sette tests:???
 
-'''Step 4: additional simplifications'''
+'''Step 4: additional simplifications/corrections'''
+- Correct momentum trend computation with time splitting
+- Agrif cleaning and compatibility with explicit free surface
 - Remove key_dynspg_ts and key_dynspg_exp cpp keys (create namelist block to choose between the two)
 - Suppress dynspg_oce module ?