4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
5 |
============================== |
============================== |
6 |
|
|
7 |
|
o verification/adjustment.cs-32x32x1: |
8 |
|
add a "minimal" set-up test (code_min & input_min) which only compiles |
9 |
|
eesupp/src and the 2 pkgs (exch2 & debug) which don't need model/src & inc. |
10 |
|
o tools/genmake2 |
11 |
|
- extend pkg_depend rules to model & eesupp (e.g., if model depend on [pkg])) |
12 |
|
- fix details of checking for mnc & profile pkg when NetCDF is unavailable. |
13 |
|
o pkg/exf: merge exf_bulk_largeyeager04 into exf_bulkformulae because these |
14 |
|
files are nearly the same anyway. |
15 |
|
Step 1: make exf_bulkformulae look like exf_bulk_largeyeager04: |
16 |
|
- redefinition of tau(i,i) (by a factor of ustar) |
17 |
|
- 1./exf(Const/T) -> exf(-Const/T) (big change in the results) |
18 |
|
- replace rhonil by rhoConstFresh in computation of evap (bug fix) |
19 |
|
- replace u/vstress=tau*rd*ws*cw by tau*rd*u/vwind (changes results) |
20 |
|
- replace |
21 |
|
rd(i,j)= rdn(i,j)/(exf_one - rdn(i,j)/karman*psimh ) |
22 |
|
shn = sh(i,j,bi,bj)*rd(i,j)/rdn(i,j) |
23 |
|
by |
24 |
|
shn = sh(i,j,bi,bj)/(exf_one - rdn(i,j)/karman*psimh ) |
25 |
|
(changes results) |
26 |
|
-> exf_bulkformulae v1.22, |
27 |
|
update verification experiments lab_sea, offline_exf_seaice, seaice_obcs |
28 |
|
(lab_sea: output.hb87.txt 1.11, output.lsr.txt 1.25, output.salt_plume.txt 1.13, |
29 |
|
output.txt 1.63 |
30 |
|
offline_exf_seaice: output.seaicetd.txt 1.7 |
31 |
|
seaice_obcs: output.txt 1.10) |
32 |
|
Step 2: do the actual merge and remove exf_bulk_largeyeager04.F |
33 |
|
-> exf_bulkformulae v1.23, |
34 |
|
unfortunately requires updating lab_sea.salt_plume (6 matching digits), |
35 |
|
and global_ocean.cs32x15.icedyn (only 11 matching digits) |
36 |
|
Also update output_adm of exp. lab_sea (all 4) and offline_exf_seaice. |
37 |
|
o adjoint of exch-1: |
38 |
|
- pass high-level EXCH S/R through TAF (eesupp/src/exch_ad_diff.list) |
39 |
|
(note: was already like this when using EXCH-2) and remove flow dir |
40 |
|
and hand-written AD version of high-level EXCH S/R (pkg/autodiff); |
41 |
|
- provide flow-dir for low-level exch1 S/R (eesupp/src/exch1_ad.flow); |
42 |
|
- update pkg/exch2 diff list. |
43 |
|
|
44 |
|
checkpoint62g (2010/05/19) |
45 |
|
o eesupp/src (exch-1): |
46 |
|
- split exch_rx.F into Forward S/R (exch1_rx.F) and Adjoint S/R (exch1_rx_ad.F) |
47 |
|
and remove argument "simulationMode"; same thing for all exch_*rx_cube.F |
48 |
|
(renamed to exch1_*rx_cube.F). |
49 |
|
- change accordingly exch_*_rx.template and AD version in pkg/autodiff |
50 |
|
to call new exch1 S/R ; update Makefile and .cvsignore. |
51 |
|
- replace call to exch2_xy_rx S/R with exch2_*3d_rx(myNz=1) (will remove |
52 |
|
exch2_xy_rx later) |
53 |
|
Note: after updating your code, need to run "make clean" in eesupp/src |
54 |
|
o test experiment bottom_ctrl_5x5: |
55 |
|
new output_adm.txt with fewer grdchk & less monitor output. |
56 |
|
o model/src: |
57 |
|
remove the call to ini_cori in initialise_varia.F (just keep the usual one |
58 |
|
in initialise_fixed.F). More consistent with coriolis & horizontal grid |
59 |
|
beeing treated in the same way (ini_grid is not called a 2nd time). |
60 |
|
o pkg/exf: |
61 |
|
- rewrite exf_bulk_largeyeager04.F for better vectorization, changes |
62 |
|
global_ocean.cs32x15/input.icedyn (11 digits of agreement), updated |
63 |
|
- fix adjoint (used worked for ALLOW_ATM_WIND defined, works for this now) |
64 |
|
- cosmetic changes to exf_bulkformulae.F to make it look more like |
65 |
|
exf_bulk_largeyeager04.F |
66 |
|
o eesupp/src (exch-1): |
67 |
|
- Fix AD EXCH code for 2-D domain special cases (Nx=1 or Ny=1). |
68 |
|
- Separate buffer filling and MPI sending: allow EXCH-1 to work for local |
69 |
|
array (non-shared) when using MPI+MTH. Also reduces number of BARRIER |
70 |
|
(even without using MPI). |
71 |
|
- Message mode: move RecvAck setting (indicator of buffer being ready) |
72 |
|
from send_put to recv_get S/R (was useless before, but not sure if |
73 |
|
it's much more useful now); |
74 |
|
- Switch the order of sync: MPI-proc 1rst and then threads; |
75 |
|
- take out spin-waiting code (#undef EXCH_USE_SPINNING), use BARRIER instead. |
76 |
|
- fix AD code for communication mode ='get' (but not used). |
77 |
|
o model/src: |
78 |
|
initialise dPhiHydX,dPhiHydY inside calc_grad_phi_hyd.F (so that TAF does |
79 |
|
not drop it in S/R mddynamics, producing NaN in hs94.1x64x5 AD-test exp). |
80 |
|
o utils/matlab/rdmnc.m: adapt to be able to use the generic matlab netcdf API |
81 |
|
o pkg/ctrl: require a specific set of control variables and exf-flags for |
82 |
|
adjoint of seaice (stop in ctrl_check.F if conditions are not satisfied) |
83 |
|
o model/src: |
84 |
|
a) 1rst version of a new driver S/R to turn-off output freq and flags after the |
85 |
|
1rst forward sweep, to avoid over-writing output files during recomputations |
86 |
|
recomputations (& grdchk); called from cost_final.F (& ctrl_init_ctrlvar.F ?) |
87 |
|
b) 2nd version: |
88 |
|
- add specific pkg S/R for seaice & ptracers (called from S/R TURNOFF_MODEL_IO); |
89 |
|
- switch-off more output flags (exf_monFreq, chkPtFreq, dumpInitAndLast, |
90 |
|
writePickupAtEnd) + seaice & ptracers output; |
91 |
|
- do not switch off MNC flags (otherwise, won't get admonitor netcdf file) |
92 |
|
o model/src: |
93 |
|
- new S/R rotate_uv2en_rl (..._rs) that rotates vector fields. |
94 |
|
o pkg/exch2: |
95 |
|
- remove "simulationMode" from arg. list of S/R EXCH2_RX1,2_CUBE ; |
96 |
|
- add argument "signOption" to EXCH2_RX1_CUBE (will be needed for SM exch). |
97 |
|
o pkg/autodiff & exch2: |
98 |
|
process exch_uv_agrid_3d_rx.f & exch2_uv_agrid_3d_rx.f through TAF |
99 |
|
(for now, only if ALLOW_EXCH2 is defined). |
100 |
|
o pkg/autodiff: |
101 |
|
- include PACKAGES_CONFIG.h in exch*ad.flow & exch*ad.F |
102 |
|
(not supposed to be in CPP_OPTIONS.h) to get ALLOW_EXCH2 defined correctly. |
103 |
|
+ clean-up comments (_RS is not always real*4) |
104 |
|
- fix bug in ADEXCH_UV_3D_RX (without EXCH2, was calling the forward exch S/R) |
105 |
|
- add a stop in ADEXCH_UV_AGRID_3D_RX if trying to use it with EXCH2: |
106 |
|
was calling a forward exch2, which is wrong ; used in SEAICE & EXF pkgs. |
107 |
|
o testreport (AD-tests): |
108 |
|
no longer link files from dir "../input" (1rst link files from |
109 |
|
"../input_ad.[xx]" if secondary test "xx" and then from "../input_ad") |
110 |
|
|
111 |
|
checkpoint62f (2010/04/29) |
112 |
|
o model/src/the_main_loop.F: |
113 |
|
adjust store directives for seaice evp and multicategory according to |
114 |
|
pkg/ecco/the_main_loop.F |
115 |
|
o adjoint: |
116 |
|
global_sum_ad was incomplete (noted many times by many people ;) |
117 |
|
nice description by Jean U. as follows: |
118 |
|
* allreduce(var) in global_sum.F can be interpreted as |
119 |
|
reduce(sum) followed by broadcast of sum. |
120 |
|
* The adjoint of that is |
121 |
|
sum(adjoint) followed by broadcast(summed adjoint value) |
122 |
|
or in other words, allreduce(sum, adjoint vars) |
123 |
|
We were missing the sum(adjoint)! |
124 |
|
Benign, except(!) for "global" scalars appearing in tile loops (bi,bj) |
125 |
|
(noted by J.M. Campin, e.g., in cost_ssh_mean, variable offset). |
126 |
|
The fix should produce 16-digit accuracy between |
127 |
|
single-CPU multi-tile (e.g. nSx=2, nPx=1) and |
128 |
|
multi-CPU single-tile (e.g. nSx=1, nPx-2) |
129 |
|
We'll probably change this yet again shortly |
130 |
|
o eesupp/src: |
131 |
|
- fix global_sum_r4 (wrong from previous modif in Jun 2009; ever used ?) |
132 |
|
o pkg/icefront |
133 |
|
- ALLOW_SUBGLACIAL_RUNOFF option adds capability for time-evolving |
134 |
|
specification of addmass array based on a 2D field, which is added |
135 |
|
at bottommost wet level at each horizontal location. |
136 |
|
o model/src: |
137 |
|
horizontal grid initialisation (other than curvilinear): add code for Exch2 IO |
138 |
|
layout; (not always compatible with delX,delY setting; commented out for now). |
139 |
|
o pkg/exch2: |
140 |
|
change tile to processor setting (case nSy*nPy > 1): allows default model mapping |
141 |
|
(myX and myYGlobalLow) to be used for simple (1 facet, no blank tile) config. |
142 |
|
|
143 |
|
checkpoint62e (2010/04/15) |
144 |
|
o pkg/obcs: |
145 |
|
- fix wrong index in OBCS_SPONGE_V for Eastern-OB (reported by David Hebert). |
146 |
|
o pkg/dic: |
147 |
|
-fix bug in fe_chem.F for multi-tile (nSx*nSy > 1) set-up; |
148 |
|
(note: not presently tested in any forward verification exp.) |
149 |
|
update tutorial_global_oce_biogeo AD output file. |
150 |
|
o pkg/exf: |
151 |
|
- account for the variation of albedo as a function of zenith angle |
152 |
|
by defining ALLOW_ZENITHANGLE in EXF_OPTIONS.h |
153 |
|
o build_options: |
154 |
|
- rename "linux_ia64_ifort+mpi_ice_nas" to "linux_amd64_ifort+mpi_ice_nas" |
155 |
|
o pkg/dic: |
156 |
|
change DIC atmos-CO2 box (method 3, dic_int1=3) initialisation & restart: |
157 |
|
- use binary pickup file for restart; |
158 |
|
- initialise atmos-CO2 box from dic_pCO2 parameter; |
159 |
|
- fix multi-threaded. |
160 |
|
o pkg/rbcs: |
161 |
|
- clean-up RBCS code: add RBCS_OPTIONS.h file ; fix multi-threaded ; |
162 |
|
change (+ fix IO calls) type of array RBC_mask from _RL to _RS. |
163 |
|
- add RBCS to exp4 passive tracer 01 (rbcs was previously untested) |
164 |
|
and update output.txt |
165 |
|
o pkg/kpp: |
166 |
|
- add diagnostics for KPP non-local flux of Temp, Salinity and pTracers; |
167 |
|
- change diagnostic of KPP non-local term: output average of KPPghat*KPPdiffKzS |
168 |
|
(= ratio of KPP non-local (salt) flux relative to surface-flux) when |
169 |
|
using pkg/timeave & pkg/diagnostics (diag 'KPPghatK' replaces 'KPPghat '); |
170 |
|
- change pkg/offline to read-in average KPPghat*KPPdiffKzS from file |
171 |
|
KPP_ghatKFile (instead of the average of KPPghat from file KPP_ghatFile). |
172 |
|
o pkg/offline: |
173 |
|
fix bug in S/R OFFLINE_FIELDS_LOAD regarding surfaceForcing conversion factor |
174 |
|
(bug was added in version 1.10 on 2007-08-23). |
175 |
|
o pkg/offline: |
176 |
|
- no need to specify periodicExternalForcing, etc ... to use offline pack |
177 |
|
- offline input files no longer just single prec --> parameter offlineLoadPrec |
178 |
|
o verification/lab_sea: |
179 |
|
change AD test to multi-tile setup and update all 4 AD output. |
180 |
|
NOTE: while cost function values remain very similar, |
181 |
|
gradients are very different, although gradient checks are self-consistent |
182 |
|
Should try to get to the bottom of this (cost function summation order?) |
183 |
|
o pkg/dic: |
184 |
|
in carbon_chem.F, replace log10 (AUTODIFF) with explicit form to avoid |
185 |
|
a weakness of TAF that drops part of double precision in log10 derivative. |
186 |
|
<= changes removed since problem has been fixed in current TAF version 1.9.71 |
187 |
|
update AD output of exp. tutorial_dic_adjoffline & tutorial_global_oce_biogeo. |
188 |
|
o pkg/ecco: |
189 |
|
add missing "_d 0" in cost_readers.F, cost_readtopex.F & cost_readtopexmean.F |
190 |
|
update all 4 lab_sea AD output. |
191 |
|
|
192 |
checkpoint62d (2010/03/22) |
checkpoint62d (2010/03/22) |
193 |
o build_options: |
o build_options: |
194 |
remove '-r8' (and '-i4') from linux_*pgf and linux_*ifort optfiles. |
remove '-r8' (and '-i4') from linux_*pgf and linux_*ifort optfiles. |