[MITgcm-devel] [MITgcm-cvs] MITgcm/pkg/ctrl CVS Commit

Tue Apr 4 22:04:49 EDT 2017

Hi Jean-Michel and Gael,

I have run both forward and adjoint testreports after correcting a couple of bugs. Attached please find the the reports. All but a few of the experiments passed the tests. Are those failed experiments a concern or not? I ran the tests on Pleiades.

I have also run the adjoint testreport for Gael's experiments of global_oce_cs32 and global_oce_llc90. They both passed. 

If the report looks fine, I will check in my updated code. 

Thanks,
Ou 
________________________________________
From: Jean-Michel Campin [jmc at mit.edu]
Sent: Monday, April 03, 2017 10:15 PM
To: mitgcm-devel at mitgcm.org
Cc: Wang, Ou (329B)
Subject: Re: [MITgcm-devel] [MITgcm-cvs] MITgcm/pkg/ctrl CVS Commit

Hi Ou,

Seems that even with forward testreport, one experiment (global_with_exf)
fails to compile, because of changes in pkg/profiles
For now, I've just turned this pkg off in this experiment.

Regarding the TAF adjoint testreport, 4 experiments are no longer compiling:
 1D_ocean_ice_column
 global_ocean.90x40x15
 global_ocean_ebm
 lab_sea
And the interesting thing is that it's not due to a single error but
the first one that shows up in the log file is different from one
experiment to the other.

We always advise to run testreport (at least a forward one) before any check-in;
But if you have access to TAF and specially when any of these 4 pkgs:
autodiff/ctrl/cost/ecco is involved, a second "-adm" one seems necessary,
at least to me.

So, at this point, I don't know what to suggest.
But let's be clear, witching off all the "-adm" daily testreport (easy)
would save me significant amount of time.

Cheers,
Jean-Michel

On Mon, Apr 03, 2017 at 09:03:34PM -0400, Jean-Michel Campin wrote:
> Hi Ou,
>
> Are you done with today check-in ? I am asking this because I tried to
> run a tesrreport -adm and does not seem to work very well (but I
> was also testing some of my changes).
>
> But in case you plan to to check-in more changes, just a comment
> regarding the time in the day to check-in changes to the code
> (since you are on Pacific time), it's generally better to avoid the
> midnight - 1.am (EST) (or even better, 2.am) window since most of
> the automatic testreport we run here are starting around this time.
>
> Cheers,
> Jean-Michel
>
> On Mon, Apr 03, 2017 at 07:16:38PM -0400, Wang, Ou wrote:
> > Update of /u/gcmpack/MITgcm/pkg/ctrl
> > In directory forge:/tmp/cvs-serv18128/pkg/ctrl
> >
> > Modified Files:
> >     ctrl_get_gen.F ctrl_map_ini_genarr.F
> > Log Message:
> > Add and update some files related to ECCO v4 release 3. The main changes are
> >  adding cost_gencost_glbmean.F to compute the costs for global mean time-series of OBP
> >  & SSH, splitting the profile cost to sample-mean and -anomaly parts,
> >  updating the cost formula for the proxy sea-ice costs, adding initial velocity
> >  as part of the control variables, and applying the correction of global
> >  mean steric height change to OBP.
> > - cost_gencost_glbmean.F:    compute cost for global mean OBP & SSH time-series
> > - cost_profiles.F:           add a CPP option (ALLOW_PROFILES_SAMPLESPLIT_COST)
> >                               that if defined splits the profile cost to
> >                               sample-mean and sample-anomaly costs.
> > - cost_gencost_seaicev4.F:   add a CPP option (SEAICECOST_JPL) that if defined
> >                              reads in the weights for the proxy sea-ice costs.
> > - cost_gencost_curstomize.F: if ALLOW_PSBAR_STERIC is defined, correct OBP with
> >                               the global mean steric ssh change to be consistent
> >                               with SSH.
> > - ecco_phys.F:               include sterGloH in a common block to be used in
> >                               cost_gencost_customize.F to correct OBP when
> >                               ALLOW_PSBAR_STERIC is defined.
> > - ctrl_map_ini_genarr.F:     add CPP options to include initial UV as the control
> >                               variables.
> > - cost_gencost_all.F:        changes related to cost for the global mean time-series.
> > - cost_gencost_bpv4.F:       minor change related to cost for the global mean time-series.
> > - ecco_readparms.F:          changes related to cost for the global mean time-series.
> > - ecco_ad_diff.list:         include the new added cost_gencost_glbmean.F.
> > - ctrl_get_gen.F:            check if the first seven characters of xx_tauu_file &
> >                               xx_tauv_filei matches "xx_tauu" or "xx_tauv".
> > - ecco_cost_final.F:         add modification related to the sample-mean profile cost.
> > - ecco_cost_init_fixed.F:    initialize the data and weights for the global mean costs
> >                               for OBP & SSH, if ALLOW_GENCOST_1D is defined.
> > - profiles_readvector.F:     read in climatology field in the profile file, and mask
> >                               out the weight where climatology data is flagged. Active
> >                               if ALLOW_PROFILES_CLIMMASK is defined.
> > - PROFILES_SIZE.h:           changes related to the split of profile cost to sample-mean
> >                               and sample-anomaly.
> > - profiles.h:                changes related to the split of profile cost to sample-mean
> >                               and sample-anomaly.
> > - ecco.h:                    changes related to the global mean costs
> > - profiles_readparms.F:      changes related to the profile cost splitting and masking
> >                               by climatology data.
> > - profiles_init_fixed.F:     changes related to the profile cost splitting.
> > - profiles_init_varia.F:     changes related to the profile cost splitting.
> >
> >
> >
> >
> > _______________________________________________
> > MITgcm-cvs mailing list
> > MITgcm-cvs at mitgcm.org
> > http://mitgcm.org/mailman/listinfo/mitgcm-cvs
>
> _______________________________________________
> MITgcm-devel mailing list
> MITgcm-devel at mitgcm.org
> http://mitgcm.org/mailman/listinfo/mitgcm-devel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tr_pfe21_20170404_3.tar.gz
Type: application/x-gzip
Size: 37278 bytes
Desc: tr_pfe21_20170404_3.tar.gz
URL: <http://mitgcm.org/pipermail/mitgcm-devel/attachments/20170405/5df59c06/attachment-0002.gz>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: tr_pfe21_20170404_17.tar.gz
Type: application/x-gzip
Size: 11568 bytes
Desc: tr_pfe21_20170404_17.tar.gz
URL: <http://mitgcm.org/pipermail/mitgcm-devel/attachments/20170405/5df59c06/attachment-0003.gz>