4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
5 |
============================== |
============================== |
6 |
|
|
7 |
|
o pkg/bbl: modification so it can work with z* |
8 |
|
o eesupp/src: |
9 |
|
- PRINT_MESSAGE: use OpenMP critical instruction (similar to PRINT_ERROR); |
10 |
|
writing to unit 0 (if unit=errorMessageUnit) extended to the case myThid=1 |
11 |
|
- CHECK_THREADS: specific code if using OpenMP threading: improve checking of |
12 |
|
# of threads across processors (e.g., different env OMP_NUM_THREADS setting) |
13 |
|
- improve special preliminary call to EESET_PARMS (if eedata is read 2 times). |
14 |
|
- when using MPI, always set usingMPI=T ; set default accordingly and stop |
15 |
|
if not consistent. |
16 |
|
- ALL_PROC_DIE: add a BARRIER after MPI_FINALIZE call ; skip this when thread |
17 |
|
argument is zero (in case called within single-thread section). |
18 |
|
o pkg/fizhi: |
19 |
|
clean-up turbulence cold-start switch: decided in fizhi_init_vars.F, stored |
20 |
|
in common bloc (fizhi_coms.h) and then passed as argument up to S/R TURBIO. |
21 |
|
o pkg/exch2: |
22 |
|
- add argument "updateCorners" to S/R exch2_get_uv_bounds (enable to |
23 |
|
activate EXCH_IGNORE_CORNERS in vector EXCH S/R). |
24 |
|
- switch 1rst EXCH call to IGNORE_CORNERS (instead of UPDATE_CORNERS) |
25 |
|
but keep the 2nd with UPDATE_CORNERS: this prevents overwriting good data |
26 |
|
with bad ones coming from a not-yet-updated halo. |
27 |
|
This should fix the problem when sNx <> sNy and GCD(sNx,sNy) < 2*Olx. |
28 |
|
o pkg/frazil: |
29 |
|
move FrzForcT diag from frazil_diagnostics_state.F to frazil_calc_rhs.F |
30 |
|
http://mitgcm.org/pipermail/mitgcm-devel/2012-March/005384.html |
31 |
|
o model/src/config_check.F: add a check for OLx>sNx/OLy>sNy |
32 |
|
o verification/lab_sea |
33 |
|
* #undef SEAICE_EXCLUDE_FOR_EXACT_AD_TESTING |
34 |
|
* #define ALLOW_SST_CONTROL & ALLOW_SST_CONTROL |
35 |
|
* nTimeSteps 3 -> 4 |
36 |
|
* changed reference output accordingly |
37 |
|
o pkg/ctrl: |
38 |
|
* fix ctrl_get_gen for xx_sst, xx_sss |
39 |
|
o pkg/fizhi: |
40 |
|
- fix 2 calls to S/R GRD2MSC (argument igrd) in update_earth_exports.F |
41 |
|
update results of exp. fizhi-cs-32x32x40 (fail with only 6 digits for cg2d) |
42 |
|
- by default, use new version of S/R GETPWHERE (#define TRY_NEW_GETPWHERE); |
43 |
|
fix few out-of-bounds indices in referencing vars (initialise igrd to 1); |
44 |
|
clean-up fizhi_turb.F (remove unused & uninitialised qdiaglsm array). |
45 |
|
o pkg/gmredi: |
46 |
|
- add diagnostic of Redi (main diagnonal) vertical diffusive flux of Temp |
47 |
|
o pkg/fizhi: |
48 |
|
- fix few initialisation and multi-tile Pb: aqualev & gridalt now pass the |
49 |
|
(old)aces open64 tuv test. |
50 |
|
o pkg/icefront: fix bug and reorder loop in icefront_init_varia.F |
51 |
|
o pkg/mom_vecinv: |
52 |
|
- fix 1 loop range in mom_vecinv.F to be like all the other loops which |
53 |
|
set/update gu,gv/Diss (fix Pb of uninitialised fVerUkp in admom_vecinv |
54 |
|
with non-lin free-surf). |
55 |
|
o model/src: move k-loop outside of i/j-loops when (re-)computing R_low |
56 |
|
in ini/update_masks_etc.F in order to avoid -O3 optimization problems with |
57 |
|
some compilers (gfortran 4.6.0 on MacOS) |
58 |
|
o momentum: |
59 |
|
- separate fVer?(:,:,kUp) & fVer?(:,:,kDown) in argument list of MOM_FLUXFORM |
60 |
|
& MOM_VECINV subroutines (to help TAF). This fixes Pb of uninitialised |
61 |
|
fVer?(kUp) in admom_fluxform with non-lin free-surf or bottom control. |
62 |
|
|
63 |
|
checkpoint63k (2012/03/17) |
64 |
|
o model/src: |
65 |
|
- selectAddFluid (ALLOW_ADDFLUID): add missing term in horizontal momentum |
66 |
|
flux-form (mom_fluxform.F) and vertical momentum (calc_gw.F) equations. |
67 |
|
- SOLVE_DIAGONAL: switch default to use adjoinable & vectorized version |
68 |
|
(#undef SOLVE_DIAGONAL_LOWMEMORY); update output of exp. front_relax.bvp. |
69 |
|
o model/src: solve_*diagonal.F |
70 |
|
- replace division with multiplication by inverse (similar to original version) |
71 |
|
update output of AD exp. global_ocean.90x40x15. |
72 |
|
- fix pkg/gmredi/gmredi_calc_psi_bvp.F & pkg/ggl90/ggl90_calc.F |
73 |
|
to enable the use of the vectorized & differentiable solve_tridiagonal.F |
74 |
|
- test exp. global_ocean.90x40x15/input_ad: switch to advection scheme 20 for |
75 |
|
vertical advection (to get solve_tridiagonal tested in AD mode). |
76 |
|
o pkg/seaice: ECCO-CLIVAR merge: |
77 |
|
1. seaice AREA relaxation |
78 |
|
* CPP option: SEAICE_ALLOW_AREA_RELAXATION |
79 |
|
* runtime flag: SEAICE_tauAreaObsRelax |
80 |
|
2. modified global mean cost imbalance code: CPP option |
81 |
|
ALLOW_SSH_GLOBMEAN_COST_CONTRIBUTION |
82 |
|
o another iteration on solve_*diagonal.F |
83 |
|
Make choices independent of AUTODIFF |
84 |
|
Default will be the adjoinable vectorized version. |
85 |
|
Alternative choices are: |
86 |
|
* non-adjoinable original version with low memory footprint: |
87 |
|
#define SOLVE_DIAGONAL_LOWMEMORY |
88 |
|
* still adjoinable but does not vectorize (k as inner-most loop): |
89 |
|
#define SOLVE_DIAGONAL_KINNER (keeping SOLVE_DIAGONAL_LOWMEMORY undef) |
90 |
|
o testreport |
91 |
|
- add option to use a home made wrapper around "make" |
92 |
|
- remove MITGCM_IEEE env. variable (no longer active) |
93 |
|
o pkg/shelfice |
94 |
|
- improve computation of friction velocity for velocity dependent |
95 |
|
exchange coefficients: include velocities into SHELFICEBoundaryLayer |
96 |
|
block to average velocities vertically |
97 |
|
- add TAF store directives for SHI_ALLOW_GAMMAFRICT |
98 |
|
o Modify diagonal solvers, now three versions (upon request): |
99 |
|
#ifndef SOLVE_DIAGONAL_AUTODIFF |
100 |
|
1. default version (dating back a while ago) |
101 |
|
#else /* def SOLVE_DIAGONAL_AUTODIFF */ |
102 |
|
# ifdef SOLVE_DIAGONAL_KINNER |
103 |
|
2. adjoinable version with k-innermost loop (G. Forget) |
104 |
|
# else |
105 |
|
3. adjoinable vectorized version of 2. |
106 |
|
# endif |
107 |
|
#endif /* SOLVE_DIAGONAL_AUTODIFF */ |
108 |
|
All flags are currently set within routines, but could/should be moved to CPP-options. |
109 |
|
Array bounds for 1. are as before, for 2.,3. cover the entire domain |
110 |
|
(could be changed for 1. too). |
111 |
|
o verification/lab_sea (fwd): |
112 |
|
- update the 4 fwd lab_sea test experiment: no longer SEAICE_GROWTH_LEGACY, |
113 |
|
use 7 categories (except hb87), new solve4temp params (except salt_plume), |
114 |
|
constant salinity (=4) or variable salinity (salt_plume) done through |
115 |
|
SItracer. |
116 |
|
o pkg/seaice: |
117 |
|
- rename parameters: SIsalFrac to SEAICE_saltFrac & SIsal0 to SEAICE_salt0; |
118 |
|
- remove old parameters ( SEAICE_freeze, SEAICEturbFluxFormula, SEAICE_gamma_t, |
119 |
|
SEAICE_gamma_t_frz, SEAICE_availHeatTaper & SEAICE_availHeatFracFrz) |
120 |
|
from SEAICE_PARMS.h ; still need to fix seaice_cost_sst.F (broken when |
121 |
|
SEAICE_freeze got retired) but already better than previously with |
122 |
|
SEAICE_freeze=123456.7 deg.C; |
123 |
|
- move setting of facOpenGrow/Melt from SEAICE_INIT_FIXED to SEAICE_READPARMS |
124 |
|
(safer multi-threaded setting); always set SEAICEuseEVP; |
125 |
|
- setting of ocean-ice turb. flux coeff: moved from SEAICE_CHECK to |
126 |
|
SEAICE_READPARMS & SEAICE_INIT_FIXED (this fixes wrong summary report); |
127 |
|
stop if multiple specifications for the same coeff; make sure default |
128 |
|
SEAICE_mcPheePiston is compatible with drF(1) & deltaT. |
129 |
|
o pkg/obcs: |
130 |
|
- allow to switch to upwind 1rst order advection scheme for the advective |
131 |
|
flux computation at the open-boundary; disabled for adjoint built. |
132 |
|
o pkg/ptracers: |
133 |
|
- rename PTRACERS_RESTART.h -> PTRACERS_START.h |
134 |
|
- rename ptracers_fields_blocking_exchanges.F -> ptracers_fields_blocking_exch.F |
135 |
|
- add parameter to decide when to start a tracer; use internal flag to |
136 |
|
know which tracer to step-forward; switch this internal flag at the top |
137 |
|
of forward_step.F (new S/R PTRACERS_SWITCH_ONOFF); for now, not enabled |
138 |
|
for adjoint built. |
139 |
|
o pkg/seaice: |
140 |
|
- add SEAICE_SIZE.h anywhere in the code where SEAICE.h is included ; |
141 |
|
- move MULTDIM declaration and setting from SEAICE.h to SEAICE_SIZE.h ; |
142 |
|
- keep experiment offline_exf_seaice truly single category (set MULTDIM=1). |
143 |
|
o pkg/ecco: |
144 |
|
- fix writing of output file DiagnosCost_ClimSalt/Theta |
145 |
|
and DiagnosCost_DriftSalt/Theta (if multi-tiles per proc). |
146 |
|
o pkg/seaice: |
147 |
|
- add missing initialisation of COR_ICE array in dynsolver.F |
148 |
|
- merging SEAICE_MULTICATEGORY define and undef cases. Now the same code is |
149 |
|
compiled either way, and is tested in all exps (multidim was only tested |
150 |
|
in lab_sea ad before). |
151 |
|
- cosmetic change to prepare for future addition of ITD. no change to results. |
152 |
|
no defaults change. |
153 |
|
- added run time param SEAICE_multDim (7 for SEAICE_MULTICATEGORY, 1 otherwise). |
154 |
|
- by lack of adequate verification experiments, I further used custom versions |
155 |
|
of global_ocean.cs32x15 to test multicat or not, with pickup or not. All |
156 |
|
should be correct. |
157 |
|
- avoid recomputations in ad. |
158 |
|
o array initialisation in adjoint mode: |
159 |
|
1) pkg/mom_common/mom_calc_ke.F: |
160 |
|
add full array initialisation of KE inside mom_calc_ke.F (ALLOW_AUTODIFF_TAMC): |
161 |
|
The S/R which calls MOM_CALC_KE initialises KE but in some recomputation |
162 |
|
part this initialisation is gone, resulting in Floating Point Exception |
163 |
|
(caught by open64 compiler with debug option). |
164 |
|
2) model/src/convective_weights.F |
165 |
|
set output convectCount over the full array range (instead of doing |
166 |
|
an update): this fix a similar issue when called within recomputation (adjoint) |
167 |
|
which could by-pass the initialisation (normally done in the caller S/R). |
168 |
|
o pkg/seaice: |
169 |
|
- final version of turbulent flux params, as agreed at MIT ice meet: |
170 |
|
SEAICE_mcPheePiston (default is ~ 0.000875) |
171 |
|
SEAICE_mcPheeTaper (default is 0.) |
172 |
|
SEAICE_frazilFrac (defualt is 1.) |
173 |
|
SEAICE_mcPheeStepFunc (default is false) |
174 |
|
- changes results at trunction error level |
175 |
|
- ifdef SEAICE_EXTERNAL_FLUXES, disallow SEAICE_waterAlbedo |
176 |
|
o added pkg/frazil: |
177 |
|
- see frazil_description.tex for details |
178 |
|
- verification experiment is global_with_exf/input.yearly; changes output.yearly.txt |
179 |
|
o pkg/shelfice/shelfice_thermodynamics.F |
180 |
|
- fix exponents for velocity dependent exchange coefficients: change |
181 |
|
from 2/3(=0) to 0.66666666666666667 |
182 |
|
- move computation of velocity dependent exchange coefficients into |
183 |
|
separate loop, so that it applies to both thermodynamics |
184 |
|
- move some divisions by constants out of loops |
185 |
|
|
186 |
|
checkpoint63j (2012/02/16) |
187 |
|
o pkg/seaice: |
188 |
|
- overhaul of seaice_summary.F after feb 2012 clean-up of seaice_growth.F etc. |
189 |
|
- SItracer: added run time params, and allow coupling to ocn salinity. |
190 |
|
o pkg/seaice: remove (almost) all divisions by constants within loops |
191 |
|
in seaice_growth.F and seaice_solve4temp.F, this changes (only) the cg2d |
192 |
|
results of global_ocean.cs32x15.seaice from 16 to 11 digits (but only |
193 |
|
those, caused by 1/QI -> recip_QI) -> update output.seaice.txt, |
194 |
|
output_adm.seaice_dynmix.txt and also lab_sea: output_adm.evp.txt |
195 |
|
o pkg/seaice : |
196 |
|
- simplify McPhee tapering, and allow to use it with SEAICEturbFluxFormula |
197 |
|
= 1 or 2, using newly added run time parameter SEAICE_availHeatTaper. |
198 |
|
- fix d_AREAbyATM, d_AREAbyOCN, d_AREAbyICE diags I broke in r1.148. |
199 |
|
- add permanent SItflux diag that corresponds to TFLUX but includes |
200 |
|
ice+snow. Hence SItflux-TFLUX should match the ice+snow heat budget. |
201 |
|
- allow activation/testing of a fix for suspected missing term in |
202 |
|
ocn-ice heat budget (to be confirmed). To test this, you want to |
203 |
|
undef SEAICE_DISABLE_HEATCONSFIX, and then set the run time param |
204 |
|
SEAICEheatConsFix to .TRUE. that also allows the 'SIaaflux' diagnostic. |
205 |
|
- no change in results, fwd or ad. |
206 |
|
o global_ocean.cs32x15/results: 3 changes following from the pkg/seaice commit below: |
207 |
|
- switch to variable freezing point. |
208 |
|
Results change : cg2d -> 1 digits; ad -> 3-6-6 digits. |
209 |
|
- bug fix : the sublimation term that was missing is now activated. |
210 |
|
Results change : cg2d -> 2 digits; ad -> 8-6-6 digits. |
211 |
|
- bug fix : flooding sign term for SIsal0.NE.0. |
212 |
|
Result change : cg2d -> 8 digits; ad -> 11-12-9 digits. |
213 |
|
o pkg/seaice: |
214 |
|
- unified freezing point treatment : the old SEAICE_VARIABLE_FREEZING_POINT |
215 |
|
is now the default, and the old default constant freezing point is recovered |
216 |
|
with SEAICE_tempFrz0 = -1.96, |
217 |
|
SEAICE_dTempFrz_dS = 0., |
218 |
|
- retire SEAICE_freeze (= the old way of specifycing the constant freezing point) |
219 |
|
- remove ALLOW_SEAICE_FLOODING brackets; run time switch is already there. |
220 |
|
- bug fix (thanks to M. Losch) : the sublimation term that was missing is now |
221 |
|
activated. To reproduce old results with this bug define SEAICE_DISABLE_SUBLIM. |
222 |
|
- bug fix (silly me) : flooding sign term (seaice_growth.F r 1.149line 1211) |
223 |
|
was wrong for SIsal0.NE.0. Changes cs32x15 results (see above item for detail). |
224 |
|
- bug fix (thanks to O. Jahn) : area loss for melting got messed up in |
225 |
|
seaice_growth r149 for legacy branch (no results change). |
226 |
|
- introduce SEAICE_CAP_SUBLIM : caps sublimation heat flux in solve4temp (code |
227 |
|
from I. Fenty). |
228 |
|
- results did not change except for global_ocean.cs32x15, mostly due to the |
229 |
|
switch to variable freezing point (see above item for detail). |
230 |
|
o pkg/thsice: |
231 |
|
- fix and simplify interface to salt_plume pkg ; diagnose directly amount of |
232 |
|
seawater freezing in thsice_calc_thickn (not used). |
233 |
|
o pkg/seaice: turbulent flux term specification via run time params. |
234 |
|
- replace CPP switches with run time switches to facilitate testing |
235 |
|
- retired CPPs : MCPHEE_OCEAN_ICE_HEAT_FLUX, |
236 |
|
GRADIENT_MIXED_LAYER_TURBULENCE_FACTOR |
237 |
|
- added runtime parameter : SEAICEturbFluxFormula |
238 |
|
- move Mcphee constants to SEAICE_PARAMS.h |
239 |
|
- seaice_readparms.F : read turbulent flux term params (SEAICEturbFluxFormula; |
240 |
|
SEAICE_availHeatFrac/Frz, or SEAICE_gamma_t/_frz if apply); |
241 |
|
remove seaice_summary call (moved to seaice_init_fixed) |
242 |
|
- seaice_init_fixed.F : set SEAICE_availHeatFrac/Frz depending on |
243 |
|
SEAICEturbFluxFormula, and seaice_summary call (moved from seaice_readparms) |
244 |
|
- seaice_summary.F : add missing report of read turbulent flux term set up |
245 |
|
and SITRACER. |
246 |
|
- seaice_check.F : check range; inform and stop if retired CPPs are used |
247 |
|
(incl. SEAICE_AGE). |
248 |
|
- seaice_growth.F : unified use of SEAICE_availHeatFrac/Frz, without |
249 |
|
CPP switch, same computation as before for the different options. |
250 |
|
- change results at the truncation level. |
251 |
|
o pkg/seaice: |
252 |
|
- remove un-used parameters LAD & MAX_TICE ; add new run-time parameters: |
253 |
|
useMaykutSatVapPoly (default=F), postSolvTempIter (default=2) and |
254 |
|
SEAICE_wetAlbTemp (default=-1e-3). Remove option SEAICE_SOLVE4TEMP_LEGACY. |
255 |
|
- update verification/*/input*/data.seaice: |
256 |
|
to reproduce previous results with former #defined SEAICE_SOLVE4TEMP_LEGACY |
257 |
|
code, set useMaykutSatVapPoly=T & postSolvTempIter=0 ; |
258 |
|
and to reproduce previous results with former non-SOLVE4TEMP_LEGACY code, |
259 |
|
set SEAICE_wetAlbTemp=0 & SEAICE_snowThick=0. |
260 |
|
o pkg/seaice/seaice_solve4temp.F (SOLVE4TEMP_LEGACY): |
261 |
|
- remove LEGACY code for solving for tsurf (A1,A2,A3) but maintain the same |
262 |
|
algorithm (same choice: useMaykutPolySatVap=T, postSolvTempIter=0); |
263 |
|
difference in results only due to machine truncation. |
264 |
|
- remove MAX_TICE (tsurf is always =< TMELT anyway); keep MIN_TICE if using |
265 |
|
MaykutPolySatVap; keep MIN_LWDOWN. |
266 |
|
- adapt SEAICE_MODIFY_GROWTH_ADJ code (untested) to non-legacy formulation. |
267 |
|
- update output of verification exp. lab_sea (4 fwd: 11 cg2d digits except hb87, |
268 |
|
+ 3 AD: 6 digits except evp) and offline_exf_seaice.seaice (12 digits). |
269 |
|
o pkg/salt_plume: |
270 |
|
- isolate saltPlumeFlux array exch into a specific S/R called from |
271 |
|
do_oceanic_phys.F |
272 |
|
- add interface to pkg/salt_plume (filling of saltPlumeFlux) in pkg/thsice. |
273 |
|
o verification: |
274 |
|
- move untested set-up to MITgcm_contrib/verification_other. |
275 |
|
o pkg/seaice: |
276 |
|
- retire old way of SEAICE_AGE. Now replaced with ALLOW_SITRACER. |
277 |
o pkg/seaice: |
o pkg/seaice: |
278 |
- seaice_solve4temp.F (keeping results unchanged): |
- seaice_solve4temp.F (keeping results unchanged): |
279 |
* add 2 future run-time params to select which saturation vapor pressure |
* add 2 future run-time params to select which saturation vapor pressure |