| 4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
| 5 |
============================== |
============================== |
| 6 |
|
|
| 7 |
|
o pkg/ecco: |
| 8 |
|
- fix writing of output file DiagnosCost_ClimSalt/Theta |
| 9 |
|
and DiagnosCost_DriftSalt/Theta (if multi-tiles per proc). |
| 10 |
|
o pkg/seaice : |
| 11 |
|
- add missing initialisation of COR_ICE array in dynsolver.F |
| 12 |
|
- merging SEAICE_MULTICATEGORY define and undef cases. Now the same code is |
| 13 |
|
compiled either way, and is tested in all exps (multidim was only tested |
| 14 |
|
in lab_sea ad before). |
| 15 |
|
- cosmetic change to prepare for future addition of ITD. no change to results. |
| 16 |
|
no defaults change. |
| 17 |
|
- added run time param SEAICE_multDim (7 for SEAICE_MULTICATEGORY, 1 otherwise). |
| 18 |
|
- by lack of adequate verification experiments, I further used custom versions |
| 19 |
|
of global_ocean.cs32x15 to test multicat or not, with pickup or not. All |
| 20 |
|
should be correct. |
| 21 |
|
- avoid recomputations in ad. |
| 22 |
|
o array initialisation in adjoint mode: |
| 23 |
|
1) pkg/mom_common/mom_calc_ke.F: |
| 24 |
|
add full array initialisation of KE inside mom_calc_ke.F (ALLOW_AUTODIFF_TAMC): |
| 25 |
|
The S/R which calls MOM_CALC_KE initialises KE but in some recomputation |
| 26 |
|
part this initialisation is gone, resulting in Floating Point Exception |
| 27 |
|
(caught by open64 compiler with debug option). |
| 28 |
|
2) model/src/convective_weights.F |
| 29 |
|
set output convectCount over the full array range (instead of doing |
| 30 |
|
an update): this fix a similar issue when called within recomputation (adjoint) |
| 31 |
|
which could by-pass the initialisation (normally done in the caller S/R). |
| 32 |
|
o pkg/seaice: |
| 33 |
|
- final version of turbulent flux params, as agreed at MIT ice meet: |
| 34 |
|
SEAICE_mcPheePiston (default is ~ 0.000875) |
| 35 |
|
SEAICE_mcPheeTaper (default is 0.) |
| 36 |
|
SEAICE_frazilFrac (defualt is 1.) |
| 37 |
|
SEAICE_mcPheeStepFunc (default is false) |
| 38 |
|
- changes results at trunction error level |
| 39 |
|
- ifdef SEAICE_EXTERNAL_FLUXES, disallow SEAICE_waterAlbedo |
| 40 |
|
o added pkg/frazil: |
| 41 |
|
- see frazil_description.tex for details |
| 42 |
|
- verification experiment is global_with_exf/input.yearly; changes output.yearly.txt |
| 43 |
|
o pkg/shelfice/shelfice_thermodynamics.F |
| 44 |
|
- fix exponents for velocity dependent exchange coefficients: change |
| 45 |
|
from 2/3(=0) to 0.66666666666666667 |
| 46 |
|
- move computation of velocity dependent exchange coefficients into |
| 47 |
|
separate loop, so that it applies to both thermodynamics |
| 48 |
|
- move some divisions by constants out of loops |
| 49 |
|
|
| 50 |
|
checkpoint63j (2012/02/16) |
| 51 |
|
o pkg/seaice: |
| 52 |
|
- overhaul of seaice_summary.F after feb 2012 clean-up of seaice_growth.F etc. |
| 53 |
|
- SItracer: added run time params, and allow coupling to ocn salinity. |
| 54 |
|
o pkg/seaice: remove (almost) all divisions by constants within loops |
| 55 |
|
in seaice_growth.F and seaice_solve4temp.F, this changes (only) the cg2d |
| 56 |
|
results of global_ocean.cs32x15.seaice from 16 to 11 digits (but only |
| 57 |
|
those, caused by 1/QI -> recip_QI) -> update output.seaice.txt, |
| 58 |
|
output_adm.seaice_dynmix.txt and also lab_sea: output_adm.evp.txt |
| 59 |
|
o pkg/seaice : |
| 60 |
|
- simplify McPhee tapering, and allow to use it with SEAICEturbFluxFormula |
| 61 |
|
= 1 or 2, using newly added run time parameter SEAICE_availHeatTaper. |
| 62 |
|
- fix d_AREAbyATM, d_AREAbyOCN, d_AREAbyICE diags I broke in r1.148. |
| 63 |
|
- add permanent SItflux diag that corresponds to TFLUX but includes |
| 64 |
|
ice+snow. Hence SItflux-TFLUX should match the ice+snow heat budget. |
| 65 |
|
- allow activation/testing of a fix for suspected missing term in |
| 66 |
|
ocn-ice heat budget (to be confirmed). To test this, you want to |
| 67 |
|
undef SEAICE_DISABLE_HEATCONSFIX, and then set the run time param |
| 68 |
|
SEAICEheatConsFix to .TRUE. that also allows the 'SIaaflux' diagnostic. |
| 69 |
|
- no change in results, fwd or ad. |
| 70 |
|
o global_ocean.cs32x15/results: 3 changes following from the pkg/seaice commit below: |
| 71 |
|
- switch to variable freezing point. |
| 72 |
|
Results change : cg2d -> 1 digits; ad -> 3-6-6 digits. |
| 73 |
|
- bug fix : the sublimation term that was missing is now activated. |
| 74 |
|
Results change : cg2d -> 2 digits; ad -> 8-6-6 digits. |
| 75 |
|
- bug fix : flooding sign term for SIsal0.NE.0. |
| 76 |
|
Result change : cg2d -> 8 digits; ad -> 11-12-9 digits. |
| 77 |
|
o pkg/seaice: |
| 78 |
|
- unified freezing point treatment : the old SEAICE_VARIABLE_FREEZING_POINT |
| 79 |
|
is now the default, and the old default constant freezing point is recovered |
| 80 |
|
with SEAICE_tempFrz0 = -1.96, |
| 81 |
|
SEAICE_dTempFrz_dS = 0., |
| 82 |
|
- retire SEAICE_freeze (= the old way of specifycing the constant freezing point) |
| 83 |
|
- remove ALLOW_SEAICE_FLOODING brackets; run time switch is already there. |
| 84 |
|
- bug fix (thanks to M. Losch) : the sublimation term that was missing is now |
| 85 |
|
activated. To reproduce old results with this bug define SEAICE_DISABLE_SUBLIM. |
| 86 |
|
- bug fix (silly me) : flooding sign term (seaice_growth.F r 1.149line 1211) |
| 87 |
|
was wrong for SIsal0.NE.0. Changes cs32x15 results (see above item for detail). |
| 88 |
|
- bug fix (thanks to O. Jahn) : area loss for melting got messed up in |
| 89 |
|
seaice_growth r149 for legacy branch (no results change). |
| 90 |
|
- introduce SEAICE_CAP_SUBLIM : caps sublimation heat flux in solve4temp (code |
| 91 |
|
from I. Fenty). |
| 92 |
|
- results did not change except for global_ocean.cs32x15, mostly due to the |
| 93 |
|
switch to variable freezing point (see above item for detail). |
| 94 |
|
o pkg/thsice: |
| 95 |
|
- fix and simplify interface to salt_plume pkg ; diagnose directly amount of |
| 96 |
|
seawater freezing in thsice_calc_thickn (not used). |
| 97 |
|
o pkg/seaice: turbulent flux term specification via run time params. |
| 98 |
|
- replace CPP switches with run time switches to facilitate testing |
| 99 |
|
- retired CPPs : MCPHEE_OCEAN_ICE_HEAT_FLUX, |
| 100 |
|
GRADIENT_MIXED_LAYER_TURBULENCE_FACTOR |
| 101 |
|
- added runtime parameter : SEAICEturbFluxFormula |
| 102 |
|
- move Mcphee constants to SEAICE_PARAMS.h |
| 103 |
|
- seaice_readparms.F : read turbulent flux term params (SEAICEturbFluxFormula; |
| 104 |
|
SEAICE_availHeatFrac/Frz, or SEAICE_gamma_t/_frz if apply); |
| 105 |
|
remove seaice_summary call (moved to seaice_init_fixed) |
| 106 |
|
- seaice_init_fixed.F : set SEAICE_availHeatFrac/Frz depending on |
| 107 |
|
SEAICEturbFluxFormula, and seaice_summary call (moved from seaice_readparms) |
| 108 |
|
- seaice_summary.F : add missing report of read turbulent flux term set up |
| 109 |
|
and SITRACER. |
| 110 |
|
- seaice_check.F : check range; inform and stop if retired CPPs are used |
| 111 |
|
(incl. SEAICE_AGE). |
| 112 |
|
- seaice_growth.F : unified use of SEAICE_availHeatFrac/Frz, without |
| 113 |
|
CPP switch, same computation as before for the different options. |
| 114 |
|
- change results at the truncation level. |
| 115 |
|
o pkg/seaice: |
| 116 |
|
- remove un-used parameters LAD & MAX_TICE ; add new run-time parameters: |
| 117 |
|
useMaykutSatVapPoly (default=F), postSolvTempIter (default=2) and |
| 118 |
|
SEAICE_wetAlbTemp (default=-1e-3). Remove option SEAICE_SOLVE4TEMP_LEGACY. |
| 119 |
|
- update verification/*/input*/data.seaice: |
| 120 |
|
to reproduce previous results with former #defined SEAICE_SOLVE4TEMP_LEGACY |
| 121 |
|
code, set useMaykutSatVapPoly=T & postSolvTempIter=0 ; |
| 122 |
|
and to reproduce previous results with former non-SOLVE4TEMP_LEGACY code, |
| 123 |
|
set SEAICE_wetAlbTemp=0 & SEAICE_snowThick=0. |
| 124 |
|
o pkg/seaice/seaice_solve4temp.F (SOLVE4TEMP_LEGACY): |
| 125 |
|
- remove LEGACY code for solving for tsurf (A1,A2,A3) but maintain the same |
| 126 |
|
algorithm (same choice: useMaykutPolySatVap=T, postSolvTempIter=0); |
| 127 |
|
difference in results only due to machine truncation. |
| 128 |
|
- remove MAX_TICE (tsurf is always =< TMELT anyway); keep MIN_TICE if using |
| 129 |
|
MaykutPolySatVap; keep MIN_LWDOWN. |
| 130 |
|
- adapt SEAICE_MODIFY_GROWTH_ADJ code (untested) to non-legacy formulation. |
| 131 |
|
- update output of verification exp. lab_sea (4 fwd: 11 cg2d digits except hb87, |
| 132 |
|
+ 3 AD: 6 digits except evp) and offline_exf_seaice.seaice (12 digits). |
| 133 |
|
o pkg/salt_plume: |
| 134 |
|
- isolate saltPlumeFlux array exch into a specific S/R called from |
| 135 |
|
do_oceanic_phys.F |
| 136 |
|
- add interface to pkg/salt_plume (filling of saltPlumeFlux) in pkg/thsice. |
| 137 |
|
o verification: |
| 138 |
|
- move untested set-up to MITgcm_contrib/verification_other. |
| 139 |
|
o pkg/seaice: |
| 140 |
|
- retire old way of SEAICE_AGE. Now replaced with ALLOW_SITRACER. |
| 141 |
|
o pkg/seaice: |
| 142 |
|
- seaice_solve4temp.F (keeping results unchanged): |
| 143 |
|
* add 2 future run-time params to select which saturation vapor pressure |
| 144 |
|
formula is used and how post-iteration fluxes are set. |
| 145 |
|
Temporary set these 2 params according to option SOLVE4TEMP_LEGACY; |
| 146 |
|
* New piece of code (not used) to update surf. atmos. heat flux F_ia using |
| 147 |
|
the linearized expression to stay consistent with tsurf finding. |
| 148 |
|
- seaice_freedrift.F avoid floating exception in sqrt(y) since y=sqrt(x*x)-x |
| 149 |
|
can be < 0. |
| 150 |
|
o pkg/seaice/seaice_solve4temp.F (SOLVE4TEMP_LEGACY undef): |
| 151 |
|
- remove from F_ia (and from it's derivative dFiDTs1) the contribution of |
| 152 |
|
conductive heat flux F_c , and add it explicitly when updating tsurf as |
| 153 |
|
solution of: Fc = Fia + d/dT(Fia - Fc) *delta.tsurf |
| 154 |
|
so that now F_ia has a consistent meaning through the entire routine. |
| 155 |
|
This causes differences at truncation level: update output of fwd exp. |
| 156 |
|
global_ocean.cs32x15.seaice. |
| 157 |
|
o pkg/seaice: |
| 158 |
|
- lots of editing/cleaning in seaice_solve4temp.F + better documentation |
| 159 |
|
of arrays, sign/direction of fluxes; no effect on results. |
| 160 |
|
- replace hard coded 273.16 (=previous value of celsius2K) by celsius2K |
| 161 |
|
in seaice_budget_ocean.F & seaice_solve4temp.F (SOLVE4TEMP_LEGACY part); |
| 162 |
|
note: this should have been done before changing celsius2K default. |
| 163 |
|
- to keep results unchanged, add back old value of celsius2K in verification |
| 164 |
|
exp. input*/data : lab_sea (AD), offline_exf_seaice (fwd+AD) & seaice_obcs |
| 165 |
|
- replace hard coded seawater freezing temp in seaice_solve4temp.F (part |
| 166 |
|
SOLVE4TEMP_LEGACY) of 271.20 (=previous value of celsius2K + default |
| 167 |
|
value of SEAICE_freeze) by celsius2K+SEAICE_freeze so that seawater |
| 168 |
|
freezing temp is consistent with value in seaice_growth.F ; this causes |
| 169 |
|
differences at machine truncation level: update lab_sea AD output (3/4). |
| 170 |
|
o pkg/seaice: |
| 171 |
|
- mask (in exf_init_fixed.F) atemp,aqh,evap,precip,runoff,lwflx,areamask |
| 172 |
|
and climsst/sss even when using seaice (to prevent calculation that might |
| 173 |
|
use SST on land); does not affect any verification exp. |
| 174 |
|
- apply mask to taux,tauy (in seaice_get_dynforcing.F); update output |
| 175 |
|
of lab_sea.fd & lab_sea.hb87 and global_ocean.cs32x15.seaice_dynmix. |
| 176 |
|
o pkg/seaice/seaice_growth.F: add mask to avoid non-zero ice thickness |
| 177 |
|
over land (non-legacy code) |
| 178 |
|
o vectorize salt_plume package: |
| 179 |
|
- add extra code to salt_plume_frac and salt_plume_tendency_apply_s |
| 180 |
|
- this code computes plumebk everywhere (also over land), thus requires |
| 181 |
|
code to avoid divisions by zero and catch the case of kLev=Nr. |
| 182 |
|
- efficiency of salt_plume_frac is still sub-optimal |
| 183 |
|
o tools (genmake2): |
| 184 |
|
- fix makefile NAME specification (option: -makefile NAME) by adding |
| 185 |
|
"-f $MAKEFILE" in few places where it was missing including makedepend |
| 186 |
|
command (contribution from Paul). |
| 187 |
|
|
| 188 |
|
checkpoint63i (2012/01/24) |
| 189 |
|
o model/src: |
| 190 |
|
- add factorized versions of some EOS code to find_rho.F, |
| 191 |
|
define USE_FACTORIZED_EOS to turn it on (default for TARGET_NEC_SX) |
| 192 |
|
o model/src: |
| 193 |
|
- add celsius2K to namelist PARM01; remove unused recip_rhoNil from PARAMS.h; |
| 194 |
|
- change default celsius2K from 273.16 to 273.15 |
| 195 |
|
- to keep results unchanged, add back old value of celsius2K in verification |
| 196 |
|
exp. input*/data : aim.5l_cs, 1D_ocean_ice_column, lab_sea (fwd), |
| 197 |
|
global_ocean.cs32x15(seaice) & cpl_aim+ocn(input_atm). |
| 198 |
|
o model/src: add a little initialisation trick to find_rho.F and find_alpha.F |
| 199 |
|
to make TAF generate vectorizable code |
| 200 |
|
o pkg/seaice/seaice_evp.F: add code (copied from CICE) that avoids underflows. |
| 201 |
|
Turn on by defining SEAICE_EVP_ELIMINATE_UNDERFLOWS. Most compilers |
| 202 |
|
have flags that do this more efficiently. |
| 203 |
|
o pkg/diagnostics/diagstats_calc.F |
| 204 |
|
- for TARGET_NEC_SX fix the treatment of the scaling factor tmpFac=scaleFact |
| 205 |
|
o pkg/obcs (with exch2): |
| 206 |
|
- derive OB indices in overlap region from OB-InsideMask (in obcs_init_fixed.F) |
| 207 |
|
and remove code (in obcs_readparms.F) which was based on EXCH of indices |
| 208 |
|
(through buffer) and did not account for exch2 (e.g., CS-grid) topology. |
| 209 |
|
o pkg/exf and seaice exp: |
| 210 |
|
- change offline_exf_seaice.seaicetd test exp. to use default DIFF1 value (=0) |
| 211 |
|
so that switching off area & heff advection flags does not affect results. |
| 212 |
|
- change default EXF_LWDOWN_WITH_EMISSIVITY to #define in EXF_OPTIONS.h |
| 213 |
|
- keep exp. seaice_obcs & 1D_ocean_ice_column(ad) unchanged (using #undef |
| 214 |
|
EXF_LWDOWN_WITH_EMISSIVITY) and, using pkg/exf default emissivities, update |
| 215 |
|
results of verification experiments: 1D_ocean_ice_column(fwd), |
| 216 |
|
global_ocean.cs32x15.seaice(fwd+ad) & seaice_dynmix(ad), lab_sea (all 8) |
| 217 |
|
and offline_exf_seaice (all 4). |
| 218 |
|
o pkg/seaice: |
| 219 |
|
- remove array YNEG, RIVER, TMIX (replaced by local var) from common block; |
| 220 |
|
remove also areaNm1 & hEffNm1 when SEAICE_GROWTH_LEGACY is undef. |
| 221 |
|
- fix bug: areaNm1 & hEffNm1 are needed in seaice_growth (SEAICE_GROWTH_LEGACY) |
| 222 |
|
but were not filled when resp. SEAICEadvArea=F & SEAICEadvHeff=F and |
| 223 |
|
not multi-dim advection; |
| 224 |
|
- fix seaice_growth.F to compile with both SEAICE_GROWTH_LEGACY and |
| 225 |
|
SHORTWAVE_HEATING #undef. |
| 226 |
o optim: make Makefile more robust (?) |
o optim: make Makefile more robust (?) |
| 227 |
o pkg/shelfice: add missing initialisation of num_shifwflx |
o pkg/obcs: |
| 228 |
|
* fix treatment of obcs_read_pickup for TAF |
| 229 |
|
o pkg/shelfice: |
| 230 |
|
* add missing initialisation of num_shifwflx |
| 231 |
|
* new adjoint verification exp. for isomip |
| 232 |
|
* change Prandtl and Schmidt numbers from generic formulations |
| 233 |
|
(in terms of model diffus. and visc.) to separate runtime |
| 234 |
|
parameters, with Holland and Jenkins (1999) default values. |
| 235 |
o pkg/exf (interpolation): |
o pkg/exf (interpolation): |
| 236 |
- extended input field (2 rows) near the N & S poles: |
- extended input field (2 rows) near the N & S poles: |
| 237 |
* fill in with the symetric value (when even Nb of data in longitude) |
* fill in with the symetric value (when even Nb of data in longitude) |
Parent Directory
| 
Revision Log
| 
Revision Graph
| 
Patch