4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
5 |
============================== |
============================== |
6 |
|
|
7 |
|
o pkg/layers : |
8 |
|
- add diagnostics to pkg/layers |
9 |
|
- move the meat of pkg/layers from layers_calc.F to layers_fluxcalc.F |
10 |
|
(from R.Tulloch). Layers_calc.F now calls layers_fluxcalc.F accordingly. |
11 |
|
o pkg/generic_advdiff/gad_implicit_r.F : |
12 |
|
- fix ADVr diagnostics in case of implicit vertical advection. |
13 |
|
o pkg/seaice/seaice_growth.F : |
14 |
|
- fix SItflux diagnostics in virtual salt flux + balanceEmpmr case. |
15 |
|
o pkg/obcs: |
16 |
|
- change default setting of OBCS_UVICE_OLD from defined to undef; |
17 |
|
- use new parameter OB_indexNone for null index value (instead of hard-coded 0) |
18 |
|
in all pkg/obcs code and in other pkgs; remove k loop in obcs_check_depths.F; |
19 |
|
- pass OB_indexNone as (new) argument to S/R CTRL_MASK_SET_[X,Y]Z; |
20 |
|
- change OB null index value (OB_indexNone) to -99 ; this fix the problem |
21 |
|
with Southern & Western OB at a tile edge (at least in a simple set-up). |
22 |
|
o pkg/obcs: |
23 |
|
- change settings of OB index in case OB coincides with tile edge (shared by |
24 |
|
2 tiles), so that common OB location is registered by both tiles (instead of |
25 |
|
only 1). However, for Southern and Western OB, index value = 0 is not |
26 |
|
recognized as valid OB index (=default null value) and needs to be fixed. |
27 |
|
- modify accordingly OBCS_BALANCE_FLOW and OBCS_MONITOR (avoid double counting) |
28 |
|
- new parameter "OB_indexNone" for null index value (for now, still zero) |
29 |
|
to replace hard-coded null index value of zero. |
30 |
|
o pkg/exch2: |
31 |
|
- improve W2_USE_E2_SAFEMODE (faster) and W2_E2_DEBUG_ON: |
32 |
|
always check buffer-size (but outside the loop); move checking for valid |
33 |
|
index from W2_USE_E2_SAFEMODE to W2_E2_DEBUG_ON; in W2_E2_DEBUG_ON, |
34 |
|
print each tile and point connection only if |W2_printMsg|>= 2 and 3. |
35 |
|
o pkg/ctrl (pack & unpack): |
36 |
|
- in ctrl_pack.F & ctrl_unpack.F, use the same local file-name variable |
37 |
|
(locally defined) for different variables (+ remove the variable-specific |
38 |
|
file name in CTRL_GENARR.h). For now, only implemented for GENARR2/3D_CONTROL |
39 |
|
(to be extended to other local file names in ctrl.h). |
40 |
|
o pkg/flt: |
41 |
|
- add a FLT_Iter0 option (similar to PTRACERS_Iter0). |
42 |
|
o pkg/mdsio: |
43 |
|
- fix missing STOP in mdsio_readvec_loc.F (case ioUnit=0) when file is missing |
44 |
|
o tools/genmake2: |
45 |
|
- add similar dependences for target EXE_AD & EXE_FTL as we have for EXECUTABLE |
46 |
|
(added by Alistair in version 1.51 for building without make depend); |
47 |
|
- remove Header files list (H_SRC_FILES) from the dependence of targets |
48 |
|
ad_input_code.f & ftl_input_code.f (not clear why it was added in the first |
49 |
|
place in version 1.23): this save going through staf when unrelated header |
50 |
|
file has been modified. |
51 |
|
Note: for an intermediate target (e.g., ad_input_code.f or adtaf) without |
52 |
|
"make depend" step, still need to do "make links" before "make adtaf". |
53 |
|
o eesupp/src (+ few pkg src files) & genmake2: |
54 |
|
- finish to remove ALWAYS_USE_MPI in source code that TAF does not see; |
55 |
|
- no longer set ALWAYS_USE_MPI in Makefile (was done by genmake2). |
56 |
|
- improve genmake2 checking for ALLOW_${PKG} in CPP_OPTIONS.h files (distinguish |
57 |
|
upper/lower case ; account for space between "#" and undef/define) |
58 |
|
o pkg/flt (with exch2): |
59 |
|
- cleaning: fix so that it compiles; move remaining MPI calls out of flt_exch2.F; |
60 |
|
- simplify threads synchronisation; fix (many issues) multi-threaded with MPI. |
61 |
|
|
62 |
|
checkpoint63s (2012/09/05) |
63 |
|
o pkg/ecco: |
64 |
|
- remove #ifdef ALLOW_SMOOTH_CORREL3D brackets. |
65 |
|
- add more relevant #ifdef ALLOW_SMOOTH ones. |
66 |
|
- sort out useAtmWind, useSMOOTH, ctrlSmoothCorrel2D. |
67 |
|
o pkg/ctrl: |
68 |
|
- CTRL_OPTIONS.h : document CPPs related to smoothing; ALLOW_SMOOTH_CORREL2D/3D, |
69 |
|
ALLOW_CTRL_SMOOTH, ALLOW_SMOOTH_CTRL2D, ALLOW_SMOOTH_CTRL3D. |
70 |
|
- ctrl.h : document run time params doInitXX, doMainPack, doMainUnpack, doPackDiag, |
71 |
|
doSinglePrecTapelev (audodiff param!), ctrlSmoothCorrel2D (new) and |
72 |
|
ctrlSmoothCorrel3D (new). |
73 |
|
The added ctrlSmoothCorrel2D/3D replace the old ALLOW_SMOOTH_CORREL2D/3D CPP. |
74 |
|
- ctrl_readparms.F : initialise ctrlSmoothCorrel2D/3D and read from namelist. |
75 |
|
- ctrl_init.F ctrl_pack.F ctrl_unpack.F : switch off ustress/vstress |
76 |
|
controls when useAtmWind, and vice versa. |
77 |
|
- ctrl_map_ini_ecco.F : UVEL0, VVEL0, edtaux, edtauy cannot be smoothed as tracers. |
78 |
|
- ctrl_get_gen.F : replace #ifdef ALLOW_SMOOTH_CORREL2D with |
79 |
|
#ifdef ALLOW_SMOOTH, IF ctrlSmoothCorrel2D, IF useSMOOTH. |
80 |
|
- ctrl_map_ini_gen.F : replace #ifdef ALLOW_SMOOTH_CORREL3D with |
81 |
|
#ifdef ALLOW_SMOOTH, IF ctrlSmoothCorrel3D, IF useSMOOTH. |
82 |
|
- ctrl_set_pack_xy[z].F, ctrl_set_unpack_xy[z].F: |
83 |
|
replace ALLOW_SMOOTH_CORREL2D/3D brackets with ctrlSmoothCorrel2D/3D. |
84 |
|
o pkg/smooth: |
85 |
|
- remove un-necessary includes. |
86 |
|
- remove ALLOW_SMOOTH*D and ALLOW_SMOOTH_CORREL*D brakets. |
87 |
|
Those CPP options were never defined, and not necessary. |
88 |
|
o model/src: |
89 |
|
- fix CPP brackets in ini_mixing.F (GM_background_K belongs to gmredi). |
90 |
|
- standard treatment of useSMOOTH. |
91 |
|
o pkg/autodiff: |
92 |
|
- add LOGICAL useGGL90inAdMode, useGGL90inFwdMode, useSALT_PLUMEinAdMode, |
93 |
|
useSALT_PLUMEInFwdMode allowing to switch off ggl90 and salt_plume adjoint. |
94 |
|
o pkg/exch2: |
95 |
|
- add "if usingMPI" where it was missing (in exch2_rx?_cube*.template) |
96 |
|
+ start to remove ALWAYS_USE_MPI in source code that TAF does not see. |
97 |
|
- make new S/R W2_MAP_PROCS out of w2_eeboot.F to setup exch2 CommonFlag |
98 |
|
and tiles to process mapping. |
99 |
|
- re-arrange 3 header files (W2_EXCH2_PARAMS.h, _BUFFER.h & _TOPOLOGY.h) |
100 |
|
moving /EXCH2_FILLVAL_RX/ & /W2_EXCH2_COMMFLAG/ to W2_EXCH2_TOPOLOGY.h; |
101 |
|
rename "exch2_tProc" to "W2_tileProc" and move seting of e2FillValue_RX |
102 |
|
from w2_eeboot.F to w2_readparms.F. |
103 |
|
o eesupp/src: |
104 |
|
- set exchNeedsMemsync & exchUsesBarrier according to "usingMPI" (instead of |
105 |
|
ALWAYS_USE_MPI) + moved from ini_communication_patterns.F to eeset_parms.F; |
106 |
|
also move setting of exchCollectStatistics from exch_init.F to eeset_parms.F |
107 |
|
- in eedie.F and all_proc_die.F, always call MPI_FINALIZE when ALLOW_USE_MPI |
108 |
|
is defined (remove condition on usingMPI) to match cases where MPI_INIT |
109 |
|
has been called. |
110 |
|
- fix bug I put in previous modif of exch_sm_3d_rx.template (not much used); |
111 |
|
- change default (non-MPI) value of myPid (from 1 to 0) to match MPI settings |
112 |
|
(so that on 1 proc, get same myPid independently of usingMPI) |
113 |
|
- add a stop if multi-procs but usingMPI=F ; stop also if usingMPI=T and |
114 |
|
useCubedSphereExchange with EXCH-1 (safer). |
115 |
|
- add "if usingMPI" where it was missing (+ start to remove ALWAYS_USE_MPI). |
116 |
|
o pkg/ggl90: |
117 |
|
- fix bug in filling corner mask array "mskCor" (FILL_CS_CORNER_TR_RL call) |
118 |
|
used in ALLOW_GGL90_SMOOTH code. |
119 |
|
o pkg/flt: |
120 |
|
- fix typo in FLT_WITHOUT_Y_PERIODICITY (the effect was to ignore stopping |
121 |
|
floats at the Southern end of the domain). |
122 |
|
o eesupp/inc: |
123 |
|
- add a list of CPP-options which can be set by Makefile (DEF_IN_MAKEFILE.h); |
124 |
|
but not used (and not to be used) anywhere, just for documentation. |
125 |
|
o pkg/autodiff, exf, ptracers & seaice (AD & TL output): |
126 |
|
- comment out the very nasty "writeBinaryPrec = writeStatePrec" line, since: |
127 |
|
a) can damage previously written records in any other multi-record files. |
128 |
|
b) if needed, can easily do it cleanly, by calling S/R WRITE_REC_3D_RL/S. |
129 |
|
o tools/genmake2: |
130 |
|
- define variable MAKE within Makefile and use it in place of 'make'; |
131 |
|
- update AD_CONFIG.h (if needed) before building any of the executable |
132 |
|
(forward, adjoint or tangent-lin); this involves a 2nd invocation of make; |
133 |
|
and add the usual #ifndef/#define AD_CONFIG_H in AD_CONFIG.h; |
134 |
|
- always print usual warning comment 'this file is automatically generated' |
135 |
|
from script convert_cpp_cmd2defines (+ remove this warning from genmake2). |
136 |
|
o pkg/cheapaml: |
137 |
|
- updated to correspond to the code used in accepted paper (Nico Wienders). |
138 |
|
o pkg/exf: |
139 |
|
added run time switch useAtmWind to replace ALLOW_ATM_WIND cpp switch. |
140 |
|
ALLOW_ATM_WIND now just sets the useAtmWind default (see exf_readparms.F) |
141 |
|
and force defines ALLOW_BULKFORMULAE (EXF_OPTIONS.h). |
142 |
|
o pkg/exf, autodiff, ctrl, ecco and seaice: |
143 |
|
remove ALLOW_ATM_WIND brackets, or replace them with useAtmWind ones. |
144 |
|
o pkg/ctrl, ecco: |
145 |
|
allow to compile both ALLOW_U/VSTRESS_CONTROL and ALLOW_U/VWIND_CONTROL. |
146 |
|
Depending on useAtmWind, one is inactive, and the other is active (see |
147 |
|
exf_getffields.F/exf_getsurfacefluxes.F). |
148 |
|
o tools/genmake2: |
149 |
|
- improve error msg when failing in 2nd test of check_for_broken_Ff |
150 |
|
- fix option '-makefile' short-name (-mf according to genmake2 -help) |
151 |
|
- source local config file "genmake_local" just after processing the genmake2 |
152 |
|
command arguments (previously done before processing arg). |
153 |
|
o pkg/grdchk: |
154 |
|
- add a call to TURNOFF_MODEL_IO for Tangent-Linear run (in this case, call |
155 |
|
to this routine from cost_final has been dropped in g_cost_final) |
156 |
|
- improve printed information (more explicit msg, no longer using "print *," |
157 |
|
fix fcpertminus printed value). |
158 |
|
o model/src: |
159 |
|
- move call to COST_INIT_VARIA out of packages_init_variables.F directly |
160 |
|
into initialise_varia.F (just after calling PACKAGES_INIT_VARIABLES): |
161 |
|
this prevents getting cost-funct reset in adinitialise_varia recomputations. |
162 |
|
o pkg/dic: |
163 |
|
- move initialisation of "freefe" array from from dic_biotic_forcing.F to |
164 |
|
fe_chem.F: this prevents TAF from dropping this initialisation in TLM code. |
165 |
|
o pkg/grdchk: |
166 |
|
- grdchk_main.F : added CALL GLOBAL_SUM_INT( ierr , myThid ) |
167 |
|
- grdchk_getadxx.F etc. : omit I/O if ierr.EQ.0 |
168 |
|
o pkg/autodiff/ctrl/grdchk : |
169 |
|
- attempt to sort out what works and what does not, |
170 |
|
with regard to I/O, and to warn/stop accordingly. |
171 |
|
o pkg/grdchk: |
172 |
|
- attempt to fix grdchk in the case of useSingleCpuIO. |
173 |
|
- "if ( myProcId .EQ. grdchkwhichproc .AND. ierr .EQ. 0 )" |
174 |
|
statements were moved from grdchk_main.F and into |
175 |
|
grdchk_getadxx.F grdchk_getxx.F grdchk_setxx.F. |
176 |
|
- a global sum is applied to record numbers icvrec/layer |
177 |
|
out of grdchk_loc so that they be passed to I/O routines. |
178 |
|
o pkg/smooth: |
179 |
|
- fix recomputation that appeared with nlfs : use smooth_recip_hFacC, |
180 |
|
smooth_hFacW, smooth_hFacS (copied at init_fixed) in 3d case. |
181 |
|
o verification/testreport: |
182 |
|
- add option '-tlm' for Tangent-Linear test (-adm,-oad,-tlm are all exclusive); |
183 |
|
- can also use compressed (gzip) reference output file; |
184 |
|
- change format (in pkg/grdchk) of TLM output (+ add forward gradient) |
185 |
|
to be easier to collect by testreport. |
186 |
|
- update existing TLM output (half out-of-date by more than 3 yrs). |
187 |
|
- report NETCDF & LAPACK settings (from genmake2) in output dir. |
188 |
|
- report run-time in each individual summary (previously only for AD test); |
189 |
|
for TLM test, add also TAF version. |
190 |
|
- add daily TLM test on (old) aces cluster + weekly test on baudelaire. |
191 |
|
|
192 |
checkpoint63r (2012/08/15) |
checkpoint63r (2012/08/15) |
193 |
o tools/genmake2: |
o tools/genmake2: |
194 |
- fix Makefile for target ftlall (+ update files to clean after make ftlall) |
- fix Makefile for target ftlall (+ update files to clean after make ftlall) |