4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
5 |
============================== |
============================== |
6 |
|
|
7 |
o pkg/exf/exf_bulk_largeyeager04.F: rewrite for better vectorization, changes |
o eesupp/src: |
8 |
|
fix flow dir for exch1_z_RX_cube, exch1_uv_RX_cube & exch1_bg_RX_cube |
9 |
|
o pkg/seaice/seaice_growth.F |
10 |
|
Revision 1.89 of seaice_growth.F completes the overall october 2010 revision and |
11 |
|
merging process. The merged routine has two branches, and one activates the 'evolution' |
12 |
|
branch by setting SEAICE_GROWTH_LEGACY to undef (at the top of the routine for now). |
13 |
|
In summary, here is what the '10/10 revision' did consist of |
14 |
|
part 1: seaice_growth.F revisions 1.71 to 1.75 |
15 |
|
- Preliminary seaice_growth.F clean-up, keeping test results strictly unchanged. |
16 |
|
The emphasis here was on improving readability (e.g. by trying to use meaningful |
17 |
|
variable names) and modularity (e.g. by adding arrays to store heff etc. updates). |
18 |
|
part 1.5: |
19 |
|
- revising the lab_sea experiment to make it a more useful (i.e. stable |
20 |
|
and representative of an actual model run). This was done by generating a |
21 |
|
(winter) pickup to start from, switching back the main experiment dynamic |
22 |
|
solver from evp to lsr, having one of the experiments use the free drift solution. |
23 |
|
part 2: seaice_growth.F revisions 1.76 to 1.85 |
24 |
|
- Sorting out hard-coded constants that prevented heat and fresh water conservation. |
25 |
|
- Sorting out the various fields units and sign (set early on and reset at the end) |
26 |
|
to avoid numerous scalings/re-scalings. By v1.85 the basic unit for ice, heat and |
27 |
|
fresh water stock is 'effective ice meters', and >0 implies more ice/snow. |
28 |
|
- Using tendency terms arrays (d_HEFF* etc.) to alleviate remaining code locks, |
29 |
|
and simplifying logical expressions to further improve readibility. |
30 |
|
part 3: seaice_growth.F revisions 1.86 to 1.89 |
31 |
|
- Re-organize the blocks of code more logically. |
32 |
|
- Merging the '_if.F' pieces that add something new. |
33 |
|
o pkg/thsice: |
34 |
|
- vectorize thsice_solve4temp and thsice_get_exf. |
35 |
|
Problem: thsice_solve4temp has an unvectorizable statement that |
36 |
|
might also be problematic for TAF |
37 |
|
- thsice_get_exf: add the ALLOW_BULK_LARGEYEAGER04 CPP-flag. This |
38 |
|
does not change the verifcation experiment, but will change all |
39 |
|
experiments that have this flag undefined (not tested in verification); |
40 |
|
a little reorganization to save a few 2D fields and recomputations |
41 |
|
- 1st step of revisiting thsice adjoint (major recomp. sorted out) |
42 |
|
o pkg/obcs: add support for exch2 |
43 |
|
- the position of the boundary and prescribed values are specified using |
44 |
|
a global domain with exch2 facets stacked |
45 |
|
- in x for N,S boundaries (like W2_mapIO=-1) |
46 |
|
- in y for E,W boundaries (so E,W boundaries do not overlap) |
47 |
|
o pkg/mdsio: add support for writing global sections with exch2 |
48 |
|
file layout is as for obcs described above: |
49 |
|
- for x-z sections, facets are stacked in x (like W2_mapIO=-1) |
50 |
|
- for y-z sections, facets are stacked in y |
51 |
|
o pkg/thsice: |
52 |
|
- change a few routines for better vectorization: thsice_albedo.F, |
53 |
|
thsice_step_temp.F, and in thsice_calc_thickn.F. Results are |
54 |
|
not affected, but in particular the latter routine has changed |
55 |
|
dramatically (not necessarily easier to read now). |
56 |
|
- re-arrange calculation in THSICE_SOLVE4TEMP: move outside of i,j loops |
57 |
|
iteration loop and call to S/R THSICE_GET_BULKF & THSICE_GET_EXF. |
58 |
|
o verification/lab_sea: |
59 |
|
- overhaul of the forward model experiments. The main motivation |
60 |
|
for this revision is that lab_sea has proven quite fragile (when |
61 |
|
changing the sea-ice code, and switching platforms) to a point |
62 |
|
when failures were very hard to attribute to specific code problems. |
63 |
|
- the suspected reason for this fragility is noise amplification by evp so, |
64 |
|
to start with, we decided to replace evp with lsr in the main experiment. |
65 |
|
- the input.lsr experiment thus became obsolete and was removed. |
66 |
|
- it was replaced with the input.fd experiment that uses an |
67 |
|
analytical "free drift" solution as the ice velocity field. |
68 |
|
- Taken together, input (lsr), input.fd (free drift) and input.hb87 (evp) |
69 |
|
will hopefully allow us to discriminate problems due to dynamic solvers |
70 |
|
from problems due to the thermodynamic code. input.salt_plume (lsr) allows |
71 |
|
us to test additional options, along with the salt plume package. |
72 |
|
- Finally a pickup was generated (at the end of a one year run) to |
73 |
|
make the tests more representative of an actual model run. Previously |
74 |
|
the tests starting point was a 1m thickness of ice everywhere. |
75 |
|
o pkg/seaice: |
76 |
|
- introducing S\R seaice_freedrift.F that compute the |
77 |
|
analytical solution to an approximate ice momentum |
78 |
|
equation (i.e. the simple stress/coriolis balance). |
79 |
|
- define SEAICE_ALLOW_FREEDRIFT to trigger the computation |
80 |
|
of the free drift velocity field, and enable its use. |
81 |
|
To actually use the free drift solution as the model ice |
82 |
|
velocity field set SEAICEuseFREEDRIFT to .TRUE. in data.seaice. |
83 |
|
o move "useOASIS" from PARAMS.h to EEPARAMS.h and read it from "eedata" |
84 |
|
|
85 |
|
checkpoint62l (2010/10/04) |
86 |
|
o pkg/seaice: |
87 |
|
-fix the USE_ORIGINAL_SBI version (for variable freezing point); |
88 |
|
-use "celsius2K" in SEAICE_VARIABLE_FREEZING_POINT and in new |
89 |
|
version (with USE_ORIGINAL_SBI undef); |
90 |
|
o add hooks for new packages OASIS, the package will follow |
91 |
|
(hooks exclude the seaice pkg for now) |
92 |
|
o pkg/autodiff: ALLOW_AUTODIFF_WHTAPEIO |
93 |
|
Option to use mdsio_read_whalos.F/mdsio_write_whalos.F, |
94 |
|
rather than mdsio_readvector.F/mdsio_writevector.F, within |
95 |
|
adread_adwrite.F to read/write adjoint-related tapes to disk. |
96 |
|
o eesupp and pkg/mdsio: ALLOW_WHIO |
97 |
|
Option to write/read 2D fields to files including tiles halos. |
98 |
|
The main purpose is for adjoint related "tape I/O". |
99 |
|
The secondary purpose is for debugging phases. |
100 |
|
A word on those rather specifically intended purposes. |
101 |
|
The code was meant to be stripped down to the bare minimum. |
102 |
|
Typically, there is a unique and basic mapping to the global |
103 |
|
buffer (tile by tile, then proc by proc, then level by level), and |
104 |
|
byteswaps and meta-files are omitted. Indeed the (*_WH*) code is not |
105 |
|
meant to generate user friendly outputs of various shapes and forms. |
106 |
|
Rather it is intended, for the knowledgeabale developers, |
107 |
|
to be largely independent of the many mdsio options, to allow |
108 |
|
outputs of tiles+halos in debugging phases and in adjoint runs, |
109 |
|
and to allow some flexibility in tuning (adoint) I/O depending on each |
110 |
|
file system behavior. With regard to tuning, most of the potential knobs |
111 |
|
(to leave files open, to allocate the I/O indep. of usesinglecpuio, |
112 |
|
to specify directories, to specify the typical size of I/O calls, etc.) |
113 |
|
are not yet included, and will be added if they prove useful. |
114 |
|
o pkg/seaice: |
115 |
|
fix uninitialised arrays in USE_ORIGINAL_SBI version of seaice_solve4temp.F |
116 |
|
o pkg/seaice: |
117 |
|
new S/R seaice_solve4temp.F: this is the merged version of Ian Fenty |
118 |
|
code (seaice_budget_ice_if.F) which enable to recover old standard |
119 |
|
code (seaice_budget_ice.F) when option USE_ORIGINAL_SBI is defined. |
120 |
|
temporary: add "#define USE_ORIGINAL_SBI" at the top of seaice_solve4temp.F. |
121 |
|
o model/src (rStar): |
122 |
|
use simple average (not area weighted) of rStarFac at U,V point when using |
123 |
|
vectorInvariant and selectKEscheme=1,3 (consistent with mom. vertical advection) |
124 |
|
update output of exp. fizhi-cs-32x32x40 and fizhi-cs-aqualev20. |
125 |
|
|
126 |
|
checkpoint62k (2010/09/17) |
127 |
|
o adjoint |
128 |
|
* changed verification/global_ocean.cs32x15 ; adjoint now tests |
129 |
|
Adams-Bashforth 3 + NLFS + RSTAR |
130 |
|
* some more modifs required for ECCO-GODAE v4 |
131 |
|
o pkg/seaice |
132 |
|
add SEAICE_AGE_VOL option to use volume instead of area for |
133 |
|
integrating SEAICE_AGE |
134 |
|
o model/src: |
135 |
|
first check-in of sigma & hybrid-sigma coordinate code ; also includes |
136 |
|
non-linear free-surface parts ; but pressure-gradient not yet coded. |
137 |
|
o pkg/ptracers: |
138 |
|
option for ptracer to convert Salt Relax into additional EmP. |
139 |
|
tested in exp. global_ocean.90x40x15.dwnslp (update output) |
140 |
|
|
141 |
|
checkpoint62j (2010/08/25) |
142 |
|
o pkg/ggl90: |
143 |
|
other minor changes: remove 3-D temp array gTKE (+ add argument myIter) |
144 |
|
o pkg/dic: car_flux.F, phos_flux.F |
145 |
|
- reduce number of exp-calls in order to improve performance |
146 |
|
- clean up a little (replace fields by their macros, etc., fix |
147 |
|
typo in CPP-flag) |
148 |
|
o pkg/ggl90: minor changes |
149 |
|
- GGL90diffKrS was removed --> always use GGL90diffKr |
150 |
|
- GGL90viscAr was removed --> replaced with GGL90viscArU, GGL90viscArV |
151 |
|
- hack of mxlMaxFlag=2 --> ensure mixing between first and second level |
152 |
|
(commented out for now) |
153 |
|
- change in max/min operations to ensure that smoothing is ok |
154 |
|
- smoothing of GGL90viscAr was moved to ggl90_calc.F (as done for GGL90diffKr) |
155 |
|
- always use diffKrNrT as background profile (i.e. never use diffKr field) |
156 |
|
o pkg/cfc: bug fix, AREA(i,j,1,bi,bj) -> AREA(i,j,bi,bj) |
157 |
|
o pkg/dic: |
158 |
|
- model/src/impldiff.F: if TARGET_NEC_SX, extend loop ranges for better |
159 |
|
vectorization |
160 |
|
- small bug fix in insol for the unlikely case of a cartesian grid |
161 |
|
- towards vectorizing the dic package: bio_export.F, car_flux.F, phos_flux.F |
162 |
|
+ now vectorize, which also removes adjoint recomputations; |
163 |
|
+ replace integer maksp1 by _RL maskp1 |
164 |
|
+ replace a**x by exp(x*log(a)) in phos_flux.F for speed |
165 |
|
|
166 |
|
checkpoint62i (2010/07/21) |
167 |
|
o pkg/flt: |
168 |
|
- with OBCS: stop floats which enter the OB region |
169 |
|
o model/src: |
170 |
|
- use tempVertAdvScheme & saltVertAdvScheme (instead of horizontal ones) |
171 |
|
for implicit vertical advection. |
172 |
|
o pkg/seaice: |
173 |
|
- account for melting in seaice age evolution. |
174 |
|
- remove double counting of seaice ridging effect on seaice age. |
175 |
|
|
176 |
|
checkpoint62h (2010/06/18) |
177 |
|
o pkg/exch2: |
178 |
|
- remove exch2_*xy_rx files: call was replaced with exch2_*3d_rx(myNz=1). |
179 |
|
o tools/genmake2 |
180 |
|
- remove "-pdefault" option: redundant with the use of "packages.conf"; |
181 |
|
- remove file pkg/pkg_default : set the default as a pkg group |
182 |
|
("default_pkg_list") defined in pkg/pkg_groups; |
183 |
|
- fix for case pkg/pkg_depend is missing (was hanging before); |
184 |
|
- add option "-pgroups" to specify a different pkg_groups than the default. |
185 |
|
o verification/adjustment.cs-32x32x1: |
186 |
|
add a "minimal" set-up test (code_min & input_min) which only compiles |
187 |
|
eesupp/src and the 2 pkgs (exch2 & debug) which don't need model/src & inc. |
188 |
|
o tools/genmake2 |
189 |
|
- extend pkg_depend rules to model & eesupp (e.g., if model depend on [pkg])) |
190 |
|
- fix details of checking for mnc & profile pkg when NetCDF is unavailable. |
191 |
|
o pkg/exf: merge exf_bulk_largeyeager04 into exf_bulkformulae because these |
192 |
|
files are nearly the same anyway. |
193 |
|
Step 1: make exf_bulkformulae look like exf_bulk_largeyeager04: |
194 |
|
- redefinition of tau(i,i) (by a factor of ustar) |
195 |
|
- 1./exf(Const/T) -> exf(-Const/T) (big change in the results) |
196 |
|
- replace rhonil by rhoConstFresh in computation of evap (bug fix) |
197 |
|
- replace u/vstress=tau*rd*ws*cw by tau*rd*u/vwind (changes results) |
198 |
|
- replace |
199 |
|
rd(i,j)= rdn(i,j)/(exf_one - rdn(i,j)/karman*psimh ) |
200 |
|
shn = sh(i,j,bi,bj)*rd(i,j)/rdn(i,j) |
201 |
|
by |
202 |
|
shn = sh(i,j,bi,bj)/(exf_one - rdn(i,j)/karman*psimh ) |
203 |
|
(changes results) |
204 |
|
-> exf_bulkformulae v1.22, |
205 |
|
update verification experiments lab_sea, offline_exf_seaice, seaice_obcs |
206 |
|
(lab_sea: output.hb87.txt 1.11, output.lsr.txt 1.25, output.salt_plume.txt 1.13, |
207 |
|
output.txt 1.63 |
208 |
|
offline_exf_seaice: output.seaicetd.txt 1.7 |
209 |
|
seaice_obcs: output.txt 1.10) |
210 |
|
Step 2: do the actual merge and remove exf_bulk_largeyeager04.F |
211 |
|
-> exf_bulkformulae v1.23, |
212 |
|
unfortunately requires updating lab_sea.salt_plume (6 matching digits), |
213 |
|
and global_ocean.cs32x15.icedyn (only 11 matching digits) |
214 |
|
Also update output_adm of exp. lab_sea (all 4) and offline_exf_seaice. |
215 |
|
o adjoint of exch-1: |
216 |
|
- pass high-level EXCH S/R through TAF (eesupp/src/exch_ad_diff.list) |
217 |
|
(note: was already like this when using EXCH-2) and remove flow dir |
218 |
|
and hand-written AD version of high-level EXCH S/R (pkg/autodiff); |
219 |
|
- provide flow-dir for low-level exch1 S/R (eesupp/src/exch1_ad.flow); |
220 |
|
- update pkg/exch2 diff list. |
221 |
|
|
222 |
|
checkpoint62g (2010/05/19) |
223 |
|
o eesupp/src (exch-1): |
224 |
|
- split exch_rx.F into Forward S/R (exch1_rx.F) and Adjoint S/R (exch1_rx_ad.F) |
225 |
|
and remove argument "simulationMode"; same thing for all exch_*rx_cube.F |
226 |
|
(renamed to exch1_*rx_cube.F). |
227 |
|
- change accordingly exch_*_rx.template and AD version in pkg/autodiff |
228 |
|
to call new exch1 S/R ; update Makefile and .cvsignore. |
229 |
|
- replace call to exch2_xy_rx S/R with exch2_*3d_rx(myNz=1) (will remove |
230 |
|
exch2_xy_rx later) |
231 |
|
Note: after updating your code, need to run "make clean" in eesupp/src |
232 |
|
o test experiment bottom_ctrl_5x5: |
233 |
|
new output_adm.txt with fewer grdchk & less monitor output. |
234 |
|
o model/src: |
235 |
|
remove the call to ini_cori in initialise_varia.F (just keep the usual one |
236 |
|
in initialise_fixed.F). More consistent with coriolis & horizontal grid |
237 |
|
beeing treated in the same way (ini_grid is not called a 2nd time). |
238 |
|
o pkg/exf: |
239 |
|
- rewrite exf_bulk_largeyeager04.F for better vectorization, changes |
240 |
global_ocean.cs32x15/input.icedyn (11 digits of agreement), updated |
global_ocean.cs32x15/input.icedyn (11 digits of agreement), updated |
241 |
|
- fix adjoint (used worked for ALLOW_ATM_WIND defined, works for this now) |
242 |
|
- cosmetic changes to exf_bulkformulae.F to make it look more like |
243 |
|
exf_bulk_largeyeager04.F |
244 |
o eesupp/src (exch-1): |
o eesupp/src (exch-1): |
245 |
- Fix AD EXCH code for 2-D domain special cases (Nx=1 or Ny=1). |
- Fix AD EXCH code for 2-D domain special cases (Nx=1 or Ny=1). |
246 |
- Separate buffer filling and MPI sending: allow EXCH-1 to work for local |
- Separate buffer filling and MPI sending: allow EXCH-1 to work for local |
249 |
- Message mode: move RecvAck setting (indicator of buffer being ready) |
- Message mode: move RecvAck setting (indicator of buffer being ready) |
250 |
from send_put to recv_get S/R (was useless before, but not sure if |
from send_put to recv_get S/R (was useless before, but not sure if |
251 |
it's much more useful now); |
it's much more useful now); |
252 |
- switch the order of sync: MPI-proc 1rst and then threads; |
- Switch the order of sync: MPI-proc 1rst and then threads; |
253 |
- take out spin-waiting code (#undef EXCH_USE_SPINNING), use BARRIER instead. |
- take out spin-waiting code (#undef EXCH_USE_SPINNING), use BARRIER instead. |
254 |
|
- fix AD code for communication mode ='get' (but not used). |
255 |
o model/src: |
o model/src: |
256 |
initialise dPhiHydX,dPhiHydY inside calc_grad_phi_hyd.F (so that TAF does |
initialise dPhiHydX,dPhiHydY inside calc_grad_phi_hyd.F (so that TAF does |
257 |
not drop it in S/R mddynamics, producing NaN in hs94.1x64x5 AD-test exp). |
not drop it in S/R mddynamics, producing NaN in hs94.1x64x5 AD-test exp). |