| 4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
| 5 |
============================== |
============================== |
| 6 |
|
|
| 7 |
|
o pkg/ctrl: |
| 8 |
|
- new header file CTRL_SIZE.h for parameters maxCtrlArr2D, maxCtrlArr3D |
| 9 |
|
- redefine CPP option ALLOW_BOTTOMDRAG_CONTROL to |
| 10 |
|
ALLOW_BOTTOMDRAG_CONTROL_NONGENERIC |
| 11 |
|
- verification/hs94.1x64x5/ tests ALLOW_GENARR{2D,3D}_CONTROL |
| 12 |
|
checkpoint63q (2012/07/30) |
| 13 |
|
o pkg/autodiff: |
| 14 |
|
note about monitor and snap-shot of ad-variables: |
| 15 |
|
a) need to apply ADEXCH before monitor or output of ad-vars to include |
| 16 |
|
overlap contributions (was done for adjDump but missing for monitor). |
| 17 |
|
b) since ADEXCH implies adding overlap values to interior array, machine |
| 18 |
|
truncation error is expected when adding an ADEXCH call in the middle of |
| 19 |
|
the time-stepping (although mathematically identical). |
| 20 |
|
modifs: |
| 21 |
|
- fix some incorrect type (RL/RS, UV/scalar) in ADEXCH calls and ad-vars |
| 22 |
|
declaration (adEddyPsiX,adEddyPsiY in adcommon.h). |
| 23 |
|
- add new run-time params (dumpAdVarExch & mon_AdVarExch) to select how to |
| 24 |
|
apply ADEXCH before monitor or snap-shot of ad-variables: |
| 25 |
|
=0 : no ADEXCH ; =1 : do ADEXCH ; =2 : apply ADEXCH on a local copy. |
| 26 |
|
- implement new method (copy+adexch) for snap-shot and monitor of main model |
| 27 |
|
ad-vars (initially not used by default -> output unchanged). |
| 28 |
|
- change default (set to 2) to use new method and update few output_adm.txt |
| 29 |
|
with accurate ad-monitor (global_ocean.90x40x15, tutorial_global_oce_biogeo |
| 30 |
|
and both tutorial_tracer_adjsens tests). |
| 31 |
|
- switch 2 global_ocean.cs32x15 AD-tests to AB-2 (seaice_dynmix & thsice) |
| 32 |
|
and turn off momDissip_In_AB in the 2 other (needed for stable AB-3). |
| 33 |
|
o pkg/ctrl: |
| 34 |
|
enable the use of a variable number (maxCtrlArr2D, maxCtrlArr3D) of |
| 35 |
|
generic control variables via |
| 36 |
|
#define ALLOW_GENARR2D_CONTROL and/or #define ALLOW_GENARR3D_CONTROL |
| 37 |
|
Only required customization should be S/R ctrl_map_ini_genarr.F |
| 38 |
|
(routine in repository contains example) |
| 39 |
|
o pkg/exf: |
| 40 |
|
- fix and refine checking of surface-relaxation setting when using EXF |
| 41 |
|
o verification (adjoint exp): |
| 42 |
|
- reduce length of simulation by half for experiment global_ocean.cs32x15 |
| 43 |
|
(std, seaice & seaice_dynmix) and exp tutorial_dic_adjoffline; update output. |
| 44 |
|
o pkg/mom_common (mom_u/v_bottomdrag), pkg/shelfice (shelfice_u/v_drag) |
| 45 |
|
- replace if statements KE.ne.0 with KE.gt.0 to avoid sqrt of negative numbers |
| 46 |
|
of KE that can occur with aggressive optimization |
| 47 |
|
+ put quadratic bottom drag calculation in separated loops (only done if |
| 48 |
|
using bottomDragQuadratic) |
| 49 |
|
o model/src: CG2D_NSA |
| 50 |
|
- clean-up: stop if cg2dNormaliseRHS=T and AUTODIFF |
| 51 |
|
and replace GLOBAL_SUM calls with GLOBAL_SUM_TILE. |
| 52 |
|
- in test-exp that uses CG2D_NSA (bottom_ctrl & tutorial_tracer_adjsens), |
| 53 |
|
change cg2d residual target to be expressed in Wunits (=> do not normalise |
| 54 |
|
RHS ; this fix global_max Pb with TAF); update output. |
| 55 |
|
o verification/tutorial_tracer_adjsens: |
| 56 |
|
improve parameter consistency (and get rid of warnings): |
| 57 |
|
- use deltaTFreeSurf=deltaTtracer (better with NonLin FreeSurf); |
| 58 |
|
- switch Temp Adv Scheme from 81 to 80 (no limiter since theta < 0 is physical) |
| 59 |
|
- use staggerTimeStep (for stability) in som81; use default rotation period. |
| 60 |
|
o pkg/exch2: |
| 61 |
|
- with empty facet: fix compact-format (W2_mapIO=1) definition (was previously |
| 62 |
|
reset to 0 = 1 long line in X) |
| 63 |
|
o pkg/mdsio: |
| 64 |
|
- fix reading of dimensions from meta file for large-size domain |
| 65 |
|
o model/src: |
| 66 |
|
- fix saltIsActiveTr setting for IdealGas eosType (eosType is 6 character long); |
| 67 |
|
- to refine CS-grid check, add logical flag "hasWetCSCorners": true if using |
| 68 |
|
Cubed-Sphere Exch with CS-corners inside the domain; |
| 69 |
|
make new S/R INI_GLOBAL_DOMAIN from code in ini_linear_phisurf.F to calculate |
| 70 |
|
globalArea and to set hasWetCSCorners. |
| 71 |
|
- add warning if usingCurvilinearGrid and momAdvection in flux-form (missing |
| 72 |
|
metric terms); stop if hasWetCSCorners=T and momAdvection in flux-form. |
| 73 |
|
- re-activate the stop if useCDscheme and hasWetCSCorners=T (previously if |
| 74 |
|
useCDscheme and useCubedSphereExchange). |
| 75 |
|
o pkg/seaice/seaice_growth.F : |
| 76 |
|
- compute the SEAICE_DISABLE_HEATCONSFIX term accordingly |
| 77 |
|
for the (.NOT.useRealFreshWaterFlux).OR.(nonlinFreeSurf.EQ.0) |
| 78 |
|
when the previous code was not correct. |
| 79 |
|
- in all cases diagnoze the boundary condition mismatch to SIaaflux |
| 80 |
|
- in the case of useRealFreshWaterFlux.AND.(nonlinFreeSurf.GT.0) add |
| 81 |
|
that term to QNET (essentially to set the melt/freeze water to 0degC). |
| 82 |
|
when SEAICEheatConsFix is true, so that the ocean+ice budget heat is closed. |
| 83 |
|
o verification/global_ocean.cs32x15/input.seaice : |
| 84 |
|
- add SEAICEheatConsFix in data.seaice, ensuring closed ocean+ice heat budget. |
| 85 |
|
- add diagnostics (commented out) for budget test using my matlab codes (gcmfaces). |
| 86 |
|
- update results (output.seaice.txt) accordingly. |
| 87 |
|
o verification/lab_sea/input/data.seaice : |
| 88 |
|
- remove SEAICEheatConsFix, which is now be tested |
| 89 |
|
in global_ocean.cs32x15. The presently coded fix |
| 90 |
|
is only consistent with real fresh water + nlfs. |
| 91 |
|
- update results (output.txt) accordingly. |
| 92 |
|
o pkg/salt_plume: added diagnotic for salt tendency (oceSPtnd). |
| 93 |
|
o AD_CONFIG.h file: |
| 94 |
|
- in genmake2: remove "#undef ALLOW_ECCO_OPTIMIZATION" from AD_CONFIG.h; |
| 95 |
|
- remove #include "AD_CONFIG.h" from all included OPTIONS files and |
| 96 |
|
add it explicitly in each fortran src file where it is needed. |
| 97 |
|
motivation: header file AD_CONFIG.h is generated/modified after "make depend" |
| 98 |
|
thus introducing potentially wrong dependency; therefore, better to include |
| 99 |
|
it only where it's really needed, and after any other included OPTIONS file |
| 100 |
|
(less tempting to bring conditional includes according to ALLOW_ADJOINT_RUN |
| 101 |
|
or ALLOW_TANGENTLINEAR_RUN). |
| 102 |
|
|
| 103 |
|
checkpoint63p (2012/07/06) |
| 104 |
|
o pkg/autodiff: |
| 105 |
|
- rename S/R INI_AUTODIFF to AUTODIFF_INIT_VARIA ; |
| 106 |
|
- create new header file "AUTODIFF_PARAMS.h" for pkg/autodiff parameters |
| 107 |
|
which are now read from new parameter file "data.autodiff" (inAdExact |
| 108 |
|
previously in "data", use{KPP/GMRedi/SEAICE}inAdMode previously in data.pkg) |
| 109 |
|
- add file "data.autodiff" in all verification/*/input_ad/ directories. |
| 110 |
|
- add run-time parameter to turn On/Off SEAICEuseFREEDRIFT in adjoint/forward |
| 111 |
|
mode (used in global_ocean.cs32x15.seaice_dynmix test exp.) |
| 112 |
|
o verification/tutorial_tracer_adjsens/ |
| 113 |
|
- add extra sub-verification which uses "SOM" advection scheme |
| 114 |
|
(2nd Order-Moment Advection Scheme, Prather Limiter) with |
| 115 |
|
#define GAD_ALLOW_TS_SOM_ADV in GAD_OPTIONS.h |
| 116 |
|
and in data: |
| 117 |
|
multiDimAdvection=.TRUE., |
| 118 |
|
tempAdvScheme=81, |
| 119 |
|
saltAdvScheme=81, |
| 120 |
|
o model/src: |
| 121 |
|
- introduce ALLOW_BALANCE_RELAX which allow the removal |
| 122 |
|
of the global mean of relaxation terms by setting |
| 123 |
|
balanceThetaClimRelax and balanceSaltClimRelax |
| 124 |
|
- disable balanceEmPmR and balanceQnet in the case when useSeaice. |
| 125 |
|
This case is now treated appropriately in seaice_growth.F |
| 126 |
|
o pkg/seaice: |
| 127 |
|
- do balanceEmPmR and balanceQnet based on the net fluxes |
| 128 |
|
(SItflux) that enter the ocean+ice system rather than QNET |
| 129 |
|
and EMPMR. Works best in real fresh water. |
| 130 |
|
- SIatmFW, SIatmQnet, and SItflux computations had to be moved |
| 131 |
|
out of the diagnotics bloc, and stored in global arrays. |
| 132 |
|
|
| 133 |
|
checkpoint63o (2012/06/29) |
| 134 |
|
o vertical mixing schemes: |
| 135 |
|
- remove unused S/R CALC_BUOYANCY (left from branch-atmos merging) |
| 136 |
|
- replace computation of density (in Function STATE1, opps_calc.F) by a call |
| 137 |
|
to S/R FIND_RHO_SCALAR |
| 138 |
|
- pass sigmaR to S/R PP81_CALC, MY82_CALC (but not yet used) and to GGL90_CALC |
| 139 |
|
and use it for N^2 calculation in ggl90_calc.F (save 2 density computations); |
| 140 |
|
affects output (vermix.ggl90) at truncation level. |
| 141 |
|
o testreport: |
| 142 |
|
- change adjustment of SIZE.h for mpi+mth testing: preserve enough tiles |
| 143 |
|
(nSx,nSy) for the number of threads (from eedata.mth) |
| 144 |
|
o model/src: |
| 145 |
|
- remove 2nd set of calls at the top of initialise_varia.F (Adjoint part) |
| 146 |
|
related to Non_Lin Free-Surf or Bottom-Control (but not longer necessary). |
| 147 |
|
o verification/tutorial_tracer_adjsens/ |
| 148 |
|
- compile with ALLOW_CG2D_NSA |
| 149 |
|
- run with nonlinFreeSurf=3 |
| 150 |
|
- update reference output_adm.txt |
| 151 |
|
o tools/genmake2: |
| 152 |
|
- add "-DHAVE_NETCDF" to the DEFINES list when NetCDF is available ; |
| 153 |
|
refine usePROFILES switching off (only when NetCDF is not available). |
| 154 |
|
o verification (atmospheric set-up): |
| 155 |
|
- change eosType='IDEALGAS' to eosType='IDEALG' to match variable declaration |
| 156 |
|
o pkg/profiles : refine profiles and files handling. |
| 157 |
|
o model/src and pkg/ecco : introduce run-time switch usePROFILES. |
| 158 |
|
o model/src/the_main_loop.F, forward_step.F and pkg/exf/EXF_OPTIONS.h : |
| 159 |
|
- changes to use model/src/the_main_loop.F rather than the ecco version |
| 160 |
|
in ecco type adjoint runs. |
| 161 |
|
o model/src: |
| 162 |
|
- move computation of recip of grid distance & area from INI_MASKS_ETC to INI_GRID |
| 163 |
|
- move setting of h0Fac[C,W,S] from INI_LINEAR_PHISURF to INI_MASKS_ETC |
| 164 |
|
- update_masks_etc.F: rename SMOOTH*_R4,R8 function to the corresponding type (RS,RL) |
| 165 |
|
o model/src: |
| 166 |
|
- fix S/R UPDATE_CG2D for ALLOW_DEPTH_CONTROL case (was empty before); |
| 167 |
|
update output of test experiment bottom_ctrl_5x5. |
| 168 |
|
o pkg/generic_advdiff: |
| 169 |
|
- implement vertical bi-harmonic diffusion (for now, only for T & S and |
| 170 |
|
only explicitly). |
| 171 |
|
o verification/bottom_ctrl_5x5/ |
| 172 |
|
update reference output (results changed quite some time ago, |
| 173 |
|
not sure when; but new reference results look ok in terms of grdchk) |
| 174 |
|
o verification/1D_ocean_ice_column/ |
| 175 |
|
For verification purposes, the exact adjoint rather than an approx. |
| 176 |
|
should be used (e.g., to comare against F.D. or TLM). |
| 177 |
|
removed autodiff_inadmode*_ad.F routines in code_ad/ |
| 178 |
|
Update results/ accordingly. |
| 179 |
|
o pkg/cal: |
| 180 |
|
- change cal_getdate.F so that output date is function of input time only |
| 181 |
|
(less ambiguous, not mixed up with input iteration number). |
| 182 |
|
- fix calendarDumps for non-zero phase-lag (S/R CAL_TIME2DUMP). |
| 183 |
|
o pkg/diagnostics: |
| 184 |
|
- implement calendarDumps also for stats-diags |
| 185 |
|
- implement calendarDumps also for snap-shot output (negative freq) |
| 186 |
|
- for now, ignores calendarDumps if non-zero phase-lag. |
| 187 |
|
|
| 188 |
|
checkpoint63n (2012/06/04) |
| 189 |
|
o model/src: |
| 190 |
|
- recompute rSurfW & S to ensure consistent column thickness (needed for rStar) |
| 191 |
|
between Sum_k(hFac*drF) and rSurf-rLow, for now only when using pkg/shelfice; |
| 192 |
|
- use h0FacC, W & S to compute hFac_surfC, W & S in calc_surf_dr.F (more |
| 193 |
|
similar to calc_r_star.F); update results of experiment internal_wave and |
| 194 |
|
tutorial_global_oce_in_p (affected @ truncation level). |
| 195 |
|
o pkg/shelfice: |
| 196 |
|
- add parameterisation of advective-diffusive flux into the ice shelf |
| 197 |
|
according to Holland and Jenkins (1999), eq.22-33, as suggested by Jan De Ryd |
| 198 |
|
- simply eps3/3a computation |
| 199 |
|
o eesupp: |
| 200 |
|
- add new CPP option (DISCONNECTED_TILES) to disconnect tiles (no exchange |
| 201 |
|
between tiles, just fill-in edges assuming locally periodic subdomain) and |
| 202 |
|
add corresponding exchange subroutines (EXCH0); |
| 203 |
|
- new CG-solver version (_EX0) for disconnected-tiles special case. |
| 204 |
|
o model/src (CG-solvers): |
| 205 |
|
- modify Single-Reduction version (CG2D_SR) in case cg2dMaxIters is reached |
| 206 |
|
(similar final residual output and max-Iter number as with standard version); |
| 207 |
|
- allows to store and use lowest-residual solution (standard & _SR versions); |
| 208 |
|
- cleaning up (remove old, commented out code, LETS_MAKE_JAM ...). |
| 209 |
|
- by default, also compile CG2D_SR version (define ALLOW_SRCG in CPP_OPTIONS.h) |
| 210 |
|
o eesup/src: |
| 211 |
|
- global_vec_sum.F cleaning: add a BARRIER ; update comments ; remove ndef |
| 212 |
|
ALWAYS_USE_MPI (no longer needed after fixing usingMPI settings - March 2012) |
| 213 |
|
o pkg/salt_plume added to lab_sea/input_ad.noseaicedyn |
| 214 |
|
|
| 215 |
|
checkpoint63m (2012/04/27) |
| 216 |
|
o pkg/shelfice: |
| 217 |
|
- account for partical cells in surface level below ice-shelf for hydrostatic |
| 218 |
|
pressure calculion (set uniformFreeSurfLev to F); update isomip experiments. |
| 219 |
|
o ad test experiments: |
| 220 |
|
- update and rename to "ad_optfile.local" local AD-optfile specific to a |
| 221 |
|
particular adjoint verification experiment (in verification/*/code_ad/) |
| 222 |
|
o pkg/seaice & pkg/exf: |
| 223 |
|
- Part two of ECCO-CLIVAR code merges. |
| 224 |
|
Completes merge of seaice area relax. code. |
| 225 |
|
Requires unified CPP option EXF_ALLOW_SEAICE_RELAX |
| 226 |
|
to be set in EXF_OPTIONS.h since this pkg controls its I/O |
| 227 |
|
(and avoids second CPP option in SEAICE_OPTIONS.h) |
| 228 |
|
- change default value of costIceStart & costIceEnd to a valid date. |
| 229 |
|
- add snowPrecip to HSNOW in seaice_growth |
| 230 |
|
o pkg/ptracers: |
| 231 |
|
- move ptracers summary and setup check into separated S/R ; add a check for |
| 232 |
|
valid PTRACERS_Iter0 value (error+stop or just warning). |
| 233 |
|
o pkg/cal (+ pkg/ecco): |
| 234 |
|
- avoid calling a S/R with 2 times the same arg (FWD: unless both are only used |
| 235 |
|
as input; and for AD, unless both are not differentiable (e.g., k index)). |
| 236 |
|
o model/src: |
| 237 |
|
- define internal flag "uniformFreeSurfLev" (=T when free-surface level-index |
| 238 |
|
is everywhere k=1) to speed-up some calculations (by pass kSurf reference); |
| 239 |
|
temporary set to usingZCoords value (to be changed to F when using shelfice). |
| 240 |
|
- hydrostatic pressure: implement partial cell near the surface also for |
| 241 |
|
Z-coordinates (for the case where uniformFreeSurfLev=F); replace test on |
| 242 |
|
usingZCoords with test on uniformFreeSurfLev where needed (in NonHyd code). |
| 243 |
|
o pkg/ctrl/ctrl_init.F: |
| 244 |
|
- add a formal parameter to s/r ctrl_init_rec to print |
| 245 |
|
xx_$(ctrl_valiable)_file for easier debugging |
| 246 |
|
- move call of cal_* routines into the if-block where they are really |
| 247 |
|
needed so that xx_$(ctrl_variable)_period can be 0 or -12 without having |
| 248 |
|
to specify a startdate |
| 249 |
|
o pkg/seaice: |
| 250 |
|
- update comments in SEAICE_OPTIONS.h (+ similar update of lab_sea/code and |
| 251 |
|
global_ocean.cs32x15/code versions). |
| 252 |
|
- turn on seaice Multi-category (in data.seaice, SEAICE_multDim=7) for |
| 253 |
|
verification experiment global_ocean.cs32x15.seaice and update results. |
| 254 |
|
o pkg/exf: |
| 255 |
|
- new S/R EXF_GETFIELD_START to return starting-time of a forcing field |
| 256 |
|
(according to useExfYearlyFields). |
| 257 |
|
o pkg/cal: |
| 258 |
|
- move calendar setting & summary (previously in cal_readparams.F) into new |
| 259 |
|
S/R CAL_INIT_FIXED to be called later in the initialisation process, once |
| 260 |
|
length of the simulation is set (after SET_PARMS call); This is an attempt |
| 261 |
|
to fix Pb when, e.g.: pkg/longstep, modifies the length of the run. |
| 262 |
|
- For safety: keep a record of pkg/cal parameter settings status |
| 263 |
|
to allow to STOP if one pkg/cal S/R is called too early. |
| 264 |
|
- add a stop in all basic pkg/cal subroutines if called before setting is done. |
| 265 |
|
- modify cal_getdate.F: stop when nothing was returned; |
| 266 |
|
if called with myIter=-1, simply retrun modelBaseDate (1,2 = startDate_1,_2); |
| 267 |
|
this allows to remove "cal.h" from seaice_read_parms.F & init_mnc_vars.F. |
| 268 |
|
|
| 269 |
|
checkpoint63l (2012/04/04) |
| 270 |
|
o pkg/bbl: |
| 271 |
|
- fix call to FIND_RHO_2D in bbl_calc_rho.F (case multi-tiles per proc); |
| 272 |
|
update global_with_exf output.yearly.txt file. |
| 273 |
|
o pkg/cal: |
| 274 |
|
- start to clean-up cal_set.F ; fix for the case where baseTime is non-zero. |
| 275 |
|
- remove 2 unsused and very short (1 line only) functions ; |
| 276 |
|
- remove unsused subroutines from AD-Diff list. |
| 277 |
|
o model/src: |
| 278 |
|
- start to implement clean wait to stop in initialization S/R (config_check.F, |
| 279 |
|
packages_check.F, ini_parms.F and pkg/obcs/obcs_check.F + pkg/mypackage) |
| 280 |
|
o folowing PRINT_MESSAGE changes: |
| 281 |
|
- remove reminder multiple print to the same file (STDOUT) when multi-threads. |
| 282 |
|
o pkg/bbl: |
| 283 |
|
- modification so it can work with z* |
| 284 |
|
- replace deltaT with dTtracerLev(k) for asynchronous time stepping |
| 285 |
|
- global_with_exf/input.yearly verification; changes output.yearly.txt |
| 286 |
|
o eesupp/src: |
| 287 |
|
- PRINT_MESSAGE: use OpenMP critical instruction (similar to PRINT_ERROR); |
| 288 |
|
writing to unit 0 (if unit=errorMessageUnit) extended to the case myThid=1 |
| 289 |
|
- CHECK_THREADS: specific code if using OpenMP threading: improve checking of |
| 290 |
|
# of threads across processors (e.g., different env OMP_NUM_THREADS setting) |
| 291 |
|
- improve special preliminary call to EESET_PARMS (if eedata is read 2 times). |
| 292 |
|
- when using MPI, always set usingMPI=T ; set default accordingly and stop |
| 293 |
|
if not consistent. |
| 294 |
|
- ALL_PROC_DIE: add a BARRIER after MPI_FINALIZE call ; skip this when thread |
| 295 |
|
argument is zero (in case called within single-thread section). |
| 296 |
|
o pkg/fizhi: |
| 297 |
|
clean-up turbulence cold-start switch: decided in fizhi_init_vars.F, stored |
| 298 |
|
in common bloc (fizhi_coms.h) and then passed as argument up to S/R TURBIO. |
| 299 |
|
o pkg/exch2: |
| 300 |
|
- add argument "updateCorners" to S/R exch2_get_uv_bounds (enable to |
| 301 |
|
activate EXCH_IGNORE_CORNERS in vector EXCH S/R). |
| 302 |
|
- switch 1rst EXCH call to IGNORE_CORNERS (instead of UPDATE_CORNERS) |
| 303 |
|
but keep the 2nd with UPDATE_CORNERS: this prevents overwriting good data |
| 304 |
|
with bad ones coming from a not-yet-updated halo. |
| 305 |
|
This should fix the problem when sNx <> sNy and GCD(sNx,sNy) < 2*Olx. |
| 306 |
|
o pkg/frazil: |
| 307 |
|
move FrzForcT diag from frazil_diagnostics_state.F to frazil_calc_rhs.F |
| 308 |
|
http://mitgcm.org/pipermail/mitgcm-devel/2012-March/005384.html |
| 309 |
|
o model/src/config_check.F: add a check for OLx>sNx/OLy>sNy |
| 310 |
|
o verification/lab_sea |
| 311 |
|
* #undef SEAICE_EXCLUDE_FOR_EXACT_AD_TESTING |
| 312 |
|
* #define ALLOW_SST_CONTROL & ALLOW_SST_CONTROL |
| 313 |
|
* nTimeSteps 3 -> 4 |
| 314 |
|
* changed reference output accordingly |
| 315 |
|
o pkg/ctrl: |
| 316 |
|
* fix ctrl_get_gen for xx_sst, xx_sss |
| 317 |
|
o pkg/fizhi: |
| 318 |
|
- fix 2 calls to S/R GRD2MSC (argument igrd) in update_earth_exports.F |
| 319 |
|
update results of exp. fizhi-cs-32x32x40 (fail with only 6 digits for cg2d) |
| 320 |
|
- by default, use new version of S/R GETPWHERE (#define TRY_NEW_GETPWHERE); |
| 321 |
|
fix few out-of-bounds indices in referencing vars (initialise igrd to 1); |
| 322 |
|
clean-up fizhi_turb.F (remove unused & uninitialised qdiaglsm array). |
| 323 |
|
o pkg/gmredi: |
| 324 |
|
- add diagnostic of Redi (main diagnonal) vertical diffusive flux of Temp |
| 325 |
|
o pkg/fizhi: |
| 326 |
|
- fix few initialisation and multi-tile Pb: aqualev & gridalt now pass the |
| 327 |
|
(old)aces open64 tuv test. |
| 328 |
|
o pkg/icefront: fix bug and reorder loop in icefront_init_varia.F |
| 329 |
|
o pkg/mom_vecinv: |
| 330 |
|
- fix 1 loop range in mom_vecinv.F to be like all the other loops which |
| 331 |
|
set/update gu,gv/Diss (fix Pb of uninitialised fVerUkp in admom_vecinv |
| 332 |
|
with non-lin free-surf). |
| 333 |
|
o model/src: move k-loop outside of i/j-loops when (re-)computing R_low |
| 334 |
|
in ini/update_masks_etc.F in order to avoid -O3 optimization problems with |
| 335 |
|
some compilers (gfortran 4.6.0 on MacOS) |
| 336 |
|
o momentum: |
| 337 |
|
- separate fVer?(:,:,kUp) & fVer?(:,:,kDown) in argument list of MOM_FLUXFORM |
| 338 |
|
& MOM_VECINV subroutines (to help TAF). This fixes Pb of uninitialised |
| 339 |
|
fVer?(kUp) in admom_fluxform with non-lin free-surf or bottom control. |
| 340 |
|
|
| 341 |
|
checkpoint63k (2012/03/17) |
| 342 |
|
o model/src: |
| 343 |
|
- selectAddFluid (ALLOW_ADDFLUID): add missing term in horizontal momentum |
| 344 |
|
flux-form (mom_fluxform.F) and vertical momentum (calc_gw.F) equations. |
| 345 |
|
- SOLVE_DIAGONAL: switch default to use adjoinable & vectorized version |
| 346 |
|
(#undef SOLVE_DIAGONAL_LOWMEMORY); update output of exp. front_relax.bvp. |
| 347 |
|
o model/src: solve_*diagonal.F |
| 348 |
|
- replace division with multiplication by inverse (similar to original version) |
| 349 |
|
update output of AD exp. global_ocean.90x40x15. |
| 350 |
|
- fix pkg/gmredi/gmredi_calc_psi_bvp.F & pkg/ggl90/ggl90_calc.F |
| 351 |
|
to enable the use of the vectorized & differentiable solve_tridiagonal.F |
| 352 |
|
- test exp. global_ocean.90x40x15/input_ad: switch to advection scheme 20 for |
| 353 |
|
vertical advection (to get solve_tridiagonal tested in AD mode). |
| 354 |
|
o pkg/seaice: ECCO-CLIVAR merge: |
| 355 |
|
1. seaice AREA relaxation |
| 356 |
|
* CPP option: SEAICE_ALLOW_AREA_RELAXATION |
| 357 |
|
* runtime flag: SEAICE_tauAreaObsRelax |
| 358 |
|
2. modified global mean cost imbalance code: CPP option |
| 359 |
|
ALLOW_SSH_GLOBMEAN_COST_CONTRIBUTION |
| 360 |
|
o another iteration on solve_*diagonal.F |
| 361 |
|
Make choices independent of AUTODIFF |
| 362 |
|
Default will be the adjoinable vectorized version. |
| 363 |
|
Alternative choices are: |
| 364 |
|
* non-adjoinable original version with low memory footprint: |
| 365 |
|
#define SOLVE_DIAGONAL_LOWMEMORY |
| 366 |
|
* still adjoinable but does not vectorize (k as inner-most loop): |
| 367 |
|
#define SOLVE_DIAGONAL_KINNER (keeping SOLVE_DIAGONAL_LOWMEMORY undef) |
| 368 |
|
o testreport |
| 369 |
|
- add option to use a home made wrapper around "make" |
| 370 |
|
- remove MITGCM_IEEE env. variable (no longer active) |
| 371 |
|
o pkg/shelfice |
| 372 |
|
- improve computation of friction velocity for velocity dependent |
| 373 |
|
exchange coefficients: include velocities into SHELFICEBoundaryLayer |
| 374 |
|
block to average velocities vertically |
| 375 |
|
- add TAF store directives for SHI_ALLOW_GAMMAFRICT |
| 376 |
|
o Modify diagonal solvers, now three versions (upon request): |
| 377 |
|
#ifndef SOLVE_DIAGONAL_AUTODIFF |
| 378 |
|
1. default version (dating back a while ago) |
| 379 |
|
#else /* def SOLVE_DIAGONAL_AUTODIFF */ |
| 380 |
|
# ifdef SOLVE_DIAGONAL_KINNER |
| 381 |
|
2. adjoinable version with k-innermost loop (G. Forget) |
| 382 |
|
# else |
| 383 |
|
3. adjoinable vectorized version of 2. |
| 384 |
|
# endif |
| 385 |
|
#endif /* SOLVE_DIAGONAL_AUTODIFF */ |
| 386 |
|
All flags are currently set within routines, but could/should be moved to CPP-options. |
| 387 |
|
Array bounds for 1. are as before, for 2.,3. cover the entire domain |
| 388 |
|
(could be changed for 1. too). |
| 389 |
|
o verification/lab_sea (fwd): |
| 390 |
|
- update the 4 fwd lab_sea test experiment: no longer SEAICE_GROWTH_LEGACY, |
| 391 |
|
use 7 categories (except hb87), new solve4temp params (except salt_plume), |
| 392 |
|
constant salinity (=4) or variable salinity (salt_plume) done through |
| 393 |
|
SItracer. |
| 394 |
|
o pkg/seaice: |
| 395 |
|
- rename parameters: SIsalFrac to SEAICE_saltFrac & SIsal0 to SEAICE_salt0; |
| 396 |
|
- remove old parameters ( SEAICE_freeze, SEAICEturbFluxFormula, SEAICE_gamma_t, |
| 397 |
|
SEAICE_gamma_t_frz, SEAICE_availHeatTaper & SEAICE_availHeatFracFrz) |
| 398 |
|
from SEAICE_PARMS.h ; still need to fix seaice_cost_sst.F (broken when |
| 399 |
|
SEAICE_freeze got retired) but already better than previously with |
| 400 |
|
SEAICE_freeze=123456.7 deg.C; |
| 401 |
|
- move setting of facOpenGrow/Melt from SEAICE_INIT_FIXED to SEAICE_READPARMS |
| 402 |
|
(safer multi-threaded setting); always set SEAICEuseEVP; |
| 403 |
|
- setting of ocean-ice turb. flux coeff: moved from SEAICE_CHECK to |
| 404 |
|
SEAICE_READPARMS & SEAICE_INIT_FIXED (this fixes wrong summary report); |
| 405 |
|
stop if multiple specifications for the same coeff; make sure default |
| 406 |
|
SEAICE_mcPheePiston is compatible with drF(1) & deltaT. |
| 407 |
|
o pkg/obcs: |
| 408 |
|
- allow to switch to upwind 1rst order advection scheme for the advective |
| 409 |
|
flux computation at the open-boundary; disabled for adjoint built. |
| 410 |
|
o pkg/ptracers: |
| 411 |
|
- rename PTRACERS_RESTART.h -> PTRACERS_START.h |
| 412 |
|
- rename ptracers_fields_blocking_exchanges.F -> ptracers_fields_blocking_exch.F |
| 413 |
|
- add parameter to decide when to start a tracer; use internal flag to |
| 414 |
|
know which tracer to step-forward; switch this internal flag at the top |
| 415 |
|
of forward_step.F (new S/R PTRACERS_SWITCH_ONOFF); for now, not enabled |
| 416 |
|
for adjoint built. |
| 417 |
|
o pkg/seaice: |
| 418 |
|
- add SEAICE_SIZE.h anywhere in the code where SEAICE.h is included ; |
| 419 |
|
- move MULTDIM declaration and setting from SEAICE.h to SEAICE_SIZE.h ; |
| 420 |
|
- keep experiment offline_exf_seaice truly single category (set MULTDIM=1). |
| 421 |
|
o pkg/ecco: |
| 422 |
|
- fix writing of output file DiagnosCost_ClimSalt/Theta |
| 423 |
|
and DiagnosCost_DriftSalt/Theta (if multi-tiles per proc). |
| 424 |
|
o pkg/seaice: |
| 425 |
|
- add missing initialisation of COR_ICE array in dynsolver.F |
| 426 |
|
- merging SEAICE_MULTICATEGORY define and undef cases. Now the same code is |
| 427 |
|
compiled either way, and is tested in all exps (multidim was only tested |
| 428 |
|
in lab_sea ad before). |
| 429 |
|
- cosmetic change to prepare for future addition of ITD. no change to results. |
| 430 |
|
no defaults change. |
| 431 |
|
- added run time param SEAICE_multDim (7 for SEAICE_MULTICATEGORY, 1 otherwise). |
| 432 |
|
- by lack of adequate verification experiments, I further used custom versions |
| 433 |
|
of global_ocean.cs32x15 to test multicat or not, with pickup or not. All |
| 434 |
|
should be correct. |
| 435 |
|
- avoid recomputations in ad. |
| 436 |
|
o array initialisation in adjoint mode: |
| 437 |
|
1) pkg/mom_common/mom_calc_ke.F: |
| 438 |
|
add full array initialisation of KE inside mom_calc_ke.F (ALLOW_AUTODIFF_TAMC): |
| 439 |
|
The S/R which calls MOM_CALC_KE initialises KE but in some recomputation |
| 440 |
|
part this initialisation is gone, resulting in Floating Point Exception |
| 441 |
|
(caught by open64 compiler with debug option). |
| 442 |
|
2) model/src/convective_weights.F |
| 443 |
|
set output convectCount over the full array range (instead of doing |
| 444 |
|
an update): this fix a similar issue when called within recomputation (adjoint) |
| 445 |
|
which could by-pass the initialisation (normally done in the caller S/R). |
| 446 |
|
o pkg/seaice: |
| 447 |
|
- final version of turbulent flux params, as agreed at MIT ice meet: |
| 448 |
|
SEAICE_mcPheePiston (default is ~ 0.000875) |
| 449 |
|
SEAICE_mcPheeTaper (default is 0.) |
| 450 |
|
SEAICE_frazilFrac (defualt is 1.) |
| 451 |
|
SEAICE_mcPheeStepFunc (default is false) |
| 452 |
|
- changes results at trunction error level |
| 453 |
|
- ifdef SEAICE_EXTERNAL_FLUXES, disallow SEAICE_waterAlbedo |
| 454 |
|
o added pkg/frazil: |
| 455 |
|
- see frazil_description.tex for details |
| 456 |
|
- verification experiment is global_with_exf/input.yearly; changes output.yearly.txt |
| 457 |
|
o pkg/shelfice/shelfice_thermodynamics.F |
| 458 |
|
- fix exponents for velocity dependent exchange coefficients: change |
| 459 |
|
from 2/3(=0) to 0.66666666666666667 |
| 460 |
|
- move computation of velocity dependent exchange coefficients into |
| 461 |
|
separate loop, so that it applies to both thermodynamics |
| 462 |
|
- move some divisions by constants out of loops |
| 463 |
|
|
| 464 |
|
checkpoint63j (2012/02/16) |
| 465 |
|
o pkg/seaice: |
| 466 |
|
- overhaul of seaice_summary.F after feb 2012 clean-up of seaice_growth.F etc. |
| 467 |
|
- SItracer: added run time params, and allow coupling to ocn salinity. |
| 468 |
|
o pkg/seaice: remove (almost) all divisions by constants within loops |
| 469 |
|
in seaice_growth.F and seaice_solve4temp.F, this changes (only) the cg2d |
| 470 |
|
results of global_ocean.cs32x15.seaice from 16 to 11 digits (but only |
| 471 |
|
those, caused by 1/QI -> recip_QI) -> update output.seaice.txt, |
| 472 |
|
output_adm.seaice_dynmix.txt and also lab_sea: output_adm.evp.txt |
| 473 |
|
o pkg/seaice : |
| 474 |
|
- simplify McPhee tapering, and allow to use it with SEAICEturbFluxFormula |
| 475 |
|
= 1 or 2, using newly added run time parameter SEAICE_availHeatTaper. |
| 476 |
|
- fix d_AREAbyATM, d_AREAbyOCN, d_AREAbyICE diags I broke in r1.148. |
| 477 |
|
- add permanent SItflux diag that corresponds to TFLUX but includes |
| 478 |
|
ice+snow. Hence SItflux-TFLUX should match the ice+snow heat budget. |
| 479 |
|
- allow activation/testing of a fix for suspected missing term in |
| 480 |
|
ocn-ice heat budget (to be confirmed). To test this, you want to |
| 481 |
|
undef SEAICE_DISABLE_HEATCONSFIX, and then set the run time param |
| 482 |
|
SEAICEheatConsFix to .TRUE. that also allows the 'SIaaflux' diagnostic. |
| 483 |
|
- no change in results, fwd or ad. |
| 484 |
|
o global_ocean.cs32x15/results: 3 changes following from the pkg/seaice commit below: |
| 485 |
|
- switch to variable freezing point. |
| 486 |
|
Results change : cg2d -> 1 digits; ad -> 3-6-6 digits. |
| 487 |
|
- bug fix : the sublimation term that was missing is now activated. |
| 488 |
|
Results change : cg2d -> 2 digits; ad -> 8-6-6 digits. |
| 489 |
|
- bug fix : flooding sign term for SIsal0.NE.0. |
| 490 |
|
Result change : cg2d -> 8 digits; ad -> 11-12-9 digits. |
| 491 |
|
o pkg/seaice: |
| 492 |
|
- unified freezing point treatment : the old SEAICE_VARIABLE_FREEZING_POINT |
| 493 |
|
is now the default, and the old default constant freezing point is recovered |
| 494 |
|
with SEAICE_tempFrz0 = -1.96, |
| 495 |
|
SEAICE_dTempFrz_dS = 0., |
| 496 |
|
- retire SEAICE_freeze (= the old way of specifycing the constant freezing point) |
| 497 |
|
- remove ALLOW_SEAICE_FLOODING brackets; run time switch is already there. |
| 498 |
|
- bug fix (thanks to M. Losch) : the sublimation term that was missing is now |
| 499 |
|
activated. To reproduce old results with this bug define SEAICE_DISABLE_SUBLIM. |
| 500 |
|
- bug fix (silly me) : flooding sign term (seaice_growth.F r 1.149line 1211) |
| 501 |
|
was wrong for SIsal0.NE.0. Changes cs32x15 results (see above item for detail). |
| 502 |
|
- bug fix (thanks to O. Jahn) : area loss for melting got messed up in |
| 503 |
|
seaice_growth r149 for legacy branch (no results change). |
| 504 |
|
- introduce SEAICE_CAP_SUBLIM : caps sublimation heat flux in solve4temp (code |
| 505 |
|
from I. Fenty). |
| 506 |
|
- results did not change except for global_ocean.cs32x15, mostly due to the |
| 507 |
|
switch to variable freezing point (see above item for detail). |
| 508 |
|
o pkg/thsice: |
| 509 |
|
- fix and simplify interface to salt_plume pkg ; diagnose directly amount of |
| 510 |
|
seawater freezing in thsice_calc_thickn (not used). |
| 511 |
|
o pkg/seaice: turbulent flux term specification via run time params. |
| 512 |
|
- replace CPP switches with run time switches to facilitate testing |
| 513 |
|
- retired CPPs : MCPHEE_OCEAN_ICE_HEAT_FLUX, |
| 514 |
|
GRADIENT_MIXED_LAYER_TURBULENCE_FACTOR |
| 515 |
|
- added runtime parameter : SEAICEturbFluxFormula |
| 516 |
|
- move Mcphee constants to SEAICE_PARAMS.h |
| 517 |
|
- seaice_readparms.F : read turbulent flux term params (SEAICEturbFluxFormula; |
| 518 |
|
SEAICE_availHeatFrac/Frz, or SEAICE_gamma_t/_frz if apply); |
| 519 |
|
remove seaice_summary call (moved to seaice_init_fixed) |
| 520 |
|
- seaice_init_fixed.F : set SEAICE_availHeatFrac/Frz depending on |
| 521 |
|
SEAICEturbFluxFormula, and seaice_summary call (moved from seaice_readparms) |
| 522 |
|
- seaice_summary.F : add missing report of read turbulent flux term set up |
| 523 |
|
and SITRACER. |
| 524 |
|
- seaice_check.F : check range; inform and stop if retired CPPs are used |
| 525 |
|
(incl. SEAICE_AGE). |
| 526 |
|
- seaice_growth.F : unified use of SEAICE_availHeatFrac/Frz, without |
| 527 |
|
CPP switch, same computation as before for the different options. |
| 528 |
|
- change results at the truncation level. |
| 529 |
|
o pkg/seaice: |
| 530 |
|
- remove un-used parameters LAD & MAX_TICE ; add new run-time parameters: |
| 531 |
|
useMaykutSatVapPoly (default=F), postSolvTempIter (default=2) and |
| 532 |
|
SEAICE_wetAlbTemp (default=-1e-3). Remove option SEAICE_SOLVE4TEMP_LEGACY. |
| 533 |
|
- update verification/*/input*/data.seaice: |
| 534 |
|
to reproduce previous results with former #defined SEAICE_SOLVE4TEMP_LEGACY |
| 535 |
|
code, set useMaykutSatVapPoly=T & postSolvTempIter=0 ; |
| 536 |
|
and to reproduce previous results with former non-SOLVE4TEMP_LEGACY code, |
| 537 |
|
set SEAICE_wetAlbTemp=0 & SEAICE_snowThick=0. |
| 538 |
|
o pkg/seaice/seaice_solve4temp.F (SOLVE4TEMP_LEGACY): |
| 539 |
|
- remove LEGACY code for solving for tsurf (A1,A2,A3) but maintain the same |
| 540 |
|
algorithm (same choice: useMaykutPolySatVap=T, postSolvTempIter=0); |
| 541 |
|
difference in results only due to machine truncation. |
| 542 |
|
- remove MAX_TICE (tsurf is always =< TMELT anyway); keep MIN_TICE if using |
| 543 |
|
MaykutPolySatVap; keep MIN_LWDOWN. |
| 544 |
|
- adapt SEAICE_MODIFY_GROWTH_ADJ code (untested) to non-legacy formulation. |
| 545 |
|
- update output of verification exp. lab_sea (4 fwd: 11 cg2d digits except hb87, |
| 546 |
|
+ 3 AD: 6 digits except evp) and offline_exf_seaice.seaice (12 digits). |
| 547 |
|
o pkg/salt_plume: |
| 548 |
|
- isolate saltPlumeFlux array exch into a specific S/R called from |
| 549 |
|
do_oceanic_phys.F |
| 550 |
|
- add interface to pkg/salt_plume (filling of saltPlumeFlux) in pkg/thsice. |
| 551 |
|
o verification: |
| 552 |
|
- move untested set-up to MITgcm_contrib/verification_other. |
| 553 |
|
o pkg/seaice: |
| 554 |
|
- retire old way of SEAICE_AGE. Now replaced with ALLOW_SITRACER. |
| 555 |
|
o pkg/seaice: |
| 556 |
|
- seaice_solve4temp.F (keeping results unchanged): |
| 557 |
|
* add 2 future run-time params to select which saturation vapor pressure |
| 558 |
|
formula is used and how post-iteration fluxes are set. |
| 559 |
|
Temporary set these 2 params according to option SOLVE4TEMP_LEGACY; |
| 560 |
|
* New piece of code (not used) to update surf. atmos. heat flux F_ia using |
| 561 |
|
the linearized expression to stay consistent with tsurf finding. |
| 562 |
|
- seaice_freedrift.F avoid floating exception in sqrt(y) since y=sqrt(x*x)-x |
| 563 |
|
can be < 0. |
| 564 |
|
o pkg/seaice/seaice_solve4temp.F (SOLVE4TEMP_LEGACY undef): |
| 565 |
|
- remove from F_ia (and from it's derivative dFiDTs1) the contribution of |
| 566 |
|
conductive heat flux F_c , and add it explicitly when updating tsurf as |
| 567 |
|
solution of: Fc = Fia + d/dT(Fia - Fc) *delta.tsurf |
| 568 |
|
so that now F_ia has a consistent meaning through the entire routine. |
| 569 |
|
This causes differences at truncation level: update output of fwd exp. |
| 570 |
|
global_ocean.cs32x15.seaice. |
| 571 |
|
o pkg/seaice: |
| 572 |
|
- lots of editing/cleaning in seaice_solve4temp.F + better documentation |
| 573 |
|
of arrays, sign/direction of fluxes; no effect on results. |
| 574 |
|
- replace hard coded 273.16 (=previous value of celsius2K) by celsius2K |
| 575 |
|
in seaice_budget_ocean.F & seaice_solve4temp.F (SOLVE4TEMP_LEGACY part); |
| 576 |
|
note: this should have been done before changing celsius2K default. |
| 577 |
|
- to keep results unchanged, add back old value of celsius2K in verification |
| 578 |
|
exp. input*/data : lab_sea (AD), offline_exf_seaice (fwd+AD) & seaice_obcs |
| 579 |
|
- replace hard coded seawater freezing temp in seaice_solve4temp.F (part |
| 580 |
|
SOLVE4TEMP_LEGACY) of 271.20 (=previous value of celsius2K + default |
| 581 |
|
value of SEAICE_freeze) by celsius2K+SEAICE_freeze so that seawater |
| 582 |
|
freezing temp is consistent with value in seaice_growth.F ; this causes |
| 583 |
|
differences at machine truncation level: update lab_sea AD output (3/4). |
| 584 |
|
o pkg/seaice: |
| 585 |
|
- mask (in exf_init_fixed.F) atemp,aqh,evap,precip,runoff,lwflx,areamask |
| 586 |
|
and climsst/sss even when using seaice (to prevent calculation that might |
| 587 |
|
use SST on land); does not affect any verification exp. |
| 588 |
|
- apply mask to taux,tauy (in seaice_get_dynforcing.F); update output |
| 589 |
|
of lab_sea.fd & lab_sea.hb87 and global_ocean.cs32x15.seaice_dynmix. |
| 590 |
o pkg/seaice/seaice_growth.F: add mask to avoid non-zero ice thickness |
o pkg/seaice/seaice_growth.F: add mask to avoid non-zero ice thickness |
| 591 |
over land (non-legacy code) |
over land (non-legacy code) |
| 592 |
o vectorize salt_plume package: |
o vectorize salt_plume package: |
| 593 |
- add extra code to salt_plume_frac and salt_plume_tendency_apply_s |
- add extra code to salt_plume_frac and salt_plume_tendency_apply_s |
| 594 |
- this code computes plumebk everywhere (also over land), thus requires |
- this code computes plumebk everywhere (also over land), thus requires |
| 595 |
code to avoid divisions by zero and catch the case of kLev=Nr. |
code to avoid divisions by zero and catch the case of kLev=Nr. |
Parent Directory
| 
Revision Log
| 
Revision Graph
| 
Patch