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

[Wang, Ou (329B)]

Hi Jean-Michel,

Thanks for reporting this. I will fix it ASAP.

Regards,
Ou

On 4/3/17, 10:15 PM, "Jean-Michel Campin" <jmc at mit.edu> wrote:

>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