| 4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
| 5 |
============================== |
============================== |
| 6 |
|
|
| 7 |
|
o pkg/seaice: |
| 8 |
|
- seaice_preconditioner: |
| 9 |
|
+ replace one remaining press by pressLoc (mistake) |
| 10 |
|
+ adjust printing of residuals |
| 11 |
|
- improve I/O of diagnostics of JFNK in seaice_jfnk.F: |
| 12 |
|
+ add (SEAICE.h) and rename some counters (initiliased in |
| 13 |
|
seaice_init_fixed.F) |
| 14 |
|
+ add _BEGIN/END_MASTER (myThid) around print statements |
| 15 |
|
+ print total number of iterations and failures over SEAICE_monFreq |
| 16 |
|
interval to STDOUT |
| 17 |
|
- move control over output-messages by fgmres to seaice_jfnk to save |
| 18 |
|
some operations |
| 19 |
|
- move resetting initial guess for fgmres outside of seaice_fgmres_driver |
| 20 |
|
in order to make restarts with sol .ne. zero (fgmres with restarts) |
| 21 |
|
(seaice_jfnk.F, seaice_fgmres.F) |
| 22 |
|
- seaice_calc_l/rhs.F: replace maskW/S by maskInW/S; this fixes |
| 23 |
|
a serious convergence problem when open boundaries are involved |
| 24 |
|
o pkg/ggl90: fix computation of control volumes for the case of |
| 25 |
|
ALLOW_GGL90_HORIZDIFF (probably not used very often): move control volumes |
| 26 |
|
up half a grid cell from C-centered to W-centered (pointed out by C. Eden) |
| 27 |
|
o pkg/seaice: simplify seaice_evp by avoiding use of extra fields u/vIceC |
| 28 |
|
o pkg/autodiff |
| 29 |
|
- Add inAdMode switch for SEAICEuseDYNAMICS: |
| 30 |
|
SEAICEuseDYNAMICSinAdMode |
| 31 |
|
(extend logic in ADAUTODIFF_INADMODE_SET) |
| 32 |
|
o pkg/seaice |
| 33 |
|
- Step 1 of merging ice-thickness distribution (ITD) code from |
| 34 |
|
MITgcm_contrib/torge/itd/ (author: Torge Martin) |
| 35 |
|
The code implements concept laid out in Thorndike et al. (1975); |
| 36 |
|
code is enabled via CPP flag SEAICE_ITD |
| 37 |
|
nITD (in SEAICE_SIZE.h) defines number of ice thickness categories; |
| 38 |
|
code is not complete yet!! |
| 39 |
|
- SEAICE_SIZE.h now included in almost all seaice routines |
| 40 |
|
o verification (using seaice): |
| 41 |
|
- switch back seaice-dynamics (+ lower SOLV_MAX_ITERS) in ADM exp. |
| 42 |
|
global_ocean.cs32x15.seaice ; update results. |
| 43 |
|
- compile B-grid dynamics in FW test exp 1D_ocean_ice_column ; |
| 44 |
|
- compile C-grid dynamics with #define SEAICE_ALLOW_JFNK in FW test exp |
| 45 |
|
offline_exf_seaice (previously compiling B-grid). |
| 46 |
|
o pkg/exf: |
| 47 |
|
- fix some STOP conditions in exf_check.F (from Aug 28, modifs did not allow |
| 48 |
|
to specify neither just 1 comp of wind or wind-stress, nor no surface momentum |
| 49 |
|
forcing): just replace CPP option ALLOW_ATM_WIND with run-time param useAtmWind. |
| 50 |
|
o pkg/layers (adjust changes from Sept 19): |
| 51 |
|
- do snap-shot output and fill diagnostics from inside iLa loop (layers_calc.F) |
| 52 |
|
this allows to remove "layers_maxNum" dimension from all layers arrays. |
| 53 |
|
o pkg/layers (adjust changes from Sept 19): |
| 54 |
|
- remove ALLOW_LAYERS_OUTPUT ; fix snap-shot and time-ave output ; |
| 55 |
|
- snap-shot extended to all type of layers (not just the 1rst type); |
| 56 |
|
- remove layers variables when corresponding CPP option is undef. |
| 57 |
|
o pkg/layers: |
| 58 |
|
- rename pkg/layers variable and add new output (layer exist & un-weighted |
| 59 |
|
velocity in layer) |
| 60 |
|
o pkg/seaice: |
| 61 |
|
- simplify (?) ocean-drag coefficent computations by using |
| 62 |
|
the same routine in all three solvers (saves some code lines) |
| 63 |
|
- remove global fields DRAGA and DRAGS from C-grid code and |
| 64 |
|
declare them locally in lsr.F (the only place where they are |
| 65 |
|
still used) |
| 66 |
|
- update AD output of lab_sea.evp (very sensitive anyway). |
| 67 |
|
o pkg/seaice: add Jacobian-free Newton Krylov solver for ice dynamics |
| 68 |
|
(Lemieux et al. 2010, 2012). Turn on with the usual SEAICE_ALLOW_JFNK |
| 69 |
|
and SEAICEuseJFNK=.true. A few things to note: |
| 70 |
|
- involves a FGMRES, that is parallel but very inefficient, |
| 71 |
|
because of many scalar products (global sums) in the Gram-Schmidt |
| 72 |
|
orthogonalization method. Needs to be replaced. Original code |
| 73 |
|
from J.-F. Lemieux |
| 74 |
|
- preconditioner for FGMRES is a stripped down version of LSOR |
| 75 |
|
- SOLV_MAX_ITERS is now used for LSOR as a solver and as a |
| 76 |
|
preconditioner. For the preconditioner, the default is set to 10. |
| 77 |
|
- Jacobian times vector is computed with finite differences |
| 78 |
|
- marginally related to JFNK-solver: add a smooth regularization for |
| 79 |
|
zeta with tanh that will improve convergence (Lemieux and Tremblay, |
| 80 |
|
2009) for any implicit solver (not implemented for EVP because EVP |
| 81 |
|
does not need this regularization in the first place). Define |
| 82 |
|
SEAICE_ZETA_SMOOTHREG in SEAICE_OPTIONS.h to use. |
| 83 |
|
o eesupp/src: |
| 84 |
|
- check in genmake2 that intrinsic S/R flush is available. |
| 85 |
|
- add S/R to flush IO unit (if intrinsic S/R flush is available) |
| 86 |
|
- flush standard & error msg unit before MPI termination |
| 87 |
|
- if debugMode, flush IO unit after printing msg |
| 88 |
|
o pkg/ctrl: |
| 89 |
|
- ctrl_unpack.F, ALLOW_GENARR3D_CONTROL code: fix missing argument in call |
| 90 |
|
to ctrl_set_unpack_xyz. |
| 91 |
|
|
| 92 |
|
checkpoint64 (2012/10/09) |
| 93 |
|
o pkg/shelfice |
| 94 |
|
- in mom_fluxform.F, mom_vecinv.F: |
| 95 |
|
CALL SHELFICE_V_DRAG(bi,bj,k,vFld,KE,KappaRU,vF,myThid) |
| 96 |
|
Bug fix KappaRU -> KappaRV |
| 97 |
|
does not change verification/isomip but may change other setups |
| 98 |
|
(presumably constant KappaRU=KappaRV) |
| 99 |
|
- fix diagnostics SHIU/Vdrag (thanks to Tom Millgate) |
| 100 |
|
o pkg/obcs: |
| 101 |
|
- fixed some indexing bugs in obcs_seaice_sponge.F |
| 102 |
|
- added obcs relaxation subroutines for HSALT and HSNOW |
| 103 |
|
- changes verification/seaice_obcs/results/obcs_seaice_sponge.F |
| 104 |
|
o pkg/seaice: |
| 105 |
|
- add "multicategory" parameterization for snow in analogy to ice, |
| 106 |
|
default is off for now. |
| 107 |
|
o pkg/obcs: |
| 108 |
|
- added obcs_seaice_sponge.F for obcs relaxation of seaice variables; |
| 109 |
|
see verification/seaice_obcs/input.seaiceSponge for example application. |
| 110 |
|
- remove local commom bloc "OBCS_GLOBAL" in obcs_readparms.F and do all the OB |
| 111 |
|
index setting by master-thread. This fix multi-threads Pb in previous modifs. |
| 112 |
|
o pkg/layers : |
| 113 |
|
- add diagnostics to pkg/layers |
| 114 |
|
- move the meat of pkg/layers from layers_calc.F to layers_fluxcalc.F |
| 115 |
|
(from R.Tulloch). Layers_calc.F now calls layers_fluxcalc.F accordingly. |
| 116 |
|
o pkg/generic_advdiff/gad_implicit_r.F : |
| 117 |
|
- fix ADVr diagnostics in case of both implicit vertical advection and diffusion. |
| 118 |
|
o pkg/seaice/seaice_growth.F: |
| 119 |
|
- fix SItflux diagnostics in virtual salt flux + balanceEmpmr case. |
| 120 |
|
o pkg/obcs: |
| 121 |
|
- change default setting of OBCS_UVICE_OLD from defined to undef; |
| 122 |
|
- use new parameter OB_indexNone for null index value (instead of hard-coded 0) |
| 123 |
|
in all pkg/obcs code and in other pkgs; remove k loop in obcs_check_depths.F; |
| 124 |
|
- pass OB_indexNone as (new) argument to S/R CTRL_MASK_SET_[X,Y]Z; |
| 125 |
|
- change OB null index value (OB_indexNone) to -99 ; this fix the problem |
| 126 |
|
with Southern & Western OB at a tile edge (at least in a simple set-up). |
| 127 |
|
o pkg/obcs: |
| 128 |
|
- change settings of OB index in case OB coincides with tile edge (shared by |
| 129 |
|
2 tiles), so that common OB location is registered by both tiles (instead of |
| 130 |
|
only 1). However, for Southern and Western OB, index value = 0 is not |
| 131 |
|
recognized as valid OB index (=default null value) and needs to be fixed. |
| 132 |
|
- modify accordingly OBCS_BALANCE_FLOW and OBCS_MONITOR (avoid double counting) |
| 133 |
|
- new parameter "OB_indexNone" for null index value (for now, still zero) |
| 134 |
|
to replace hard-coded null index value of zero. |
| 135 |
|
o pkg/exch2: |
| 136 |
|
- improve W2_USE_E2_SAFEMODE (faster) and W2_E2_DEBUG_ON: |
| 137 |
|
always check buffer-size (but outside the loop); move checking for valid |
| 138 |
|
index from W2_USE_E2_SAFEMODE to W2_E2_DEBUG_ON; in W2_E2_DEBUG_ON, |
| 139 |
|
print each tile and point connection only if |W2_printMsg|>= 2 and 3. |
| 140 |
|
o pkg/ctrl (pack & unpack): |
| 141 |
|
- in ctrl_pack.F & ctrl_unpack.F, use the same local file-name variable |
| 142 |
|
(locally defined) for different variables (+ remove the variable-specific |
| 143 |
|
file name in CTRL_GENARR.h). For now, only implemented for GENARR2/3D_CONTROL |
| 144 |
|
(to be extended to other local file names in ctrl.h). |
| 145 |
|
o pkg/flt: |
| 146 |
|
- add a FLT_Iter0 option (similar to PTRACERS_Iter0). |
| 147 |
|
o pkg/mdsio: |
| 148 |
|
- fix missing STOP in mdsio_readvec_loc.F (case ioUnit=0) when file is missing |
| 149 |
|
o tools/genmake2: |
| 150 |
|
- add similar dependences for target EXE_AD & EXE_FTL as we have for EXECUTABLE |
| 151 |
|
(added by Alistair in version 1.51 for building without make depend); |
| 152 |
|
- remove Header files list (H_SRC_FILES) from the dependence of targets |
| 153 |
|
ad_input_code.f & ftl_input_code.f (not clear why it was added in the first |
| 154 |
|
place in version 1.23): this save going through staf when unrelated header |
| 155 |
|
file has been modified. |
| 156 |
|
Note: for an intermediate target (e.g., ad_input_code.f or adtaf) without |
| 157 |
|
"make depend" step, still need to do "make links" before "make adtaf". |
| 158 |
|
o eesupp/src (+ few pkg src files) & genmake2: |
| 159 |
|
- finish to remove ALWAYS_USE_MPI in source code that TAF does not see; |
| 160 |
|
- no longer set ALWAYS_USE_MPI in Makefile (was done by genmake2). |
| 161 |
|
- improve genmake2 checking for ALLOW_${PKG} in CPP_OPTIONS.h files (distinguish |
| 162 |
|
upper/lower case ; account for space between "#" and undef/define) |
| 163 |
|
o pkg/flt (with exch2): |
| 164 |
|
- cleaning: fix so that it compiles; move remaining MPI calls out of flt_exch2.F; |
| 165 |
|
- simplify threads synchronisation; fix (many issues) multi-threaded with MPI. |
| 166 |
|
|
| 167 |
|
checkpoint63s (2012/09/05) |
| 168 |
|
o pkg/ecco: |
| 169 |
|
- remove #ifdef ALLOW_SMOOTH_CORREL3D brackets. |
| 170 |
|
- add more relevant #ifdef ALLOW_SMOOTH ones. |
| 171 |
|
- sort out useAtmWind, useSMOOTH, ctrlSmoothCorrel2D. |
| 172 |
|
o pkg/ctrl: |
| 173 |
|
- CTRL_OPTIONS.h : document CPPs related to smoothing; ALLOW_SMOOTH_CORREL2D/3D, |
| 174 |
|
ALLOW_CTRL_SMOOTH, ALLOW_SMOOTH_CTRL2D, ALLOW_SMOOTH_CTRL3D. |
| 175 |
|
- ctrl.h : document run time params doInitXX, doMainPack, doMainUnpack, doPackDiag, |
| 176 |
|
doSinglePrecTapelev (audodiff param!), ctrlSmoothCorrel2D (new) and |
| 177 |
|
ctrlSmoothCorrel3D (new). |
| 178 |
|
The added ctrlSmoothCorrel2D/3D replace the old ALLOW_SMOOTH_CORREL2D/3D CPP. |
| 179 |
|
- ctrl_readparms.F : initialise ctrlSmoothCorrel2D/3D and read from namelist. |
| 180 |
|
- ctrl_init.F ctrl_pack.F ctrl_unpack.F : switch off ustress/vstress |
| 181 |
|
controls when useAtmWind, and vice versa. |
| 182 |
|
- ctrl_map_ini_ecco.F : UVEL0, VVEL0, edtaux, edtauy cannot be smoothed as tracers. |
| 183 |
|
- ctrl_get_gen.F : replace #ifdef ALLOW_SMOOTH_CORREL2D with |
| 184 |
|
#ifdef ALLOW_SMOOTH, IF ctrlSmoothCorrel2D, IF useSMOOTH. |
| 185 |
|
- ctrl_map_ini_gen.F : replace #ifdef ALLOW_SMOOTH_CORREL3D with |
| 186 |
|
#ifdef ALLOW_SMOOTH, IF ctrlSmoothCorrel3D, IF useSMOOTH. |
| 187 |
|
- ctrl_set_pack_xy[z].F, ctrl_set_unpack_xy[z].F: |
| 188 |
|
replace ALLOW_SMOOTH_CORREL2D/3D brackets with ctrlSmoothCorrel2D/3D. |
| 189 |
|
o pkg/smooth: |
| 190 |
|
- remove un-necessary includes. |
| 191 |
|
- remove ALLOW_SMOOTH*D and ALLOW_SMOOTH_CORREL*D brakets. |
| 192 |
|
Those CPP options were never defined, and not necessary. |
| 193 |
|
o model/src: |
| 194 |
|
- fix CPP brackets in ini_mixing.F (GM_background_K belongs to gmredi). |
| 195 |
|
- standard treatment of useSMOOTH. |
| 196 |
|
o pkg/autodiff: |
| 197 |
|
- add LOGICAL useGGL90inAdMode, useGGL90inFwdMode, useSALT_PLUMEinAdMode, |
| 198 |
|
useSALT_PLUMEInFwdMode allowing to switch off ggl90 and salt_plume adjoint. |
| 199 |
|
o pkg/exch2: |
| 200 |
|
- add "if usingMPI" where it was missing (in exch2_rx?_cube*.template) |
| 201 |
|
+ start to remove ALWAYS_USE_MPI in source code that TAF does not see. |
| 202 |
|
- make new S/R W2_MAP_PROCS out of w2_eeboot.F to setup exch2 CommonFlag |
| 203 |
|
and tiles to process mapping. |
| 204 |
|
- re-arrange 3 header files (W2_EXCH2_PARAMS.h, _BUFFER.h & _TOPOLOGY.h) |
| 205 |
|
moving /EXCH2_FILLVAL_RX/ & /W2_EXCH2_COMMFLAG/ to W2_EXCH2_TOPOLOGY.h; |
| 206 |
|
rename "exch2_tProc" to "W2_tileProc" and move seting of e2FillValue_RX |
| 207 |
|
from w2_eeboot.F to w2_readparms.F. |
| 208 |
|
o eesupp/src: |
| 209 |
|
- set exchNeedsMemsync & exchUsesBarrier according to "usingMPI" (instead of |
| 210 |
|
ALWAYS_USE_MPI) + moved from ini_communication_patterns.F to eeset_parms.F; |
| 211 |
|
also move setting of exchCollectStatistics from exch_init.F to eeset_parms.F |
| 212 |
|
- in eedie.F and all_proc_die.F, always call MPI_FINALIZE when ALLOW_USE_MPI |
| 213 |
|
is defined (remove condition on usingMPI) to match cases where MPI_INIT |
| 214 |
|
has been called. |
| 215 |
|
- fix bug I put in previous modif of exch_sm_3d_rx.template (not much used); |
| 216 |
|
- change default (non-MPI) value of myPid (from 1 to 0) to match MPI settings |
| 217 |
|
(so that on 1 proc, get same myPid independently of usingMPI) |
| 218 |
|
- add a stop if multi-procs but usingMPI=F ; stop also if usingMPI=T and |
| 219 |
|
useCubedSphereExchange with EXCH-1 (safer). |
| 220 |
|
- add "if usingMPI" where it was missing (+ start to remove ALWAYS_USE_MPI). |
| 221 |
|
o pkg/ggl90: |
| 222 |
|
- fix bug in filling corner mask array "mskCor" (FILL_CS_CORNER_TR_RL call) |
| 223 |
|
used in ALLOW_GGL90_SMOOTH code. |
| 224 |
|
o pkg/flt: |
| 225 |
|
- fix typo in FLT_WITHOUT_Y_PERIODICITY (the effect was to ignore stopping |
| 226 |
|
floats at the Southern end of the domain). |
| 227 |
|
o eesupp/inc: |
| 228 |
|
- add a list of CPP-options which can be set by Makefile (DEF_IN_MAKEFILE.h); |
| 229 |
|
but not used (and not to be used) anywhere, just for documentation. |
| 230 |
|
o pkg/autodiff, exf, ptracers & seaice (AD & TL output): |
| 231 |
|
- comment out the very nasty "writeBinaryPrec = writeStatePrec" line, since: |
| 232 |
|
a) can damage previously written records in any other multi-record files. |
| 233 |
|
b) if needed, can easily do it cleanly, by calling S/R WRITE_REC_3D_RL/S. |
| 234 |
|
o tools/genmake2: |
| 235 |
|
- define variable MAKE within Makefile and use it in place of 'make'; |
| 236 |
|
- update AD_CONFIG.h (if needed) before building any of the executable |
| 237 |
|
(forward, adjoint or tangent-lin); this involves a 2nd invocation of make; |
| 238 |
|
and add the usual #ifndef/#define AD_CONFIG_H in AD_CONFIG.h; |
| 239 |
|
- always print usual warning comment 'this file is automatically generated' |
| 240 |
|
from script convert_cpp_cmd2defines (+ remove this warning from genmake2). |
| 241 |
|
o pkg/cheapaml: |
| 242 |
|
- updated to correspond to the code used in accepted paper (Nico Wienders). |
| 243 |
|
o pkg/exf: |
| 244 |
|
added run time switch useAtmWind to replace ALLOW_ATM_WIND cpp switch. |
| 245 |
|
ALLOW_ATM_WIND now just sets the useAtmWind default (see exf_readparms.F) |
| 246 |
|
and force defines ALLOW_BULKFORMULAE (EXF_OPTIONS.h). |
| 247 |
|
o pkg/exf, autodiff, ctrl, ecco and seaice: |
| 248 |
|
remove ALLOW_ATM_WIND brackets, or replace them with useAtmWind ones. |
| 249 |
|
o pkg/ctrl, ecco: |
| 250 |
|
allow to compile both ALLOW_U/VSTRESS_CONTROL and ALLOW_U/VWIND_CONTROL. |
| 251 |
|
Depending on useAtmWind, one is inactive, and the other is active (see |
| 252 |
|
exf_getffields.F/exf_getsurfacefluxes.F). |
| 253 |
|
o tools/genmake2: |
| 254 |
|
- improve error msg when failing in 2nd test of check_for_broken_Ff |
| 255 |
|
- fix option '-makefile' short-name (-mf according to genmake2 -help) |
| 256 |
|
- source local config file "genmake_local" just after processing the genmake2 |
| 257 |
|
command arguments (previously done before processing arg). |
| 258 |
|
o pkg/grdchk: |
| 259 |
|
- add a call to TURNOFF_MODEL_IO for Tangent-Linear run (in this case, call |
| 260 |
|
to this routine from cost_final has been dropped in g_cost_final) |
| 261 |
|
- improve printed information (more explicit msg, no longer using "print *," |
| 262 |
|
fix fcpertminus printed value). |
| 263 |
|
o model/src: |
| 264 |
|
- move call to COST_INIT_VARIA out of packages_init_variables.F directly |
| 265 |
|
into initialise_varia.F (just after calling PACKAGES_INIT_VARIABLES): |
| 266 |
|
this prevents getting cost-funct reset in adinitialise_varia recomputations. |
| 267 |
|
o pkg/dic: |
| 268 |
|
- move initialisation of "freefe" array from from dic_biotic_forcing.F to |
| 269 |
|
fe_chem.F: this prevents TAF from dropping this initialisation in TLM code. |
| 270 |
|
o pkg/grdchk: |
| 271 |
|
- grdchk_main.F : added CALL GLOBAL_SUM_INT( ierr , myThid ) |
| 272 |
|
- grdchk_getadxx.F etc. : omit I/O if ierr.EQ.0 |
| 273 |
|
o pkg/autodiff/ctrl/grdchk : |
| 274 |
|
- attempt to sort out what works and what does not, |
| 275 |
|
with regard to I/O, and to warn/stop accordingly. |
| 276 |
|
o pkg/grdchk: |
| 277 |
|
- attempt to fix grdchk in the case of useSingleCpuIO. |
| 278 |
|
- "if ( myProcId .EQ. grdchkwhichproc .AND. ierr .EQ. 0 )" |
| 279 |
|
statements were moved from grdchk_main.F and into |
| 280 |
|
grdchk_getadxx.F grdchk_getxx.F grdchk_setxx.F. |
| 281 |
|
- a global sum is applied to record numbers icvrec/layer |
| 282 |
|
out of grdchk_loc so that they be passed to I/O routines. |
| 283 |
|
o pkg/smooth: |
| 284 |
|
- fix recomputation that appeared with nlfs : use smooth_recip_hFacC, |
| 285 |
|
smooth_hFacW, smooth_hFacS (copied at init_fixed) in 3d case. |
| 286 |
|
o verification/testreport: |
| 287 |
|
- add option '-tlm' for Tangent-Linear test (-adm,-oad,-tlm are all exclusive); |
| 288 |
|
- can also use compressed (gzip) reference output file; |
| 289 |
|
- change format (in pkg/grdchk) of TLM output (+ add forward gradient) |
| 290 |
|
to be easier to collect by testreport. |
| 291 |
|
- update existing TLM output (half out-of-date by more than 3 yrs). |
| 292 |
|
- report NETCDF & LAPACK settings (from genmake2) in output dir. |
| 293 |
|
- report run-time in each individual summary (previously only for AD test); |
| 294 |
|
for TLM test, add also TAF version. |
| 295 |
|
- add daily TLM test on (old) aces cluster + weekly test on baudelaire. |
| 296 |
|
|
| 297 |
|
checkpoint63r (2012/08/15) |
| 298 |
|
o tools/genmake2: |
| 299 |
|
- fix Makefile for target ftlall (+ update files to clean after make ftlall) |
| 300 |
|
o pkg/ecco/ecco_cost_final.F : |
| 301 |
|
- fix fc print statements (to stdout and costfunction0000) so |
| 302 |
|
that the printed value is always after the global sum. |
| 303 |
|
o pkg/seaice/seaice_dynsolver.F : |
| 304 |
|
- fix ice strength assymptote at SEAICE_area_max<1 |
| 305 |
|
(no change for the standard case of SEAICE_area_max=1) |
| 306 |
|
o some cleaning: |
| 307 |
|
- pkg source files: ensure that all *.F files from each pkg included the |
| 308 |
|
corresponding ${PKG}_OPTIONS.h file (except stand-alone src file which |
| 309 |
|
only includes CPP_EEOPTIONS.h); fix S/R CTRL_SMOOTH ; update AD experiment |
| 310 |
|
code_ad/*.F src files; add new option-file in pkg/admtlm + corresponding |
| 311 |
|
includes; |
| 312 |
|
- model/src: add PACKAGES_CONFIG.h wherever ALLOW_AUTODIFF[_TAMC] is used. |
| 313 |
|
- ALLOW_DIFFKR_CONTROL: add a stop in S/R CTRL_CHECK in case ALLOW_3D_DIFFKR |
| 314 |
|
is undef; also define /DYNVARS_DIFFKR/ common block in DYNVARS.h when |
| 315 |
|
ALLOW_DIFFKR_CONTROL is defined, to allow to compile and reach this stop. |
| 316 |
|
- update option-file CPP_OPTIONS.h of all AD experiments: |
| 317 |
|
* no longer include PACKAGES_CONFIG.h ; |
| 318 |
|
* includes ECCO_CPPOPTIONS.h if PACKAGES_CONFIG.h has been included ; |
| 319 |
|
* def/undef ALLOW_3D_DIFFKR independently of ALLOW_DIFFKR_CONTROL (before |
| 320 |
|
including ECCO_CPPOPTIONS.h). |
| 321 |
|
o pkg/cost, pkg/ctrl, pkg/ecco option-file: |
| 322 |
|
- clean-up and split content of old pkg/ecco/ECCO_CPPOPTIONS.h file into |
| 323 |
|
separated pkg option-file (done according to old doc from ECCO_CPPOPTIONS.h, |
| 324 |
|
not necessary accurate). |
| 325 |
|
- add a template for option-file ECCO_CPPOPTIONS.h, mostly empty, in |
| 326 |
|
pkg/autodiff, i.e., the 1rst pkg to compile for adjoint built, with lines to |
| 327 |
|
deliberately break compilation (a remainder that it needs to be customized) |
| 328 |
|
- in pkg/ecco, includes ECCO_OPTIONS.h instead of COST_OPTIONS.h |
| 329 |
|
- rename COST_CPPOPTIONS.h to COST_OPTIONS.h |
| 330 |
|
- rename CTRL_CPPOPTIONS.h to CTRL_OPTIONS.h |
| 331 |
|
o tools/genamke2: |
| 332 |
|
- check for lapack. |
| 333 |
|
o pkg/cost and pkg/ecco : attempt to sort out some of the limiting intricacies. |
| 334 |
|
- pkg/cost.cost_final.F : allow for use of pkg/ecco and pkg/cost cost functions |
| 335 |
|
jointly. The exclusive ifdefs were removed, to add costs terms (when defined) |
| 336 |
|
from both packages. |
| 337 |
|
- the fc global sum in pkg/ecco/ecco_cost_final.F was removed accordingly. |
| 338 |
|
- for objf_atl we use either the pkg/ecco or the pkg/cost term depending on CPPs. |
| 339 |
|
- pkg/ecco and model/src : add CPP brackets around pkg/cost and pkg/ctrl includes |
| 340 |
|
of "CTRL_SIZE.h" "ctrl.h" "ctrl_dummy.h" "optim.h" / "cost.h" accordingly. |
| 341 |
|
o pkg/cost : add comments to explain special and/or untested setups. |
| 342 |
|
o pkg/exf: |
| 343 |
|
- use EXF_OPTIONS.h to set pkg/exf options except if done in ECCO_CPPOPTIONS.h |
| 344 |
|
(previously, ALLOW_AUTODIFF_TAMC was controling this switch). |
| 345 |
|
o verification/*/code_ad: |
| 346 |
|
mimimal cleaning of AD experiments ECCO_CPPOPTIONS.h option-file: |
| 347 |
|
- reccord when this file is included (define ECCO_CPPOPTIONS_H) |
| 348 |
|
- remove the most obvious obsolete features (e.g., #define ALLOW_COST not |
| 349 |
|
allowed since Oct 2003 in any option-file except in PACKAGES_CONFIG.h) |
| 350 |
|
o pkg/seaice : |
| 351 |
|
- added possibility to change ice strength/PRESS0 relationship to |
| 352 |
|
HEFF, according to the following run time parameters |
| 353 |
|
SEAICEpresPow0 :: HEFF exponent for ice strength below SEAICEpresH0 (integer, >0) |
| 354 |
|
SEAICEpresPow1 :: HEFF exponent for ice strength above SEAICEpresH0 (integer, >0) |
| 355 |
|
SEAICEpresH0 :: HEFF threshold for ice strength (m) |
| 356 |
|
o pkg/ggl90 : |
| 357 |
|
- added run time flag mxlSurfFlag to include the code that ensure |
| 358 |
|
mixing between first and second level (previously included as a comment) |
| 359 |
|
o verification/1D_ocean_ice_column/input_ad/data.autodiff : |
| 360 |
|
- add test of newly added dumpAdByRec=T option where |
| 361 |
|
we write all records to one file per variable. |
| 362 |
|
o verification/global_ocean.cs32x15/code_ad/CPP_OPTIONS.h : |
| 363 |
|
- switch to #define ALLOW_BALANCE_RELAX and ALLOW_BALANCE_FLUXES |
| 364 |
|
to check that this code is ok with taf and compiles in adjoint. |
| 365 |
|
o pkg/autodiff : |
| 366 |
|
- added run time param dumpAdByRec : ad dump format switch. |
| 367 |
|
If true then write all records to one file per variable. |
| 368 |
|
Else write one file per record (remains the default). |
| 369 |
|
- revise seaice_ad_dump.F and exf_adjoint_snapshots_ad.F accordingly. |
| 370 |
|
o pkg/seaice: |
| 371 |
|
- Added SEAICEuseTILT run time switch : if set to |
| 372 |
|
false then omit the tilt term in seaice dynamics. |
| 373 |
|
- Fix free drift calculation. |
| 374 |
|
o pkg/shelfice: |
| 375 |
|
When initializing SHELFICEDragQuadratic, set default to be |
| 376 |
|
consistent with shiCdrag in case SHELFICEuseGammaFrict is used, |
| 377 |
|
but retain option to set it independently |
| 378 |
|
(inconsistenty spotted by Veronique Dansereau) |
| 379 |
|
o eesupp/inc: |
| 380 |
|
- in EEPARAMS.h, add real-type constant for some frequently used simple number |
| 381 |
|
(0,1,2,1/2), with name explicitly referring to RS/RL type. |
| 382 |
|
o model/src: |
| 383 |
|
- move balancing of relaxation terms to new S/R balance_relax.F. |
| 384 |
|
o pkg/autodiff: |
| 385 |
|
re-write active I/O high-level (control) S/R: |
| 386 |
|
- rename S/R ACTIVE_READ/WRITE_RL to ACTIVE_READ/WRITE_3D_RL |
| 387 |
|
- add 1 more argument (useCurrentDir) to replace the "_LOC" 2nd set of S/R. |
| 388 |
|
- use standard pkg/mdsio I/O routines (MDS_READ/WRITE_SEC_XZ/YZ and |
| 389 |
|
MDS_READ/WRITE_FIELD); change loops to multi-threads. |
| 390 |
|
- update and merge active_file_control_slice.flow into active_file_control.flow |
| 391 |
|
- update the calls in lower level S/R (active_file.F, active_file_g.F and |
| 392 |
|
active_file_ad.F) keeping the argument list and function unchanged. |
| 393 |
|
- also update lower-level "_LOC" version (call same S/R with "useCurrentDir |
| 394 |
|
=.TRUE.") and remove high-level (control) _loc version (not called anymore). |
| 395 |
|
- new low-level active I/O routines, one for each type (RL/RS), for active var |
| 396 |
|
of various (XY,XZ,YZ) shape and level number (arguments: arrShape, myNr); |
| 397 |
|
fix ctrl_init_wet.F & ecco_cost_weights.F by calling the new S/R with the |
| 398 |
|
right type. |
| 399 |
|
o autodiff and RS=real*4: |
| 400 |
|
- call the right type (_RS instead of _RL) of S/R in few output S/R (pkg/exf |
| 401 |
|
ad-monitor and snapshots); disable arrType='RS' in mdsio_gl*.F ; |
| 402 |
|
- change argument type of 3-D mask argument to _RS to match type in the call |
| 403 |
|
(in seaice_cost*.F, ctrl_*_gen.F and *ctrl_bound.F); to be continued. |
| 404 |
|
o pkg/thsice: |
| 405 |
|
- re-anable balanceEmPmR when useSEAICE=T with useThSIce=T. |
| 406 |
|
- option to correct ocean fresh-water flux for imbalance of Atmos+Land |
| 407 |
|
fresh-water flux: uniform correction or proportional to local precip. |
| 408 |
|
o pkg/ctrl: |
| 409 |
|
- new header file CTRL_SIZE.h for parameters maxCtrlArr2D, maxCtrlArr3D |
| 410 |
|
- redefine CPP option ALLOW_BOTTOMDRAG_CONTROL to |
| 411 |
|
ALLOW_BOTTOMDRAG_CONTROL_NONGENERIC |
| 412 |
|
- verification/hs94.1x64x5/ tests ALLOW_GENARR{2D,3D}_CONTROL |
| 413 |
|
|
| 414 |
|
checkpoint63q (2012/07/30) |
| 415 |
|
o pkg/autodiff: |
| 416 |
|
note about monitor and snap-shot of ad-variables: |
| 417 |
|
a) need to apply ADEXCH before monitor or output of ad-vars to include |
| 418 |
|
overlap contributions (was done for adjDump but missing for monitor). |
| 419 |
|
b) since ADEXCH implies adding overlap values to interior array, machine |
| 420 |
|
truncation error is expected when adding an ADEXCH call in the middle of |
| 421 |
|
the time-stepping (although mathematically identical). |
| 422 |
|
modifs: |
| 423 |
|
- fix some incorrect type (RL/RS, UV/scalar) in ADEXCH calls and ad-vars |
| 424 |
|
declaration (adEddyPsiX,adEddyPsiY in adcommon.h). |
| 425 |
|
- add new run-time params (dumpAdVarExch & mon_AdVarExch) to select how to |
| 426 |
|
apply ADEXCH before monitor or snap-shot of ad-variables: |
| 427 |
|
=0 : no ADEXCH ; =1 : do ADEXCH ; =2 : apply ADEXCH on a local copy. |
| 428 |
|
- implement new method (copy+adexch) for snap-shot and monitor of main model |
| 429 |
|
ad-vars (initially not used by default -> output unchanged). |
| 430 |
|
- change default (set to 2) to use new method and update few output_adm.txt |
| 431 |
|
with accurate ad-monitor (global_ocean.90x40x15, tutorial_global_oce_biogeo |
| 432 |
|
and both tutorial_tracer_adjsens tests). |
| 433 |
|
- switch 2 global_ocean.cs32x15 AD-tests to AB-2 (seaice_dynmix & thsice) |
| 434 |
|
and turn off momDissip_In_AB in the 2 other (needed for stable AB-3). |
| 435 |
|
o pkg/ctrl: |
| 436 |
|
enable the use of a variable number (maxCtrlArr2D, maxCtrlArr3D) of |
| 437 |
|
generic control variables via |
| 438 |
|
#define ALLOW_GENARR2D_CONTROL and/or #define ALLOW_GENARR3D_CONTROL |
| 439 |
|
Only required customization should be S/R ctrl_map_ini_genarr.F |
| 440 |
|
(routine in repository contains example) |
| 441 |
|
o pkg/exf: |
| 442 |
|
- fix and refine checking of surface-relaxation setting when using EXF |
| 443 |
|
o verification (adjoint exp): |
| 444 |
|
- reduce length of simulation by half for experiment global_ocean.cs32x15 |
| 445 |
|
(std, seaice & seaice_dynmix) and exp tutorial_dic_adjoffline; update output. |
| 446 |
|
o pkg/mom_common (mom_u/v_bottomdrag), pkg/shelfice (shelfice_u/v_drag) |
| 447 |
|
- replace if statements KE.ne.0 with KE.gt.0 to avoid sqrt of negative numbers |
| 448 |
|
of KE that can occur with aggressive optimization |
| 449 |
|
+ put quadratic bottom drag calculation in separated loops (only done if |
| 450 |
|
using bottomDragQuadratic) |
| 451 |
|
o model/src: CG2D_NSA |
| 452 |
|
- clean-up: stop if cg2dNormaliseRHS=T and AUTODIFF |
| 453 |
|
and replace GLOBAL_SUM calls with GLOBAL_SUM_TILE. |
| 454 |
|
- in test-exp that uses CG2D_NSA (bottom_ctrl & tutorial_tracer_adjsens), |
| 455 |
|
change cg2d residual target to be expressed in Wunits (=> do not normalise |
| 456 |
|
RHS ; this fix global_max Pb with TAF); update output. |
| 457 |
|
o verification/tutorial_tracer_adjsens: |
| 458 |
|
improve parameter consistency (and get rid of warnings): |
| 459 |
|
- use deltaTFreeSurf=deltaTtracer (better with NonLin FreeSurf); |
| 460 |
|
- switch Temp Adv Scheme from 81 to 80 (no limiter since theta < 0 is physical) |
| 461 |
|
- use staggerTimeStep (for stability) in som81; use default rotation period. |
| 462 |
|
o pkg/exch2: |
| 463 |
|
- with empty facet: fix compact-format (W2_mapIO=1) definition (was previously |
| 464 |
|
reset to 0 = 1 long line in X) |
| 465 |
|
o pkg/mdsio: |
| 466 |
|
- fix reading of dimensions from meta file for large-size domain |
| 467 |
|
o model/src: |
| 468 |
|
- fix saltIsActiveTr setting for IdealGas eosType (eosType is 6 character long); |
| 469 |
|
- to refine CS-grid check, add logical flag "hasWetCSCorners": true if using |
| 470 |
|
Cubed-Sphere Exch with CS-corners inside the domain; |
| 471 |
|
make new S/R INI_GLOBAL_DOMAIN from code in ini_linear_phisurf.F to calculate |
| 472 |
|
globalArea and to set hasWetCSCorners. |
| 473 |
|
- add warning if usingCurvilinearGrid and momAdvection in flux-form (missing |
| 474 |
|
metric terms); stop if hasWetCSCorners=T and momAdvection in flux-form. |
| 475 |
|
- re-activate the stop if useCDscheme and hasWetCSCorners=T (previously if |
| 476 |
|
useCDscheme and useCubedSphereExchange). |
| 477 |
|
o pkg/seaice/seaice_growth.F : |
| 478 |
|
- compute the SEAICE_DISABLE_HEATCONSFIX term accordingly |
| 479 |
|
for the (.NOT.useRealFreshWaterFlux).OR.(nonlinFreeSurf.EQ.0) |
| 480 |
|
when the previous code was not correct. |
| 481 |
|
- in all cases diagnoze the boundary condition mismatch to SIaaflux |
| 482 |
|
- in the case of useRealFreshWaterFlux.AND.(nonlinFreeSurf.GT.0) add |
| 483 |
|
that term to QNET (essentially to set the melt/freeze water to 0degC). |
| 484 |
|
when SEAICEheatConsFix is true, so that the ocean+ice budget heat is closed. |
| 485 |
|
o verification/global_ocean.cs32x15/input.seaice : |
| 486 |
|
- add SEAICEheatConsFix in data.seaice, ensuring closed ocean+ice heat budget. |
| 487 |
|
- add diagnostics (commented out) for budget test using my matlab codes (gcmfaces). |
| 488 |
|
- update results (output.seaice.txt) accordingly. |
| 489 |
|
o verification/lab_sea/input/data.seaice : |
| 490 |
|
- remove SEAICEheatConsFix, which is now be tested |
| 491 |
|
in global_ocean.cs32x15. The presently coded fix |
| 492 |
|
is only consistent with real fresh water + nlfs. |
| 493 |
|
- update results (output.txt) accordingly. |
| 494 |
|
o pkg/salt_plume: added diagnotic for salt tendency (oceSPtnd). |
| 495 |
|
o AD_CONFIG.h file: |
| 496 |
|
- in genmake2: remove "#undef ALLOW_ECCO_OPTIMIZATION" from AD_CONFIG.h; |
| 497 |
|
- remove #include "AD_CONFIG.h" from all included OPTIONS files and |
| 498 |
|
add it explicitly in each fortran src file where it is needed. |
| 499 |
|
motivation: header file AD_CONFIG.h is generated/modified after "make depend" |
| 500 |
|
thus introducing potentially wrong dependency; therefore, better to include |
| 501 |
|
it only where it's really needed, and after any other included OPTIONS file |
| 502 |
|
(less tempting to bring conditional includes according to ALLOW_ADJOINT_RUN |
| 503 |
|
or ALLOW_TANGENTLINEAR_RUN). |
| 504 |
|
|
| 505 |
|
checkpoint63p (2012/07/06) |
| 506 |
|
o pkg/autodiff: |
| 507 |
|
- rename S/R INI_AUTODIFF to AUTODIFF_INIT_VARIA ; |
| 508 |
|
- create new header file "AUTODIFF_PARAMS.h" for pkg/autodiff parameters |
| 509 |
|
which are now read from new parameter file "data.autodiff" (inAdExact |
| 510 |
|
previously in "data", use{KPP/GMRedi/SEAICE}inAdMode previously in data.pkg) |
| 511 |
|
- add file "data.autodiff" in all verification/*/input_ad/ directories. |
| 512 |
|
- add run-time parameter to turn On/Off SEAICEuseFREEDRIFT in adjoint/forward |
| 513 |
|
mode (used in global_ocean.cs32x15.seaice_dynmix test exp.) |
| 514 |
|
o verification/tutorial_tracer_adjsens/ |
| 515 |
|
- add extra sub-verification which uses "SOM" advection scheme |
| 516 |
|
(2nd Order-Moment Advection Scheme, Prather Limiter) with |
| 517 |
|
#define GAD_ALLOW_TS_SOM_ADV in GAD_OPTIONS.h |
| 518 |
|
and in data: |
| 519 |
|
multiDimAdvection=.TRUE., |
| 520 |
|
tempAdvScheme=81, |
| 521 |
|
saltAdvScheme=81, |
| 522 |
|
o model/src: |
| 523 |
|
- introduce ALLOW_BALANCE_RELAX which allow the removal |
| 524 |
|
of the global mean of relaxation terms by setting |
| 525 |
|
balanceThetaClimRelax and balanceSaltClimRelax |
| 526 |
|
- disable balanceEmPmR and balanceQnet in the case when useSeaice. |
| 527 |
|
This case is now treated appropriately in seaice_growth.F |
| 528 |
|
o pkg/seaice: |
| 529 |
|
- do balanceEmPmR and balanceQnet based on the net fluxes |
| 530 |
|
(SItflux) that enter the ocean+ice system rather than QNET |
| 531 |
|
and EMPMR. Works best in real fresh water. |
| 532 |
|
- SIatmFW, SIatmQnet, and SItflux computations had to be moved |
| 533 |
|
out of the diagnotics bloc, and stored in global arrays. |
| 534 |
|
|
| 535 |
|
checkpoint63o (2012/06/29) |
| 536 |
|
o vertical mixing schemes: |
| 537 |
|
- remove unused S/R CALC_BUOYANCY (left from branch-atmos merging) |
| 538 |
|
- replace computation of density (in Function STATE1, opps_calc.F) by a call |
| 539 |
|
to S/R FIND_RHO_SCALAR |
| 540 |
|
- pass sigmaR to S/R PP81_CALC, MY82_CALC (but not yet used) and to GGL90_CALC |
| 541 |
|
and use it for N^2 calculation in ggl90_calc.F (save 2 density computations); |
| 542 |
|
affects output (vermix.ggl90) at truncation level. |
| 543 |
|
o testreport: |
| 544 |
|
- change adjustment of SIZE.h for mpi+mth testing: preserve enough tiles |
| 545 |
|
(nSx,nSy) for the number of threads (from eedata.mth) |
| 546 |
|
o model/src: |
| 547 |
|
- remove 2nd set of calls at the top of initialise_varia.F (Adjoint part) |
| 548 |
|
related to Non_Lin Free-Surf or Bottom-Control (but not longer necessary). |
| 549 |
|
o verification/tutorial_tracer_adjsens/ |
| 550 |
|
- compile with ALLOW_CG2D_NSA |
| 551 |
|
- run with nonlinFreeSurf=3 |
| 552 |
|
- update reference output_adm.txt |
| 553 |
|
o tools/genmake2: |
| 554 |
|
- add "-DHAVE_NETCDF" to the DEFINES list when NetCDF is available ; |
| 555 |
|
refine usePROFILES switching off (only when NetCDF is not available). |
| 556 |
|
o verification (atmospheric set-up): |
| 557 |
|
- change eosType='IDEALGAS' to eosType='IDEALG' to match variable declaration |
| 558 |
|
o pkg/profiles : refine profiles and files handling. |
| 559 |
|
o model/src and pkg/ecco : introduce run-time switch usePROFILES. |
| 560 |
|
o model/src/the_main_loop.F, forward_step.F and pkg/exf/EXF_OPTIONS.h : |
| 561 |
|
- changes to use model/src/the_main_loop.F rather than the ecco version |
| 562 |
|
in ecco type adjoint runs. |
| 563 |
|
o model/src: |
| 564 |
|
- move computation of recip of grid distance & area from INI_MASKS_ETC to INI_GRID |
| 565 |
|
- move setting of h0Fac[C,W,S] from INI_LINEAR_PHISURF to INI_MASKS_ETC |
| 566 |
|
- update_masks_etc.F: rename SMOOTH*_R4,R8 function to the corresponding type (RS,RL) |
| 567 |
|
o model/src: |
| 568 |
|
- fix S/R UPDATE_CG2D for ALLOW_DEPTH_CONTROL case (was empty before); |
| 569 |
|
update output of test experiment bottom_ctrl_5x5. |
| 570 |
|
o pkg/generic_advdiff: |
| 571 |
|
- implement vertical bi-harmonic diffusion (for now, only for T & S and |
| 572 |
|
only explicitly). |
| 573 |
|
o verification/bottom_ctrl_5x5/ |
| 574 |
|
update reference output (results changed quite some time ago, |
| 575 |
|
not sure when; but new reference results look ok in terms of grdchk) |
| 576 |
|
o verification/1D_ocean_ice_column/ |
| 577 |
|
For verification purposes, the exact adjoint rather than an approx. |
| 578 |
|
should be used (e.g., to comare against F.D. or TLM). |
| 579 |
|
removed autodiff_inadmode*_ad.F routines in code_ad/ |
| 580 |
|
Update results/ accordingly. |
| 581 |
|
o pkg/cal: |
| 582 |
|
- change cal_getdate.F so that output date is function of input time only |
| 583 |
|
(less ambiguous, not mixed up with input iteration number). |
| 584 |
|
- fix calendarDumps for non-zero phase-lag (S/R CAL_TIME2DUMP). |
| 585 |
|
o pkg/diagnostics: |
| 586 |
|
- implement calendarDumps also for stats-diags |
| 587 |
|
- implement calendarDumps also for snap-shot output (negative freq) |
| 588 |
|
- for now, ignores calendarDumps if non-zero phase-lag. |
| 589 |
|
|
| 590 |
checkpoint63n (2012/06/04) |
checkpoint63n (2012/06/04) |
| 591 |
o model/src: |
o model/src: |
| 592 |
- recompute rSurfW & S to ensure consistent column thickness (needed for rStar) |
- recompute rSurfW & S to ensure consistent column thickness (needed for rStar) |
Parent Directory
| 
Revision Log
| 
Revision Graph
| 
Patch