New URL for NEMO forge!   http://forge.nemo-ocean.eu

Since March 2022 along with NEMO 4.2 release, the code development moved to a self-hosted GitLab.
This present forge is now archived and remained online for history.
ticket/1620_MERCATOR_SOLVER_SIMPLIFICATION (diff) – NEMO

Changes between Version 14 and Version 15 of ticket/1620_MERCATOR_SOLVER_SIMPLIFICATION


Ignore:
Timestamp:
2015-11-24T18:55:20+01:00 (8 years ago)
Author:
jchanut
Comment:

--

Legend:

Unmodified
Added
Removed
Modified

  • ticket/1620_MERCATOR_SOLVER_SIMPLIFICATION

    v14 v15  
    77'''Step 1: Remove filtered free surface option and associated key_dynspg_flt key''' 
    88 
    9   1) Remove parts of the code specific to "key_dynspg_flt" or lk_dynspg_flt=T and the need for solver modules in the code.[[BR]]                  2) Suppress SOL directory and dynspg_flt module[[BR]] 
     9  1) Remove parts of the code specific to "key_dynspg_flt" or lk_dynspg_flt=T and the need for solver modules in the code.[[BR]]                   2) Suppress SOL directory and dynspg_flt module[[BR]] 
    1010 
    11113) Clean momentum trend routines related to filtered free surface (trddyn, trdken, trd_oce).[[BR]] 4) Update reference configurations: [[BR]] 
    … …  
    1919==> '''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] 
    2020 
    21 '''==> Passes Sette [tests:YES tests: YES]''' 
    22  
    2321'''Step 2: Suppress implicit pressure gradient option, ln_hpg_imp=T ''' 
    2422 
    2523Since this option is not supported with the now default split explicit free surface: remove it ( => simpler step routine). ln_hpg_imp namelist variable suppressed in reference namelist.[[BR]] 
    2624 
    27 ==> '''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]] 
    28  
    29 '''==> Passes Sette tests: YES''' 
     25==> '''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]  
    3026 
    3127'''Step 3: Time Stepping readibility / cleaning '''[[BR]] 
    … …  
    51474) '''Reviewed AGRIF code in the case of explicit free surface''': slight changes in agrif_opa_interp module, but otherwise ok (quickly tested in ORCA2 Agulhas configuration) 
    5248 
    53 ==> Commit changes to revision: 5902 https://forge.ipsl.jussieu.fr/nemo/changeset?old_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&old=5869&new_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&new=5902  
     49==> Commit changes to revision: 5902 https://forge.ipsl.jussieu.fr/nemo/changeset?old_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&old=5869&new_path=%2Fbranches%2F2015%2Fdev_r5847_MERCATOR9_solveur_simplification&new=5902 
    5450 
    5551'''Step 4: Gurvan's Review:''' 
    … …  
    8884!GM:dynspg_oce  should be suppressed : all variables should be declared either in dynspg.F90 (ln_dynspg_exp and ln_dynspg_ts) or in dynspg_ts for the others 
    8985 
    90 == '''JC => DONE. But  time splitting arrays are defined in oce.F90 to avoid cyclic dependencies (this is linked to agrif bcs).''' == 
     86== '''JC => DONE. But  time splitting arrays are defined in oce.F90 to avoid cyclic dependencies (this is linked to agrif bcs).''' == 
    9187GM: Step.F90   further simplification are needed here : 
    9288