[MITgcm-devel] Re: [MITgcm-cvs] MITgcm/model/src CVS Commit

[Jean-Michel Campin]

Hi,

I propose to un-do those changes in dynamics.F since :

a) they are not likely to speed-up the model
 (in fact, if they have an impact on the speed, they should
 slow-down the code, since the list of diagnostics is checked 2 times
 instead of 1)
b) in other places of the code where diagnostics are filled directly,
 it's done without this function call to "DIAGNOSTICS_IS_ON"
 and I don't see the point of diversifying the way direct filling is done.
 (might bring more confusion for someone who wants to add new diagnostics)
c) if I do it quickly enough, I will not have to update my working version
 of dynamics.F (that CVS will not be able to successfully update, given the
 tricky modification that I put in it).

Jean-Michel

On Sat, Jun 25, 2005 at 09:42:52AM -0400, Andrea Molod wrote:
> Update of /u/gcmpack/MITgcm/model/src
> In directory forge:/tmp/cvs-serv31175
> 
> Modified Files:
> 	dynamics.F 
> Log Message:
> Insert if sequence to check for whether diagnostic is activated
> 
> _______________________________________________
> MITgcm-cvs mailing list
> MITgcm-cvs at mitgcm.org
> http://mitgcm.org/mailman/listinfo/mitgcm-cvs