| 4 |
Notes on tags used in MITgcmUV |
Notes on tags used in MITgcmUV |
| 5 |
============================== |
============================== |
| 6 |
|
|
| 7 |
|
o model/src: |
| 8 |
|
first check-in of sigma & hybrid-sigma coordinate code ; also includes |
| 9 |
|
non-linear free-surface parts ; but pressure-gradient not yet coded. |
| 10 |
|
o pkg/ptracers: |
| 11 |
|
option for ptracer to convert Salt Relax into additional EmP. |
| 12 |
|
tested in exp. global_ocean.90x40x15.dwnslp (update output) |
| 13 |
|
|
| 14 |
|
checkpoint62j (2010/08/25) |
| 15 |
|
o pkg/ggl90: |
| 16 |
|
other minor changes: remove 3-D temp array gTKE (+ add argument myIter) |
| 17 |
|
o pkg/dic: car_flux.F, phos_flux.F |
| 18 |
|
- reduce number of exp-calls in order to improve performance |
| 19 |
|
- clean up a little (replace fields by their macros, etc., fix |
| 20 |
|
typo in CPP-flag) |
| 21 |
|
o pkg/ggl90: minor changes |
| 22 |
|
- GGL90diffKrS was removed --> always use GGL90diffKr |
| 23 |
|
- GGL90viscAr was removed --> replaced with GGL90viscArU, GGL90viscArV |
| 24 |
|
- hack of mxlMaxFlag=2 --> ensure mixing between first and second level |
| 25 |
|
(commented out for now) |
| 26 |
|
- change in max/min operations to ensure that smoothing is ok |
| 27 |
|
- smoothing of GGL90viscAr was moved to ggl90_calc.F (as done for GGL90diffKr) |
| 28 |
|
- always use diffKrNrT as background profile (i.e. never use diffKr field) |
| 29 |
|
o pkg/cfc: bug fix, AREA(i,j,1,bi,bj) -> AREA(i,j,bi,bj) |
| 30 |
|
o pkg/dic: |
| 31 |
|
- model/src/impldiff.F: if TARGET_NEC_SX, extend loop ranges for better |
| 32 |
|
vectorization |
| 33 |
|
- small bug fix in insol for the unlikely case of a cartesian grid |
| 34 |
|
- towards vectorizing the dic package: bio_export.F, car_flux.F, phos_flux.F |
| 35 |
|
+ now vectorize, which also removes adjoint recomputations; |
| 36 |
|
+ replace integer maksp1 by _RL maskp1 |
| 37 |
|
+ replace a**x by exp(x*log(a)) in phos_flux.F for speed |
| 38 |
|
|
| 39 |
|
checkpoint62i (2010/07/21) |
| 40 |
|
o pkg/flt: |
| 41 |
|
- with OBCS: stop floats which enter the OB region |
| 42 |
|
o model/src: |
| 43 |
|
- use tempVertAdvScheme & saltVertAdvScheme (instead of horizontal ones) |
| 44 |
|
for implicit vertical advection. |
| 45 |
|
o pkg/seaice: |
| 46 |
|
- account for melting in seaice age evolution. |
| 47 |
|
- remove double counting of seaice ridging effect on seaice age. |
| 48 |
|
|
| 49 |
|
checkpoint62h (2010/06/18) |
| 50 |
|
o pkg/exch2: |
| 51 |
|
- remove exch2_*xy_rx files: call was replaced with exch2_*3d_rx(myNz=1). |
| 52 |
|
o tools/genmake2 |
| 53 |
|
- remove "-pdefault" option: redundant with the use of "packages.conf"; |
| 54 |
|
- remove file pkg/pkg_default : set the default as a pkg group |
| 55 |
|
("default_pkg_list") defined in pkg/pkg_groups; |
| 56 |
|
- fix for case pkg/pkg_depend is missing (was hanging before); |
| 57 |
|
- add option "-pgroups" to specify a different pkg_groups than the default. |
| 58 |
|
o verification/adjustment.cs-32x32x1: |
| 59 |
|
add a "minimal" set-up test (code_min & input_min) which only compiles |
| 60 |
|
eesupp/src and the 2 pkgs (exch2 & debug) which don't need model/src & inc. |
| 61 |
|
o tools/genmake2 |
| 62 |
|
- extend pkg_depend rules to model & eesupp (e.g., if model depend on [pkg])) |
| 63 |
|
- fix details of checking for mnc & profile pkg when NetCDF is unavailable. |
| 64 |
|
o pkg/exf: merge exf_bulk_largeyeager04 into exf_bulkformulae because these |
| 65 |
|
files are nearly the same anyway. |
| 66 |
|
Step 1: make exf_bulkformulae look like exf_bulk_largeyeager04: |
| 67 |
|
- redefinition of tau(i,i) (by a factor of ustar) |
| 68 |
|
- 1./exf(Const/T) -> exf(-Const/T) (big change in the results) |
| 69 |
|
- replace rhonil by rhoConstFresh in computation of evap (bug fix) |
| 70 |
|
- replace u/vstress=tau*rd*ws*cw by tau*rd*u/vwind (changes results) |
| 71 |
|
- replace |
| 72 |
|
rd(i,j)= rdn(i,j)/(exf_one - rdn(i,j)/karman*psimh ) |
| 73 |
|
shn = sh(i,j,bi,bj)*rd(i,j)/rdn(i,j) |
| 74 |
|
by |
| 75 |
|
shn = sh(i,j,bi,bj)/(exf_one - rdn(i,j)/karman*psimh ) |
| 76 |
|
(changes results) |
| 77 |
|
-> exf_bulkformulae v1.22, |
| 78 |
|
update verification experiments lab_sea, offline_exf_seaice, seaice_obcs |
| 79 |
|
(lab_sea: output.hb87.txt 1.11, output.lsr.txt 1.25, output.salt_plume.txt 1.13, |
| 80 |
|
output.txt 1.63 |
| 81 |
|
offline_exf_seaice: output.seaicetd.txt 1.7 |
| 82 |
|
seaice_obcs: output.txt 1.10) |
| 83 |
|
Step 2: do the actual merge and remove exf_bulk_largeyeager04.F |
| 84 |
|
-> exf_bulkformulae v1.23, |
| 85 |
|
unfortunately requires updating lab_sea.salt_plume (6 matching digits), |
| 86 |
|
and global_ocean.cs32x15.icedyn (only 11 matching digits) |
| 87 |
|
Also update output_adm of exp. lab_sea (all 4) and offline_exf_seaice. |
| 88 |
|
o adjoint of exch-1: |
| 89 |
|
- pass high-level EXCH S/R through TAF (eesupp/src/exch_ad_diff.list) |
| 90 |
|
(note: was already like this when using EXCH-2) and remove flow dir |
| 91 |
|
and hand-written AD version of high-level EXCH S/R (pkg/autodiff); |
| 92 |
|
- provide flow-dir for low-level exch1 S/R (eesupp/src/exch1_ad.flow); |
| 93 |
|
- update pkg/exch2 diff list. |
| 94 |
|
|
| 95 |
|
checkpoint62g (2010/05/19) |
| 96 |
|
o eesupp/src (exch-1): |
| 97 |
|
- split exch_rx.F into Forward S/R (exch1_rx.F) and Adjoint S/R (exch1_rx_ad.F) |
| 98 |
|
and remove argument "simulationMode"; same thing for all exch_*rx_cube.F |
| 99 |
|
(renamed to exch1_*rx_cube.F). |
| 100 |
|
- change accordingly exch_*_rx.template and AD version in pkg/autodiff |
| 101 |
|
to call new exch1 S/R ; update Makefile and .cvsignore. |
| 102 |
|
- replace call to exch2_xy_rx S/R with exch2_*3d_rx(myNz=1) (will remove |
| 103 |
|
exch2_xy_rx later) |
| 104 |
|
Note: after updating your code, need to run "make clean" in eesupp/src |
| 105 |
|
o test experiment bottom_ctrl_5x5: |
| 106 |
|
new output_adm.txt with fewer grdchk & less monitor output. |
| 107 |
|
o model/src: |
| 108 |
|
remove the call to ini_cori in initialise_varia.F (just keep the usual one |
| 109 |
|
in initialise_fixed.F). More consistent with coriolis & horizontal grid |
| 110 |
|
beeing treated in the same way (ini_grid is not called a 2nd time). |
| 111 |
|
o pkg/exf: |
| 112 |
|
- rewrite exf_bulk_largeyeager04.F for better vectorization, changes |
| 113 |
|
global_ocean.cs32x15/input.icedyn (11 digits of agreement), updated |
| 114 |
|
- fix adjoint (used worked for ALLOW_ATM_WIND defined, works for this now) |
| 115 |
|
- cosmetic changes to exf_bulkformulae.F to make it look more like |
| 116 |
|
exf_bulk_largeyeager04.F |
| 117 |
|
o eesupp/src (exch-1): |
| 118 |
|
- Fix AD EXCH code for 2-D domain special cases (Nx=1 or Ny=1). |
| 119 |
|
- Separate buffer filling and MPI sending: allow EXCH-1 to work for local |
| 120 |
|
array (non-shared) when using MPI+MTH. Also reduces number of BARRIER |
| 121 |
|
(even without using MPI). |
| 122 |
|
- Message mode: move RecvAck setting (indicator of buffer being ready) |
| 123 |
|
from send_put to recv_get S/R (was useless before, but not sure if |
| 124 |
|
it's much more useful now); |
| 125 |
|
- Switch the order of sync: MPI-proc 1rst and then threads; |
| 126 |
|
- take out spin-waiting code (#undef EXCH_USE_SPINNING), use BARRIER instead. |
| 127 |
|
- fix AD code for communication mode ='get' (but not used). |
| 128 |
|
o model/src: |
| 129 |
|
initialise dPhiHydX,dPhiHydY inside calc_grad_phi_hyd.F (so that TAF does |
| 130 |
|
not drop it in S/R mddynamics, producing NaN in hs94.1x64x5 AD-test exp). |
| 131 |
|
o utils/matlab/rdmnc.m: adapt to be able to use the generic matlab netcdf API |
| 132 |
|
o pkg/ctrl: require a specific set of control variables and exf-flags for |
| 133 |
|
adjoint of seaice (stop in ctrl_check.F if conditions are not satisfied) |
| 134 |
|
o model/src: |
| 135 |
|
a) 1rst version of a new driver S/R to turn-off output freq and flags after the |
| 136 |
|
1rst forward sweep, to avoid over-writing output files during recomputations |
| 137 |
|
recomputations (& grdchk); called from cost_final.F (& ctrl_init_ctrlvar.F ?) |
| 138 |
|
b) 2nd version: |
| 139 |
|
- add specific pkg S/R for seaice & ptracers (called from S/R TURNOFF_MODEL_IO); |
| 140 |
|
- switch-off more output flags (exf_monFreq, chkPtFreq, dumpInitAndLast, |
| 141 |
|
writePickupAtEnd) + seaice & ptracers output; |
| 142 |
|
- do not switch off MNC flags (otherwise, won't get admonitor netcdf file) |
| 143 |
|
o model/src: |
| 144 |
|
- new S/R rotate_uv2en_rl (..._rs) that rotates vector fields. |
| 145 |
|
o pkg/exch2: |
| 146 |
|
- remove "simulationMode" from arg. list of S/R EXCH2_RX1,2_CUBE ; |
| 147 |
|
- add argument "signOption" to EXCH2_RX1_CUBE (will be needed for SM exch). |
| 148 |
|
o pkg/autodiff & exch2: |
| 149 |
|
process exch_uv_agrid_3d_rx.f & exch2_uv_agrid_3d_rx.f through TAF |
| 150 |
|
(for now, only if ALLOW_EXCH2 is defined). |
| 151 |
|
o pkg/autodiff: |
| 152 |
|
- include PACKAGES_CONFIG.h in exch*ad.flow & exch*ad.F |
| 153 |
|
(not supposed to be in CPP_OPTIONS.h) to get ALLOW_EXCH2 defined correctly. |
| 154 |
|
+ clean-up comments (_RS is not always real*4) |
| 155 |
|
- fix bug in ADEXCH_UV_3D_RX (without EXCH2, was calling the forward exch S/R) |
| 156 |
|
- add a stop in ADEXCH_UV_AGRID_3D_RX if trying to use it with EXCH2: |
| 157 |
|
was calling a forward exch2, which is wrong ; used in SEAICE & EXF pkgs. |
| 158 |
|
o testreport (AD-tests): |
| 159 |
|
no longer link files from dir "../input" (1rst link files from |
| 160 |
|
"../input_ad.[xx]" if secondary test "xx" and then from "../input_ad") |
| 161 |
|
|
| 162 |
|
checkpoint62f (2010/04/29) |
| 163 |
o model/src/the_main_loop.F: |
o model/src/the_main_loop.F: |
| 164 |
adjust store directives for seaice evp and multicategory according to |
adjust store directives for seaice evp and multicategory according to |
| 165 |
pkg/ecco/the_main_loop.F |
pkg/ecco/the_main_loop.F |
| 166 |
o adjoint: |
o adjoint: |
| 167 |
global_sum_ad was incomplete (noted many times by many people ;) |
global_sum_ad was incomplete (noted many times by many people ;) |
| 168 |
nice description by Jean U. as follows: |
nice description by Jean U. as follows: |
| 169 |
* allreduce(var) in global_sum.F can be interpreted as |
* allreduce(var) in global_sum.F can be interpreted as |
| 170 |
reduce(sum) followed by broadcast of sum. |
reduce(sum) followed by broadcast of sum. |
| 171 |
* The adjoint of that is |
* The adjoint of that is |
| 172 |
sum(adjoint) followed by broadcast(summed adjoint value) |
sum(adjoint) followed by broadcast(summed adjoint value) |
| 173 |
or in other words, allreduce(sum, adjoint vars) |
or in other words, allreduce(sum, adjoint vars) |
| 174 |
We were missing the sum(adjoint)! |
We were missing the sum(adjoint)! |
| 175 |
Benign, except(!) for "global" scalars appearing in tile loops (bi,bj) |
Benign, except(!) for "global" scalars appearing in tile loops (bi,bj) |
Parent Directory
| 
Revision Log
| 
Revision Graph
| 
Patch