| 4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
| 5 |
============================== |
============================== |
| 6 |
|
|
| 7 |
|
o pkg/ctrl/ctrl_get_gen_rec.F: catch the case of xx_genperiod=0. |
| 8 |
|
o pkg/ecco |
| 9 |
|
- enable adjoint of Stevens OB |
| 10 |
|
o pkg/ctrl/ecco/grdchk: |
| 11 |
|
- remove global declaration of tmpfldx/yz and tmpfldx/yz2. |
| 12 |
|
Fortunately, we do not have any tests for the numerous cpp-flagged |
| 13 |
|
option of "obcs as control parameters" so we will never know |
| 14 |
|
if this is an improvement (but at least now things compile). |
| 15 |
|
- remove an obsolete initialization from ecco_cost_init_barfiles.F |
| 16 |
|
- fix some problems within never used ifdefs to make the code compile |
| 17 |
|
o pkg/obcs: |
| 18 |
|
- Refine Interior mask definition to work with more complex OB (e.g. |
| 19 |
|
if OB_Ie < OB_Iw); In case default setting is not sufficient, allows to |
| 20 |
|
read in Interior mask from file. Check if Interior mask is consistent |
| 21 |
|
with OB list of indices. |
| 22 |
|
o pkg/seaice |
| 23 |
|
- beginning code for a seaice tracer package, |
| 24 |
|
but currently only used for seaice age: SEAICE_AGE |
| 25 |
|
SEAICE_num = 4 |
| 26 |
|
iTracer = 1: area-weighted ice age |
| 27 |
|
= 2: volume-weighted ice age |
| 28 |
|
= 3, 4: template for one-year vs. multi-year ice |
| 29 |
|
data.seaice has new namelist SEAICE_PARM03 for arrays related to |
| 30 |
|
SEAICE_num |
| 31 |
|
|
| 32 |
|
checkpoint62t (2011/03/05) |
| 33 |
|
o pkg/obcs: |
| 34 |
|
- obcs_apply_uv.F: change to a 2 steps method, first tangential component; |
| 35 |
|
then normal component. This ensures that the normal component is set |
| 36 |
|
correctly even when it conficts with tangential setting from an other OB. |
| 37 |
|
Affects results of test-experiment seaice_obcs: update the output.txt |
| 38 |
|
o pkg/obcs: |
| 39 |
|
- add code to monitor statistics of dynamical fields at OB. |
| 40 |
|
- take OBCS_BALANCE code out of obcs_calc.F into new S/R OBCS_BALANCE_FLOW : |
| 41 |
|
- fix to OBCS_BALANCE code (right hFac ; deal with simple OB intersection). |
| 42 |
|
- change default: balance the Net inflow through all OB (instead of balancing |
| 43 |
|
each OB separately); keep the old method if setting OBC_balanceFacX < 0. |
| 44 |
|
- switch on useOBCSbalance in dome experiment: only correct Western OB outflow |
| 45 |
|
to match Nortern OB inflow (OBCS_balanceFacN=0). |
| 46 |
|
o pkg/seaice: |
| 47 |
|
- redefined actual sea ice age to be ICEAGE/HEFF ifdef SEAICE_AGE_VOL |
| 48 |
|
o pkg/seaice (seaice_growth v 1.113): |
| 49 |
|
- reverting seaice_growth to v 1.111 in order to have code with exact bit |
| 50 |
|
compatibility with MITgcm_contrib/ifenty/Fenty_seaice_thermo_code_merge |
| 51 |
|
CPP option SEAICE_OCN_MELT_ACT_ON_AREA removed in v 1.112 is reintroduced |
| 52 |
|
runtime flag SEAICEareaFormula is kept but is inactive |
| 53 |
|
o model/src: |
| 54 |
|
- add new param to scale curvilinear grid distance and area when using |
| 55 |
|
a different sphere radius from the grid-files. |
| 56 |
|
- test/check multi-threads EXCH applied to local variable (cg2d_x) |
| 57 |
|
o adjoint: |
| 58 |
|
- enable adjoint of Stevens OB |
| 59 |
|
- add dummy_in_dynamics to enable extraction of ADJrhoinsitu |
| 60 |
|
(a bit delicate because of potentially modified adj. common block structure; |
| 61 |
|
current switch is ALLOW_AUTODIFF_MONITOR_DIAG, could be changed later) |
| 62 |
|
|
| 63 |
|
checkpoint62s (2011/02/15) |
| 64 |
|
o pkg/seaice (seaice_growth v 1.112): |
| 65 |
|
- introduce SEAICEareaFormula run time parameter, which selects the recipe for |
| 66 |
|
computing the ice cover tendency from effective thickness tendencies. Include |
| 67 |
|
Ian Fenty's formula. |
| 68 |
|
- remove 'undef SEAICE_OCN_MELT_ACT_ON_AREA' which may now be achieved using |
| 69 |
|
SEAICEareaFormula. |
| 70 |
|
- simplify notations for the various d_HEFF/d_AREA terms. |
| 71 |
|
o pkg/gmredi: |
| 72 |
|
- add Boundary-Value-Problem bolus transport (Ferrari etal, OM 2010) code |
| 73 |
|
from Christopher Wolfe (UCSD); add partial cell factor in vertical derivative; |
| 74 |
|
add a verification test-run (front_relax.bvp) to test GM_useBVP. |
| 75 |
|
o pkg/seaice (seaice_growth v 1.111): |
| 76 |
|
- for SEAICE_DO_OPEN_WATER_GROWTH, replace Gael's implementation with Ian's. |
| 77 |
|
- the new SEAICE_DO_OPEN_WATER_MELT allows open-water air-sea heat fluxes to |
| 78 |
|
also melt ice. |
| 79 |
|
o pkg/seaice: |
| 80 |
|
- McPhee formula for Ice-Ocean fluxes (provided by Ian Fenty). |
| 81 |
|
- can be used by #define MCPHEE_OCEAN_ICE_HEAT_FLUX in SEAICE_OPTIONS.h |
| 82 |
|
o pkg/cheapaml: |
| 83 |
|
- from Nico Wienders: New version of this package. |
| 84 |
|
o pkg/flt: |
| 85 |
|
- add CPP options to prevent floats to re-enter the opposite side of |
| 86 |
|
a periodic domain (default: keep periodicity) |
| 87 |
|
o pkg/gmredi: |
| 88 |
|
- apply exchange to GM_PsiX,GM_PsiY to get valid values near CS-corners |
| 89 |
|
(needed for multi-dim advection). |
| 90 |
|
- switch on GM Advective form in global_ocean.cs32x15 and update output.txt |
| 91 |
|
o model/src: |
| 92 |
|
- fix wrong re-setting of addMass array (ALLOW_ADDFLUID) for atmospheric run. |
| 93 |
|
o tutorial_tracer_adjsens |
| 94 |
|
- extend to NLFS and replace maskC by hFacC |
| 95 |
|
(change reference output accordingly) |
| 96 |
|
|
| 97 |
|
checkpoint62r (2011/01/24) |
| 98 |
|
o testreport: |
| 99 |
|
- add option -deloutp (-do) to remove output file after successful run |
| 100 |
|
- add option -mfile (-mf) for MPI machile-file and for each test-exp, |
| 101 |
|
extract exact list of procs from this file ; adjust COMMAND by |
| 102 |
|
replacing TR_MFILE & TR_NPROC with local machile-file & Nb of MPI procs. |
| 103 |
|
o pkg/ecco: replace old ineffective way of penalizing deviations from |
| 104 |
|
first guess (of obcs) with code by Matt Mazloff |
| 105 |
|
o pkg/ctrl: add code that allows the use of ALLOW_OBCS?_CONTROL without |
| 106 |
|
exf/cal |
| 107 |
|
- ctrl_init: fix initialisation (computation of diffrec) for obcs-ctrl |
| 108 |
|
and handle case of obcs?period == 0 |
| 109 |
|
Not affected by these changes: There is still an inconsistency in |
| 110 |
|
the computation of start/endrec for the obcs variables for ALLOW_CAL |
| 111 |
|
defined (see comment by Menemenlis, 7-Mar-2003) |
| 112 |
|
- make S/R ctrl_get_gen_rec return something meaningful even if |
| 113 |
|
ALLOW_CAL is not defined |
| 114 |
|
- ctrl_getobcs?: simplify code so that it does not need to use |
| 115 |
|
S/R exf_swapfields |
| 116 |
|
o testreport: |
| 117 |
|
allow to test MPI with different number of processors for each experiment: |
| 118 |
|
create file "SIZE.h.mpi" in each build dir. |
| 119 |
|
Note: update of "-command" argument not yet implemented. |
| 120 |
|
o model/src, pkg/autodiff, pkg/ecco, pkg/seaice: |
| 121 |
|
alleviate the need for additional 3D tapes when using |
| 122 |
|
the non-inear free surface in the adjoint. This is done |
| 123 |
|
by adding an 'update' of hfacc etc. to their current |
| 124 |
|
value at the beginning of forward_step.F. |
| 125 |
|
o pkg/seaice |
| 126 |
|
seaice_lsr: move stress-divergence computations out of outer pseudo- |
| 127 |
|
timestepping loop |
| 128 |
|
o pkg/obcs/obcs_prescribe_read.F: move 'CALL OBCS_EXTERNAL_FIELDS_LOAD' |
| 129 |
|
up, so that the calls of the ctrl_* subroutines are really at the end |
| 130 |
|
of the routine. |
| 131 |
|
o pkg/gmredi: |
| 132 |
|
allow to specify a 2-D horizontal map and a 1-D vertical profil |
| 133 |
|
as scaling factor for Isopycnal diffusivity (Redi) and GM diffusivity. |
| 134 |
|
o pkg/seaice: |
| 135 |
|
move lsr iterations from seaice_dynsolver to seaice_lsr. |
| 136 |
|
o pkg/seaice: |
| 137 |
|
reduce memory footprint of seaice mutldim advection tapes, |
| 138 |
|
by using a 2D tape independent of the generic_advdiff ones, |
| 139 |
|
and setting GAD_AREA=1 etc. rather than GAD_AREA=101 etc. |
| 140 |
|
|
| 141 |
|
checkpoint62q (2010/12/29) |
| 142 |
|
o model/src: |
| 143 |
|
- check horizontal grid-spacing for unset or zero/negative values. |
| 144 |
|
- change AD test hs94.1x64x5 to be truly a 2-D set-up. |
| 145 |
|
o pkg/flt: |
| 146 |
|
- use (recent) mdsio S/R MDS_READVEC_LOC instead of local FLT_MDSREADVECTOR |
| 147 |
|
- precision of input and output files set according to readBinaryPrec |
| 148 |
|
and writeBinaryPrec (previously always 64.b), pickup remains at 64 bits. |
| 149 |
|
- initial float position from tiled file was not converted to index space |
| 150 |
|
(looks like a bug): fixed to make it similar to initial pos. global file. |
| 151 |
|
o global_ocean.cs32x15: |
| 152 |
|
change SIZE.h of AD set-up to get same tiling as with MPI. |
| 153 |
|
Update output_adm, output_adm.thsice (pass@14) & output_adm.seaice (fail@11) |
| 154 |
|
o model/src: |
| 155 |
|
add flag to control printing of domain fields (bathymetry, hFac ...) in |
| 156 |
|
ini_mask_etc (call to plot_fieds_xy*). |
| 157 |
|
o pkg/thsice: |
| 158 |
|
if useEXF, some forcing fields are not valid in overlap: fixed by computing |
| 159 |
|
over the interior only + apply EXCH at the end (like with SEAICE advection). |
| 160 |
|
o pkg/mdsio: |
| 161 |
|
- add 2 arguments to S/R MDS_PASS_R4/8toRL/S for overlap size of buffer |
| 162 |
|
to copy to/from, so that it can be used to replace MDS_PASS_WH_R8toRL/RS. |
| 163 |
|
- change barrier call for safe multi-threads access to 3-D shared buffer. |
| 164 |
|
- MDS_READ/WRITEVEC_LOC use buffers from MDSIO_BUFF_3D.h (instead of local ones) |
| 165 |
|
- replace call to MDS_PASS_WH_R4/8toRL with call to MDS_PASS_R4/8toRL and |
| 166 |
|
remove the 4 S/R MDS_PASS_WH_R4/8toRL/S. |
| 167 |
|
o pkg/flt: |
| 168 |
|
- add fourth-order Runge-Kutta integration |
| 169 |
|
- add basic exch2 support (only 1 facet supported so far!) |
| 170 |
|
o pkg/mdsio+rw: |
| 171 |
|
new S/R to read local (or tile independent) vector (read counterpart of |
| 172 |
|
mds_writevec_loc, and front-end write_glvec_rl/rs). |
| 173 |
|
o global_ocean.cs32x15: additional input_ad.seaice experiment |
| 174 |
|
that uses pkg/seaice rather than pkg/thsice. |
| 175 |
|
(using multidim advection for seaice) |
| 176 |
|
o towards more F95 syntax for array specification in namelist: |
| 177 |
|
- change "data.diagnostics" (data.ptracers was changed before checkpoint62p) |
| 178 |
|
of all verification experiments. |
| 179 |
|
e.g.: line: fields(1,3)='ETAN ','ETANSQ ', |
| 180 |
|
was changed to: fields(1:2,3)='ETAN ','ETANSQ ', |
| 181 |
|
- finish adding "-DNML_EXTENDED_F77" to DEFINES list in pgf77 and g77 optfiles |
| 182 |
|
o global_ocean.cs32x15: additional input.seaice experiment |
| 183 |
|
that uses pkg/seaice rather than pkg/thsice. |
| 184 |
|
|
| 185 |
|
checkpoint62p (2010/12/17) |
| 186 |
|
o pkg/thsice: rename (ad tape index) iicekey as ticekey, and move it |
| 187 |
|
to THSICE_SIZE.h to avoid conflict with pkg/seaice. |
| 188 |
|
o pkg/seaice: add seaice_diffusion.f seaice_map_thsice.f to adjoint. |
| 189 |
|
allow for use of multim. advection in seaice adjoint. |
| 190 |
|
o pkg/seaice: add diagnostics for snow precipitation and sublimation flux |
| 191 |
|
- first attempt to close budgets by taking into account the freshwater |
| 192 |
|
flux implied by the sublimation of snow and ice. So far only the |
| 193 |
|
volume budget is closed, turn on with soon-to-be-retired-again CPP-Flag |
| 194 |
|
SEAICE_ADD_SUBLIMATION_TO_FWBUDGET |
| 195 |
|
o eesupp/src: |
| 196 |
|
new S/R with new option (NML_EXTENDED_F77) to change array specification |
| 197 |
|
from F95 standard to commonly accepted extended F77 form. |
| 198 |
|
This would allow to use F95 syntax in namelist (needed with gfortran 4.4.5) |
| 199 |
|
and to revert back to the original syntax (with -DNML_EXTENDED_F77) |
| 200 |
|
for less tolerant f77 compiler (e.g., pgf77, g77). |
| 201 |
|
o globalFiles: |
| 202 |
|
commented out in parameter file "data" since it's not safe with MPI |
| 203 |
|
(e.g., wrong on aces); keep it if useSingleCpuIO=T (but not doing much |
| 204 |
|
in this case); no longer commented out in restart test (tst_2+2). |
| 205 |
|
o add adTapeDir to specify a special directory where adjoint tapes are |
| 206 |
|
written/read (adread_adwrite.F). adTapeDir is specified in data/PARM05 |
| 207 |
|
and cannot be used together with mdsioLocalDir. |
| 208 |
|
o nesting pkgs: |
| 209 |
|
- packages_init_variables : put call to OBCS_INIT_VARIABLES in last position |
| 210 |
|
- update Gianmaria's Nesting pkgs (from Contrib/nesting_sannino) |
| 211 |
|
- add parameter files + utils dir + output from short run on beagle. |
| 212 |
|
o pkg/seaice: |
| 213 |
|
fix seaice_growth.F store directives; update results of exp. lab_sea.ad |
| 214 |
|
o pkg/seaice: explicitly store tsurfloc in seaice_solve4temp.F to avoid |
| 215 |
|
unnecessary copying in adjoint code, requires new parameter NMAX_ITER |
| 216 |
|
|
| 217 |
|
checkpoint62o (2010/11/25) |
| 218 |
|
o pkg/exf: |
| 219 |
|
- change CPP option EXF_VERBOSE to run-time logical param exf_verbose. |
| 220 |
|
o pkg/seaice: |
| 221 |
|
- seaice_ocean_stress: split a loop for better vectorization of the |
| 222 |
|
adjoint code |
| 223 |
|
- seaice_growth.F: fix bug in conservation of freshwater connected to |
| 224 |
|
capping of heff, put contribution into d_HEFFbyNEG |
| 225 |
|
- replace irritating parameters SEAICE_latentWater/Ice and SEAICE_sensHeat |
| 226 |
|
by something more sensible (parameters that are what their name implies) |
| 227 |
|
- change some defaults, so that by default exf-parameters are used for |
| 228 |
|
things like rhoAir, cpAir,latent/sensible heat parameters, in order |
| 229 |
|
to reproduce the old results, many data.seaice now contain funny values |
| 230 |
|
for cpAir and latent heat of sublimation |
| 231 |
|
o packages_readparms.F: move exf_readparms up in line so that |
| 232 |
|
exf-parameters are visible for other packages (e.g. seaice) |
| 233 |
|
o debugMode & debugLevel=2: |
| 234 |
|
- differentiate between debugMode & debugLevel=2: |
| 235 |
|
debugMode: controls printing of debug messages (sequence of S/R calls). |
| 236 |
|
debugLevel: controls printing of algorithm intermediate results and stats. |
| 237 |
|
- remove "if debugLevel=2 => debugMode" but keep "debugMode => debugLevel=2"; |
| 238 |
|
=> allows to print debug-stats without all sequence of call messages. |
| 239 |
|
- switch control of debug print from debugLevel to debugMode in many S/R. |
| 240 |
|
o pkg/obcs (obcs-sponge): |
| 241 |
|
fix bug that was introduced on Oct 25 (in dynamics.F v1.151): put back the |
| 242 |
|
call to OBCS_PRESCRIBE_EXCHANGES (need to exch gU,gV when using obcs-sponge) |
| 243 |
|
o pkg/generic_advdiff: |
| 244 |
|
- record overlap minimum size for each advection scheme; |
| 245 |
|
- refine criteria on overlap minimum size; |
| 246 |
|
- check and stop (in gad_check) if overlap size is too small. |
| 247 |
|
o pkg/obcs: |
| 248 |
|
- with ptracers: remove multiple calls to obcs_apply_ptracer (only done |
| 249 |
|
once, in ptracers_implicit.F) |
| 250 |
|
- with seaice: remove test on empty file name in obcs_apply_seaice (always |
| 251 |
|
apply OB values when tile has OB) |
| 252 |
|
o model/src, coriolis setting: |
| 253 |
|
- integer parameter "selectCoriMap" (=0,1,2) replaces 3 logical |
| 254 |
|
useConstantF, useBetaPlaneF, useSphereF ; can now read coriolis from |
| 255 |
|
files (selectCoriMap=3) with any grid. |
| 256 |
|
- add parameter "fPrime" for constant second Coriolis coeff (default=0). |
| 257 |
|
o pkg/seaice/seaice_growth.F: |
| 258 |
|
a few cosmetic changes that reduce the number of lines of code |
| 259 |
|
o pkg/obcs: add boundary conditions following Stevens (1990) |
| 260 |
|
- the code follows old MOM/NAOSIM code |
| 261 |
|
- so far it is only implemented for the eastern and western boundaries |
| 262 |
|
and NOT for pTracers |
| 263 |
|
o pkg/rbcs: |
| 264 |
|
- replace rbcsInIter by rbcsForcingOffset, the time in seconds at the |
| 265 |
|
beginning of the first rbcs forcing period. |
| 266 |
|
- add option rbcsSingleTimeFiles that makes rbcs read forcing data like the |
| 267 |
|
offline package, i.e., one file per time, labeled by iteration number; |
| 268 |
|
deltaTrbcs sets the timestep with which the data was generated; |
| 269 |
|
rbcsIter0 is the forcing iteration corresponding to rbcsForcingOffset. |
| 270 |
|
- rbcsForcingCycle=0 makes forcing non-cyclic. |
| 271 |
|
- make arithmetic safe for negative times and odd forcing periods. |
| 272 |
|
|
| 273 |
|
checkpoint62n (2010/11/09) |
| 274 |
|
o pkg/seaice: |
| 275 |
|
fix masking in free-drift S/R ; update results of exp. lab_sea.fd |
| 276 |
|
o pkg/obcs: |
| 277 |
|
New S/R to copy tracer value from the OB location to narrow band region |
| 278 |
|
(width = same as overlap) beyond the OB: preserve OB value (needed for |
| 279 |
|
high-order scheme) which were overwritten by EXCH call if in overlap |
| 280 |
|
region. Applied to theta, salt, ptracer and advected seaice variables. |
| 281 |
|
With seaice: use maskInC in similar way as in gad_advection.F, |
| 282 |
|
in seaice_advection.F and advect.F. |
| 283 |
|
Update output of experiment seaice_obcs and change exp4 to test OBCS with |
| 284 |
|
4th order Adv.Scheme. for pasive salt and ptracer. |
| 285 |
|
o tools/genmake2: |
| 286 |
|
- when looking for an optfile (not specified), only search for Fortran compiler |
| 287 |
|
(remove sym-link test + setting of CPP + makedepend check + C-compiler check) |
| 288 |
|
- in get_fortran_c_namemangling: use "$CC $CFLAG" to compile C test programm |
| 289 |
|
(instead of "$MAKE genmake_test.o" since an existing Makefile can be wrong) |
| 290 |
|
- set default for CPP to "cpp -traditional -P" (was the case without specifying |
| 291 |
|
an optfile, but otherwise default was just "cpp". |
| 292 |
|
- comment out PLATFORM=$p_PLATFORM (no idea where p_PLATFORM comes from) |
| 293 |
|
- when "CC" is not set in the optfile, look for possible C compiler |
| 294 |
|
(instead of just setting CC=cc) |
| 295 |
|
- put back version v.169 modif (from 2008/02/15), was dropped in v.173 changes: |
| 296 |
|
=> add a ".F" to ".o" rule for source files with no "#include" (such as |
| 297 |
|
port_rand.F); otherwise, default ignores FOPTIM and just uses FFLAGS. |
| 298 |
|
- move gfortran before g77 in list of possible Fortran compiler (consistent |
| 299 |
|
with reference compiler switch from g77 to gfortran) |
| 300 |
|
o pkg/seaice: |
| 301 |
|
o remove SEAICE_OLD_AND_BAD_DISCRETIZATION-code, add a test to |
| 302 |
|
seaice_check that stops the model, when the CPP flag is used. |
| 303 |
|
o seaice_growth.F: make capping of ice volume conserving by returning the |
| 304 |
|
capped ice as freshwater to EmPmR. This does not change the verification |
| 305 |
|
experiments but will change runs where the capping is acutally active |
| 306 |
|
o change reference platform+compiler from faulks+g77 to baudelaire+gfortran |
| 307 |
|
(except the 2 fizhi test exp.); update output of exp.: bottom_ctrl_5x5, |
| 308 |
|
global_ocean.cs32x15 (icedyn, thsice, AD thsice), global_ocean_ebm (AD), |
| 309 |
|
lab_sea (fd, hb87, salt_plume), MLAdjust (3/5) and vermix (dd) |
| 310 |
|
o tools/genmake2: |
| 311 |
|
- merge genmake_warnings & genmake_errors into 1 log file: "genmake.log") |
| 312 |
|
- change DUMPSTATE default to false (can use "-ds" option to switch it to T) |
| 313 |
|
- clean previous genmake log files before anything else. |
| 314 |
|
- change Makefile to remove testreport log files (*.tr_log) and genmake log |
| 315 |
|
files only with "CLEAN" target (instead of "Clean"); |
| 316 |
|
o pkg/generic_advdiff: |
| 317 |
|
Change OBCS in multiDim advection (gad_advection.F): |
| 318 |
|
Use maskInC to prevent intermediate update of tracer-field in OB regions, |
| 319 |
|
thus keeping current-time-step OB values; and remove the call to OBCS_APPLY |
| 320 |
|
which were applying future time-step OB values. |
| 321 |
|
This way, all explicit calculations use current-time-step OB values |
| 322 |
|
(simpler and more consistent). |
| 323 |
|
Affects results when set-up uses time-varying OBC and multi-dim advection; |
| 324 |
|
update output of experiment seaice_obcs. |
| 325 |
|
o pkg/seaice: |
| 326 |
|
move OPTIONS (formerly locally defined) to SEAICE_OPTIONS.h: |
| 327 |
|
this includes SEAICE_GROWTH_LEGACY (+ SEAICE_DO_OPEN_WATER_GROWTH & |
| 328 |
|
SEAICE_OCN_MELT_ACT_ON_AREA ) and SEAICE_SOLVE4TEMP_LEGACY. |
| 329 |
|
Default (pkg/seaice/SEAICE_OPTIONS.h) is to define SEAICE_SOLVE4TEMP_LEGACY |
| 330 |
|
but leave SEAICE_GROWTH_LEGACY undef ; For now, keep verification exp |
| 331 |
|
unchanged (SEAICE_GROWTH_LEGACY defined in */code*/SEAICE_OPTIONS.h) |
| 332 |
|
o pkg aim (2010/10/26): |
| 333 |
|
- Introduce ALLOW_INSOLATION, moving toward a more general incoming solar radiation |
| 334 |
|
formulation with obliquity, eccentricity, ... as run-time parameters, |
| 335 |
|
- Introduce diagnostics for top-of-the-atmaophere incoming solar radiation and |
| 336 |
|
upward shortwave radition at the surface (all-sky and clear-sky). |
| 337 |
|
o pkg/obcs: |
| 338 |
|
- writing sections with globalFiles=T does not work: switch to tiled-files |
| 339 |
|
when writing sections for OBCS (change pkg/rw/write_rec.F accordingly). |
| 340 |
|
- fix index bug (OB_Ie -> OB_Iw_) in obcs_apply_uvice.F |
| 341 |
|
- apply OBCS_SEAICE_COMPUTE_UVICE to both uice and vice at all obcs |
| 342 |
|
|
| 343 |
|
checkpoint62m (2010/10/21) |
| 344 |
|
o pkg/seaice: vectorize seaice_solve4temp.F |
| 345 |
|
- move iterations loops outside of i/j-loops |
| 346 |
|
- requires 2D arrays for TB and effConduct |
| 347 |
|
- add 2D iceOrNot to replace HICE_ACTUAL.GT.0, as HICE_ACTUAL is |
| 348 |
|
modified in this routine |
| 349 |
|
- a little bit of clean up, adjusting indentation, etc. |
| 350 |
|
- replaced a few TEN**a by exp(a*log(10.)) in new code for better |
| 351 |
|
performance and '**TWO' by '**2' (because TWO is a float), |
| 352 |
|
replaced ONE, ZERO |
| 353 |
|
o eesupp/src: |
| 354 |
|
fix flow dir for exch1_z_RX_cube, exch1_uv_RX_cube & exch1_bg_RX_cube |
| 355 |
|
o pkg/seaice/seaice_growth.F |
| 356 |
|
Revision 1.89 of seaice_growth.F completes the overall october 2010 revision and |
| 357 |
|
merging process. The merged routine has two branches, and one activates the 'evolution' |
| 358 |
|
branch by setting SEAICE_GROWTH_LEGACY to undef (at the top of the routine for now). |
| 359 |
|
In summary, here is what the '10/10 revision' did consist of |
| 360 |
|
part 1: seaice_growth.F revisions 1.71 to 1.75 |
| 361 |
|
- Preliminary seaice_growth.F clean-up, keeping test results strictly unchanged. |
| 362 |
|
The emphasis here was on improving readability (e.g. by trying to use meaningful |
| 363 |
|
variable names) and modularity (e.g. by adding arrays to store heff etc. updates). |
| 364 |
|
part 1.5: |
| 365 |
|
- revising the lab_sea experiment to make it a more useful (i.e. stable |
| 366 |
|
and representative of an actual model run). This was done by generating a |
| 367 |
|
(winter) pickup to start from, switching back the main experiment dynamic |
| 368 |
|
solver from evp to lsr, having one of the experiments use the free drift solution. |
| 369 |
|
part 2: seaice_growth.F revisions 1.76 to 1.85 |
| 370 |
|
- Sorting out hard-coded constants that prevented heat and fresh water conservation. |
| 371 |
|
- Sorting out the various fields units and sign (set early on and reset at the end) |
| 372 |
|
to avoid numerous scalings/re-scalings. By v1.85 the basic unit for ice, heat and |
| 373 |
|
fresh water stock is 'effective ice meters', and >0 implies more ice/snow. |
| 374 |
|
- Using tendency terms arrays (d_HEFF* etc.) to alleviate remaining code locks, |
| 375 |
|
and simplifying logical expressions to further improve readibility. |
| 376 |
|
part 3: seaice_growth.F revisions 1.86 to 1.89 |
| 377 |
|
- Re-organize the blocks of code more logically. |
| 378 |
|
- Merging the '_if.F' pieces that add something new. |
| 379 |
|
o pkg/thsice: |
| 380 |
|
- vectorize thsice_solve4temp and thsice_get_exf. |
| 381 |
|
Problem: thsice_solve4temp has an unvectorizable statement that |
| 382 |
|
might also be problematic for TAF |
| 383 |
|
- thsice_get_exf: add the ALLOW_BULK_LARGEYEAGER04 CPP-flag. This |
| 384 |
|
does not change the verifcation experiment, but will change all |
| 385 |
|
experiments that have this flag undefined (not tested in verification); |
| 386 |
|
a little reorganization to save a few 2D fields and recomputations |
| 387 |
|
- 1st step of revisiting thsice adjoint (major recomp. sorted out) |
| 388 |
|
- 2nd step: fix some store dirs |
| 389 |
|
o seems to work somewhat, i.e. using ifort but not g77 |
| 390 |
|
o likely cause are incomplete init. or recomputation |
| 391 |
|
o start verification/global_ocean.cs32x15/input_ad.thsice |
| 392 |
|
o pkg/obcs: add support for exch2 |
| 393 |
|
- the position of the boundary and prescribed values are specified using |
| 394 |
|
a global domain with exch2 facets stacked |
| 395 |
|
- in x for N,S boundaries (like W2_mapIO=-1) |
| 396 |
|
- in y for E,W boundaries (so E,W boundaries do not overlap) |
| 397 |
|
o pkg/mdsio: add support for writing global sections with exch2 |
| 398 |
|
file layout is as for obcs described above: |
| 399 |
|
- for x-z sections, facets are stacked in x (like W2_mapIO=-1) |
| 400 |
|
- for y-z sections, facets are stacked in y |
| 401 |
|
o pkg/thsice: |
| 402 |
|
- change a few routines for better vectorization: thsice_albedo.F, |
| 403 |
|
thsice_step_temp.F, and in thsice_calc_thickn.F. Results are |
| 404 |
|
not affected, but in particular the latter routine has changed |
| 405 |
|
dramatically (not necessarily easier to read now). |
| 406 |
|
- re-arrange calculation in THSICE_SOLVE4TEMP: move outside of i,j loops |
| 407 |
|
iteration loop and call to S/R THSICE_GET_BULKF & THSICE_GET_EXF. |
| 408 |
|
o verification/lab_sea: |
| 409 |
|
- overhaul of the forward model experiments. The main motivation |
| 410 |
|
for this revision is that lab_sea has proven quite fragile (when |
| 411 |
|
changing the sea-ice code, and switching platforms) to a point |
| 412 |
|
when failures were very hard to attribute to specific code problems. |
| 413 |
|
- the suspected reason for this fragility is noise amplification by evp so, |
| 414 |
|
to start with, we decided to replace evp with lsr in the main experiment. |
| 415 |
|
- the input.lsr experiment thus became obsolete and was removed. |
| 416 |
|
- it was replaced with the input.fd experiment that uses an |
| 417 |
|
analytical "free drift" solution as the ice velocity field. |
| 418 |
|
- Taken together, input (lsr), input.fd (free drift) and input.hb87 (evp) |
| 419 |
|
will hopefully allow us to discriminate problems due to dynamic solvers |
| 420 |
|
from problems due to the thermodynamic code. input.salt_plume (lsr) allows |
| 421 |
|
us to test additional options, along with the salt plume package. |
| 422 |
|
- Finally a pickup was generated (at the end of a one year run) to |
| 423 |
|
make the tests more representative of an actual model run. Previously |
| 424 |
|
the tests starting point was a 1m thickness of ice everywhere. |
| 425 |
|
o pkg/seaice: |
| 426 |
|
- introducing S\R seaice_freedrift.F that compute the |
| 427 |
|
analytical solution to an approximate ice momentum |
| 428 |
|
equation (i.e. the simple stress/coriolis balance). |
| 429 |
|
- define SEAICE_ALLOW_FREEDRIFT to trigger the computation |
| 430 |
|
of the free drift velocity field, and enable its use. |
| 431 |
|
To actually use the free drift solution as the model ice |
| 432 |
|
velocity field set SEAICEuseFREEDRIFT to .TRUE. in data.seaice. |
| 433 |
|
o move "useOASIS" from PARAMS.h to EEPARAMS.h and read it from "eedata" |
| 434 |
|
|
| 435 |
|
checkpoint62l (2010/10/04) |
| 436 |
|
o pkg/seaice: |
| 437 |
|
-fix the USE_ORIGINAL_SBI version (for variable freezing point); |
| 438 |
|
-use "celsius2K" in SEAICE_VARIABLE_FREEZING_POINT and in new |
| 439 |
|
version (with USE_ORIGINAL_SBI undef); |
| 440 |
|
o add hooks for new packages OASIS, the package will follow |
| 441 |
|
(hooks exclude the seaice pkg for now) |
| 442 |
|
o pkg/autodiff: ALLOW_AUTODIFF_WHTAPEIO |
| 443 |
|
Option to use mdsio_read_whalos.F/mdsio_write_whalos.F, |
| 444 |
|
rather than mdsio_readvector.F/mdsio_writevector.F, within |
| 445 |
|
adread_adwrite.F to read/write adjoint-related tapes to disk. |
| 446 |
|
o eesupp and pkg/mdsio: ALLOW_WHIO |
| 447 |
|
Option to write/read 2D fields to files including tiles halos. |
| 448 |
|
The main purpose is for adjoint related "tape I/O". |
| 449 |
|
The secondary purpose is for debugging phases. |
| 450 |
|
A word on those rather specifically intended purposes. |
| 451 |
|
The code was meant to be stripped down to the bare minimum. |
| 452 |
|
Typically, there is a unique and basic mapping to the global |
| 453 |
|
buffer (tile by tile, then proc by proc, then level by level), and |
| 454 |
|
byteswaps and meta-files are omitted. Indeed the (*_WH*) code is not |
| 455 |
|
meant to generate user friendly outputs of various shapes and forms. |
| 456 |
|
Rather it is intended, for the knowledgeabale developers, |
| 457 |
|
to be largely independent of the many mdsio options, to allow |
| 458 |
|
outputs of tiles+halos in debugging phases and in adjoint runs, |
| 459 |
|
and to allow some flexibility in tuning (adoint) I/O depending on each |
| 460 |
|
file system behavior. With regard to tuning, most of the potential knobs |
| 461 |
|
(to leave files open, to allocate the I/O indep. of usesinglecpuio, |
| 462 |
|
to specify directories, to specify the typical size of I/O calls, etc.) |
| 463 |
|
are not yet included, and will be added if they prove useful. |
| 464 |
|
o pkg/seaice: |
| 465 |
|
fix uninitialised arrays in USE_ORIGINAL_SBI version of seaice_solve4temp.F |
| 466 |
|
o pkg/seaice: |
| 467 |
|
new S/R seaice_solve4temp.F: this is the merged version of Ian Fenty |
| 468 |
|
code (seaice_budget_ice_if.F) which enable to recover old standard |
| 469 |
|
code (seaice_budget_ice.F) when option USE_ORIGINAL_SBI is defined. |
| 470 |
|
temporary: add "#define USE_ORIGINAL_SBI" at the top of seaice_solve4temp.F. |
| 471 |
|
o model/src (rStar): |
| 472 |
|
use simple average (not area weighted) of rStarFac at U,V point when using |
| 473 |
|
vectorInvariant and selectKEscheme=1,3 (consistent with mom. vertical advection) |
| 474 |
|
update output of exp. fizhi-cs-32x32x40 and fizhi-cs-aqualev20. |
| 475 |
|
|
| 476 |
|
checkpoint62k (2010/09/17) |
| 477 |
|
o adjoint |
| 478 |
|
* changed verification/global_ocean.cs32x15 ; adjoint now tests |
| 479 |
|
Adams-Bashforth 3 + NLFS + RSTAR |
| 480 |
|
* some more modifs required for ECCO-GODAE v4 |
| 481 |
|
o pkg/seaice |
| 482 |
|
add SEAICE_AGE_VOL option to use volume instead of area for |
| 483 |
|
integrating SEAICE_AGE |
| 484 |
|
o model/src: |
| 485 |
|
first check-in of sigma & hybrid-sigma coordinate code ; also includes |
| 486 |
|
non-linear free-surface parts ; but pressure-gradient not yet coded. |
| 487 |
|
o pkg/ptracers: |
| 488 |
|
option for ptracer to convert Salt Relax into additional EmP. |
| 489 |
|
tested in exp. global_ocean.90x40x15.dwnslp (update output) |
| 490 |
|
|
| 491 |
|
checkpoint62j (2010/08/25) |
| 492 |
|
o pkg/ggl90: |
| 493 |
|
other minor changes: remove 3-D temp array gTKE (+ add argument myIter) |
| 494 |
|
o pkg/dic: car_flux.F, phos_flux.F |
| 495 |
|
- reduce number of exp-calls in order to improve performance |
| 496 |
|
- clean up a little (replace fields by their macros, etc., fix |
| 497 |
|
typo in CPP-flag) |
| 498 |
|
o pkg/ggl90: minor changes |
| 499 |
|
- GGL90diffKrS was removed --> always use GGL90diffKr |
| 500 |
|
- GGL90viscAr was removed --> replaced with GGL90viscArU, GGL90viscArV |
| 501 |
|
- hack of mxlMaxFlag=2 --> ensure mixing between first and second level |
| 502 |
|
(commented out for now) |
| 503 |
|
- change in max/min operations to ensure that smoothing is ok |
| 504 |
|
- smoothing of GGL90viscAr was moved to ggl90_calc.F (as done for GGL90diffKr) |
| 505 |
|
- always use diffKrNrT as background profile (i.e. never use diffKr field) |
| 506 |
|
o pkg/cfc: bug fix, AREA(i,j,1,bi,bj) -> AREA(i,j,bi,bj) |
| 507 |
|
o pkg/dic: |
| 508 |
|
- model/src/impldiff.F: if TARGET_NEC_SX, extend loop ranges for better |
| 509 |
|
vectorization |
| 510 |
|
- small bug fix in insol for the unlikely case of a cartesian grid |
| 511 |
|
- towards vectorizing the dic package: bio_export.F, car_flux.F, phos_flux.F |
| 512 |
|
+ now vectorize, which also removes adjoint recomputations; |
| 513 |
|
+ replace integer maksp1 by _RL maskp1 |
| 514 |
|
+ replace a**x by exp(x*log(a)) in phos_flux.F for speed |
| 515 |
|
|
| 516 |
|
checkpoint62i (2010/07/21) |
| 517 |
|
o pkg/flt: |
| 518 |
|
- with OBCS: stop floats which enter the OB region |
| 519 |
|
o model/src: |
| 520 |
|
- use tempVertAdvScheme & saltVertAdvScheme (instead of horizontal ones) |
| 521 |
|
for implicit vertical advection. |
| 522 |
|
o pkg/seaice: |
| 523 |
|
- account for melting in seaice age evolution. |
| 524 |
|
- remove double counting of seaice ridging effect on seaice age. |
| 525 |
|
|
| 526 |
|
checkpoint62h (2010/06/18) |
| 527 |
|
o pkg/exch2: |
| 528 |
|
- remove exch2_*xy_rx files: call was replaced with exch2_*3d_rx(myNz=1). |
| 529 |
|
o tools/genmake2 |
| 530 |
|
- remove "-pdefault" option: redundant with the use of "packages.conf"; |
| 531 |
|
- remove file pkg/pkg_default : set the default as a pkg group |
| 532 |
|
("default_pkg_list") defined in pkg/pkg_groups; |
| 533 |
|
- fix for case pkg/pkg_depend is missing (was hanging before); |
| 534 |
|
- add option "-pgroups" to specify a different pkg_groups than the default. |
| 535 |
|
o verification/adjustment.cs-32x32x1: |
| 536 |
|
add a "minimal" set-up test (code_min & input_min) which only compiles |
| 537 |
|
eesupp/src and the 2 pkgs (exch2 & debug) which don't need model/src & inc. |
| 538 |
|
o tools/genmake2 |
| 539 |
|
- extend pkg_depend rules to model & eesupp (e.g., if model depend on [pkg])) |
| 540 |
|
- fix details of checking for mnc & profile pkg when NetCDF is unavailable. |
| 541 |
|
o pkg/exf: merge exf_bulk_largeyeager04 into exf_bulkformulae because these |
| 542 |
|
files are nearly the same anyway. |
| 543 |
|
Step 1: make exf_bulkformulae look like exf_bulk_largeyeager04: |
| 544 |
|
- redefinition of tau(i,i) (by a factor of ustar) |
| 545 |
|
- 1./exf(Const/T) -> exf(-Const/T) (big change in the results) |
| 546 |
|
- replace rhonil by rhoConstFresh in computation of evap (bug fix) |
| 547 |
|
- replace u/vstress=tau*rd*ws*cw by tau*rd*u/vwind (changes results) |
| 548 |
|
- replace |
| 549 |
|
rd(i,j)= rdn(i,j)/(exf_one - rdn(i,j)/karman*psimh ) |
| 550 |
|
shn = sh(i,j,bi,bj)*rd(i,j)/rdn(i,j) |
| 551 |
|
by |
| 552 |
|
shn = sh(i,j,bi,bj)/(exf_one - rdn(i,j)/karman*psimh ) |
| 553 |
|
(changes results) |
| 554 |
|
-> exf_bulkformulae v1.22, |
| 555 |
|
update verification experiments lab_sea, offline_exf_seaice, seaice_obcs |
| 556 |
|
(lab_sea: output.hb87.txt 1.11, output.lsr.txt 1.25, output.salt_plume.txt 1.13, |
| 557 |
|
output.txt 1.63 |
| 558 |
|
offline_exf_seaice: output.seaicetd.txt 1.7 |
| 559 |
|
seaice_obcs: output.txt 1.10) |
| 560 |
|
Step 2: do the actual merge and remove exf_bulk_largeyeager04.F |
| 561 |
|
-> exf_bulkformulae v1.23, |
| 562 |
|
unfortunately requires updating lab_sea.salt_plume (6 matching digits), |
| 563 |
|
and global_ocean.cs32x15.icedyn (only 11 matching digits) |
| 564 |
|
Also update output_adm of exp. lab_sea (all 4) and offline_exf_seaice. |
| 565 |
|
o adjoint of exch-1: |
| 566 |
|
- pass high-level EXCH S/R through TAF (eesupp/src/exch_ad_diff.list) |
| 567 |
|
(note: was already like this when using EXCH-2) and remove flow dir |
| 568 |
|
and hand-written AD version of high-level EXCH S/R (pkg/autodiff); |
| 569 |
|
- provide flow-dir for low-level exch1 S/R (eesupp/src/exch1_ad.flow); |
| 570 |
|
- update pkg/exch2 diff list. |
| 571 |
|
|
| 572 |
|
checkpoint62g (2010/05/19) |
| 573 |
|
o eesupp/src (exch-1): |
| 574 |
|
- split exch_rx.F into Forward S/R (exch1_rx.F) and Adjoint S/R (exch1_rx_ad.F) |
| 575 |
|
and remove argument "simulationMode"; same thing for all exch_*rx_cube.F |
| 576 |
|
(renamed to exch1_*rx_cube.F). |
| 577 |
|
- change accordingly exch_*_rx.template and AD version in pkg/autodiff |
| 578 |
|
to call new exch1 S/R ; update Makefile and .cvsignore. |
| 579 |
|
- replace call to exch2_xy_rx S/R with exch2_*3d_rx(myNz=1) (will remove |
| 580 |
|
exch2_xy_rx later) |
| 581 |
|
Note: after updating your code, need to run "make clean" in eesupp/src |
| 582 |
|
o test experiment bottom_ctrl_5x5: |
| 583 |
|
new output_adm.txt with fewer grdchk & less monitor output. |
| 584 |
|
o model/src: |
| 585 |
|
remove the call to ini_cori in initialise_varia.F (just keep the usual one |
| 586 |
|
in initialise_fixed.F). More consistent with coriolis & horizontal grid |
| 587 |
|
beeing treated in the same way (ini_grid is not called a 2nd time). |
| 588 |
|
o pkg/exf: |
| 589 |
|
- rewrite exf_bulk_largeyeager04.F for better vectorization, changes |
| 590 |
|
global_ocean.cs32x15/input.icedyn (11 digits of agreement), updated |
| 591 |
|
- fix adjoint (used worked for ALLOW_ATM_WIND defined, works for this now) |
| 592 |
|
- cosmetic changes to exf_bulkformulae.F to make it look more like |
| 593 |
|
exf_bulk_largeyeager04.F |
| 594 |
|
o eesupp/src (exch-1): |
| 595 |
|
- Fix AD EXCH code for 2-D domain special cases (Nx=1 or Ny=1). |
| 596 |
|
- Separate buffer filling and MPI sending: allow EXCH-1 to work for local |
| 597 |
|
array (non-shared) when using MPI+MTH. Also reduces number of BARRIER |
| 598 |
|
(even without using MPI). |
| 599 |
|
- Message mode: move RecvAck setting (indicator of buffer being ready) |
| 600 |
|
from send_put to recv_get S/R (was useless before, but not sure if |
| 601 |
|
it's much more useful now); |
| 602 |
|
- Switch the order of sync: MPI-proc 1rst and then threads; |
| 603 |
|
- take out spin-waiting code (#undef EXCH_USE_SPINNING), use BARRIER instead. |
| 604 |
|
- fix AD code for communication mode ='get' (but not used). |
| 605 |
|
o model/src: |
| 606 |
|
initialise dPhiHydX,dPhiHydY inside calc_grad_phi_hyd.F (so that TAF does |
| 607 |
|
not drop it in S/R mddynamics, producing NaN in hs94.1x64x5 AD-test exp). |
| 608 |
|
o utils/matlab/rdmnc.m: adapt to be able to use the generic matlab netcdf API |
| 609 |
|
o pkg/ctrl: require a specific set of control variables and exf-flags for |
| 610 |
|
adjoint of seaice (stop in ctrl_check.F if conditions are not satisfied) |
| 611 |
|
o model/src: |
| 612 |
|
a) 1rst version of a new driver S/R to turn-off output freq and flags after the |
| 613 |
|
1rst forward sweep, to avoid over-writing output files during recomputations |
| 614 |
|
recomputations (& grdchk); called from cost_final.F (& ctrl_init_ctrlvar.F ?) |
| 615 |
|
b) 2nd version: |
| 616 |
|
- add specific pkg S/R for seaice & ptracers (called from S/R TURNOFF_MODEL_IO); |
| 617 |
|
- switch-off more output flags (exf_monFreq, chkPtFreq, dumpInitAndLast, |
| 618 |
|
writePickupAtEnd) + seaice & ptracers output; |
| 619 |
|
- do not switch off MNC flags (otherwise, won't get admonitor netcdf file) |
| 620 |
|
o model/src: |
| 621 |
|
- new S/R rotate_uv2en_rl (..._rs) that rotates vector fields. |
| 622 |
|
o pkg/exch2: |
| 623 |
|
- remove "simulationMode" from arg. list of S/R EXCH2_RX1,2_CUBE ; |
| 624 |
|
- add argument "signOption" to EXCH2_RX1_CUBE (will be needed for SM exch). |
| 625 |
|
o pkg/autodiff & exch2: |
| 626 |
|
process exch_uv_agrid_3d_rx.f & exch2_uv_agrid_3d_rx.f through TAF |
| 627 |
|
(for now, only if ALLOW_EXCH2 is defined). |
| 628 |
|
o pkg/autodiff: |
| 629 |
|
- include PACKAGES_CONFIG.h in exch*ad.flow & exch*ad.F |
| 630 |
|
(not supposed to be in CPP_OPTIONS.h) to get ALLOW_EXCH2 defined correctly. |
| 631 |
|
+ clean-up comments (_RS is not always real*4) |
| 632 |
|
- fix bug in ADEXCH_UV_3D_RX (without EXCH2, was calling the forward exch S/R) |
| 633 |
|
- add a stop in ADEXCH_UV_AGRID_3D_RX if trying to use it with EXCH2: |
| 634 |
|
was calling a forward exch2, which is wrong ; used in SEAICE & EXF pkgs. |
| 635 |
|
o testreport (AD-tests): |
| 636 |
|
no longer link files from dir "../input" (1rst link files from |
| 637 |
|
"../input_ad.[xx]" if secondary test "xx" and then from "../input_ad") |
| 638 |
|
|
| 639 |
|
checkpoint62f (2010/04/29) |
| 640 |
|
o model/src/the_main_loop.F: |
| 641 |
|
adjust store directives for seaice evp and multicategory according to |
| 642 |
|
pkg/ecco/the_main_loop.F |
| 643 |
|
o adjoint: |
| 644 |
|
global_sum_ad was incomplete (noted many times by many people ;) |
| 645 |
|
nice description by Jean U. as follows: |
| 646 |
|
* allreduce(var) in global_sum.F can be interpreted as |
| 647 |
|
reduce(sum) followed by broadcast of sum. |
| 648 |
|
* The adjoint of that is |
| 649 |
|
sum(adjoint) followed by broadcast(summed adjoint value) |
| 650 |
|
or in other words, allreduce(sum, adjoint vars) |
| 651 |
|
We were missing the sum(adjoint)! |
| 652 |
|
Benign, except(!) for "global" scalars appearing in tile loops (bi,bj) |
| 653 |
|
(noted by J.M. Campin, e.g., in cost_ssh_mean, variable offset). |
| 654 |
|
The fix should produce 16-digit accuracy between |
| 655 |
|
single-CPU multi-tile (e.g. nSx=2, nPx=1) and |
| 656 |
|
multi-CPU single-tile (e.g. nSx=1, nPx-2) |
| 657 |
|
We'll probably change this yet again shortly |
| 658 |
|
o eesupp/src: |
| 659 |
|
- fix global_sum_r4 (wrong from previous modif in Jun 2009; ever used ?) |
| 660 |
|
o pkg/icefront |
| 661 |
|
- ALLOW_SUBGLACIAL_RUNOFF option adds capability for time-evolving |
| 662 |
|
specification of addmass array based on a 2D field, which is added |
| 663 |
|
at bottommost wet level at each horizontal location. |
| 664 |
|
o model/src: |
| 665 |
|
horizontal grid initialisation (other than curvilinear): add code for Exch2 IO |
| 666 |
|
layout; (not always compatible with delX,delY setting; commented out for now). |
| 667 |
|
o pkg/exch2: |
| 668 |
|
change tile to processor setting (case nSy*nPy > 1): allows default model mapping |
| 669 |
|
(myX and myYGlobalLow) to be used for simple (1 facet, no blank tile) config. |
| 670 |
|
|
| 671 |
|
checkpoint62e (2010/04/15) |
| 672 |
|
o pkg/obcs: |
| 673 |
|
- fix wrong index in OBCS_SPONGE_V for Eastern-OB (reported by David Hebert). |
| 674 |
|
o pkg/dic: |
| 675 |
|
-fix bug in fe_chem.F for multi-tile (nSx*nSy > 1) set-up; |
| 676 |
|
(note: not presently tested in any forward verification exp.) |
| 677 |
|
update tutorial_global_oce_biogeo AD output file. |
| 678 |
|
o pkg/exf: |
| 679 |
|
- account for the variation of albedo as a function of zenith angle |
| 680 |
|
by defining ALLOW_ZENITHANGLE in EXF_OPTIONS.h |
| 681 |
|
o build_options: |
| 682 |
|
- rename "linux_ia64_ifort+mpi_ice_nas" to "linux_amd64_ifort+mpi_ice_nas" |
| 683 |
|
o pkg/dic: |
| 684 |
|
change DIC atmos-CO2 box (method 3, dic_int1=3) initialisation & restart: |
| 685 |
|
- use binary pickup file for restart; |
| 686 |
|
- initialise atmos-CO2 box from dic_pCO2 parameter; |
| 687 |
|
- fix multi-threaded. |
| 688 |
|
o pkg/rbcs: |
| 689 |
|
- clean-up RBCS code: add RBCS_OPTIONS.h file ; fix multi-threaded ; |
| 690 |
|
change (+ fix IO calls) type of array RBC_mask from _RL to _RS. |
| 691 |
|
- add RBCS to exp4 passive tracer 01 (rbcs was previously untested) |
| 692 |
|
and update output.txt |
| 693 |
|
o pkg/kpp: |
| 694 |
|
- add diagnostics for KPP non-local flux of Temp, Salinity and pTracers; |
| 695 |
|
- change diagnostic of KPP non-local term: output average of KPPghat*KPPdiffKzS |
| 696 |
|
(= ratio of KPP non-local (salt) flux relative to surface-flux) when |
| 697 |
|
using pkg/timeave & pkg/diagnostics (diag 'KPPghatK' replaces 'KPPghat '); |
| 698 |
|
- change pkg/offline to read-in average KPPghat*KPPdiffKzS from file |
| 699 |
|
KPP_ghatKFile (instead of the average of KPPghat from file KPP_ghatFile). |
| 700 |
|
o pkg/offline: |
| 701 |
|
fix bug in S/R OFFLINE_FIELDS_LOAD regarding surfaceForcing conversion factor |
| 702 |
|
(bug was added in version 1.10 on 2007-08-23). |
| 703 |
|
o pkg/offline: |
| 704 |
|
- no need to specify periodicExternalForcing, etc ... to use offline pack |
| 705 |
|
- offline input files no longer just single prec --> parameter offlineLoadPrec |
| 706 |
|
o verification/lab_sea: |
| 707 |
|
change AD test to multi-tile setup and update all 4 AD output. |
| 708 |
|
NOTE: while cost function values remain very similar, |
| 709 |
|
gradients are very different, although gradient checks are self-consistent |
| 710 |
|
Should try to get to the bottom of this (cost function summation order?) |
| 711 |
|
o pkg/dic: |
| 712 |
|
in carbon_chem.F, replace log10 (AUTODIFF) with explicit form to avoid |
| 713 |
|
a weakness of TAF that drops part of double precision in log10 derivative. |
| 714 |
|
<= changes removed since problem has been fixed in current TAF version 1.9.71 |
| 715 |
|
update AD output of exp. tutorial_dic_adjoffline & tutorial_global_oce_biogeo. |
| 716 |
|
o pkg/ecco: |
| 717 |
|
add missing "_d 0" in cost_readers.F, cost_readtopex.F & cost_readtopexmean.F |
| 718 |
|
update all 4 lab_sea AD output. |
| 719 |
|
|
| 720 |
|
checkpoint62d (2010/03/22) |
| 721 |
|
o build_options: |
| 722 |
|
remove '-r8' (and '-i4') from linux_*pgf and linux_*ifort optfiles. |
| 723 |
|
o remove unbalanced quote (single or double) in commented line of source code |
| 724 |
|
(may cause some warning with some preprocessor/makedepend versions). |
| 725 |
|
o pkg/seaice: |
| 726 |
|
fix seaice_model.F for adjoint (gael) and update AD results of experiments |
| 727 |
|
lab_sea (std + evp + noseaicedyn) and offline_exf_seaice. |
| 728 |
|
o testreport: |
| 729 |
|
add option '-runonly' (short: '-ro') to just do the run part (without make). |
| 730 |
|
o eesupp: remove JAM communication code (obsolete). |
| 731 |
|
o add option to genmake2 & testreport to switch "_RS" type to "real*4" |
| 732 |
|
(if allowed in CPP_EEOPTIONS.h); update default CPP_EEOPTIONS.h and return |
| 733 |
|
to default CPP_EEOPTIONS.h for exp. inverted_barometer & solid-body.cs-32x32x1 |
| 734 |
|
(update also the output). |
| 735 |
|
|
| 736 |
|
checkpoint62c (2010/03/02) |
| 737 |
|
o tools/build_options : |
| 738 |
|
move (some) old optfiles from build_options/ to build_options/unsupported/ |
| 739 |
|
o model/src and pkg/exf |
| 740 |
|
surface relaxation -- separating data and data.exf options |
| 741 |
|
The point is to ensure that we specify the surface relaxation |
| 742 |
|
either in data or in data.exf (but not half and half) and to |
| 743 |
|
avoid an accidental relaxation to 0 (which happened if a time |
| 744 |
|
scale was specified but the file name was left empty) |
| 745 |
|
o model/src model/inc and pkg/mom_common |
| 746 |
|
ALLOW_3D_VISCAH/ALLOW_3D_VISCA4 => 3D specification of Viscosity |
| 747 |
|
=> addition to the viscosity field as computed by others schemes |
| 748 |
|
o tools/build_options: |
| 749 |
|
- rename "_generic_mpi" -> "+mpi_generic" (like other "+mpi" optfile) |
| 750 |
|
but keep the former optfile for few weeks for smooth transition |
| 751 |
|
o pkg/ecco |
| 752 |
|
- allow user defined cost function contributions |
| 753 |
|
CPP: ALLOW_USERCOST_CONTRIBUTION |
| 754 |
|
USAGE: compute contributions in cost_usercost_all.F |
| 755 |
|
- note: this is identical to what ALLOW_GENCOST_CONTRIBUTION once was. |
| 756 |
|
the two options are complementary. |
| 757 |
|
o pkg/smooth |
| 758 |
|
- migration from contrib to the main repository |
| 759 |
|
|
| 760 |
|
checkpoint62b (2010/01/26) |
| 761 |
|
o pkg/monitor: |
| 762 |
|
- change back mon_printstats_rl.F & mon_stats_rl.F and add a stop |
| 763 |
|
(in case someone forgot to update customized pieces of code) |
| 764 |
|
o nonHydrostatic: |
| 765 |
|
- add Non-hydrostatic free-surface form (selectNHfreeSurf=1)(not fully tested); |
| 766 |
|
- add a simple 2-D experiment (short_surf_wave) to test it. |
| 767 |
|
o pkg/icefront: |
| 768 |
|
adding the shell of, and the hooks to, a new package that |
| 769 |
|
will model melting and freezing of vertical glacier ice fronts |
| 770 |
|
o momentum: |
| 771 |
|
add diagnostics of momentum tendency from Hydrostatic pressure gradient |
| 772 |
|
(without surf.pressure term: would need a 2nd filling since implicit term). |
| 773 |
|
o add 3 "_d 0"s so that fizhi compiles with xlf on iblade, and replace |
| 774 |
|
two " &end" by " &" (in fizhi-cs-*) because S/R NML_SET_TERMINATOR |
| 775 |
|
cannot deal with the former |
| 776 |
|
o cleaning: |
| 777 |
|
- remove all (except OpenAD) old TLM output from results_ad dirs. |
| 778 |
|
- remove unused S/R (MDSWRITEMETA, DIAGNOSTICS_ADD2LIST, old DFILE IO S/R) |
| 779 |
|
- comment out maskH definition (replaced by maskInC) |
| 780 |
|
o pkg/monitor: |
| 781 |
|
- change mon_printstats_rl.F & mon_stats_rl.F (were similar to mon_writestats_rl |
| 782 |
|
& mon_calc_stats_rl) to be the RL equivalent of mon_printstats_rs & mon_stats_rs. |
| 783 |
|
|
| 784 |
|
checkpoint62a (2010/01/15) |
| 785 |
|
o model/src: |
| 786 |
|
- bottom pressure diagnostic: correct for r* (when nonlinFreeSurf >= 4) |
| 787 |
|
o pkg/diagnostics: |
| 788 |
|
- add optional level number diagnostics (i.e., level number to be define |
| 789 |
|
explictly with S/R DIAGNOSTICS_SETKLEV) with parser-code(10)="X". |
| 790 |
|
- strictly check for valid parser-code(10) ; |
| 791 |
|
- check if adding diag to the list from the right place. |
| 792 |
o pkg/longstep: |
o pkg/longstep: |
| 793 |
- change parameters and fix tracer conservation with rstar/nonlinear free surface |
- change parameters and fix tracer conservation with rstar/nonlinear free surface |
| 794 |
o diagnostics: |
o diagnostics: |
Parent Directory
| 
Revision Log
| 
Revision Graph
| 
Patch