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

[Jean-Michel Campin]

Hi Ryan,

I have a problem with renaming diagnostic "LaTr#RHO" to "La  #RHO" in layers_calc.F:
a) it's not great to have white space within diagnostics name ;
b) this name change is not implemented in layers_diagnostics_init.F (this diagnostics 
  is currently broken);
c) I added this diagnostics few weeks ago ; I don't know why you want 
  to change it but we can discuss this ; and this was not in the set of 
  changes you sent me yesterday.
I would like to put back the previous diagnostics name (LaTr#RHO).

And there a a second problem:
the forward testreport fail to compile experiments exp4 & cfc_example.
(I did check the Adjoint testreport yesterday but not the forward which I thought
was OK according to your earlier email).
Could you fix this problem today ?

Cheers,
Jean-Michel

On Wed, Jun 03, 2015 at 09:39:22AM -0400, Ryan Abernathey wrote:
> Update of /u/gcmpack/MITgcm/pkg/layers
> In directory forge:/tmp/cvs-serv14197/pkg/layers
> 
> Modified Files:
> 	LAYERS.h layers_calc.F layers_diagnostics_init.F 
> 	layers_fluxcalc.F layers_init_fixed.F layers_init_varia.F 
> 	layers_save.F layers_thermodynamics.F 
> Log Message:
> Fixed bug in layers_fluxcalc.F regarding incorrect interpolation with
>  partial cells.
> Enhanced the LAYERS_THERMODYNAMICS diagnostic options.
> Renamed LaTr... diagnostic to La...
> 
> 
> 
> _______________________________________________
> MITgcm-cvs mailing list
> MITgcm-cvs at mitgcm.org
> http://mitgcm.org/mailman/listinfo/mitgcm-cvs