Changes between Version 1 and Version 2 of ticket/0944/Review

Timestamp:

2012-11-15T18:35:22+01:00 (11 years ago)

Author:

djlea

Comment:

--

ticket/0944/Review

@@ -4 +4 @@
 
 == For completion by the Sci/Tech/Code reviewer ==
-'''Reviewer:''' [ Enter your name here ]
+'''Reviewer:''' Daniel Lea
+
+=== Comments ===
+Happy to review. I'll do a full review soon.
+
+Some initial thoughts in response to https://forge.ipsl.jussieu.fr/nemo/browser/branches/2012/dev_r3342_MERCATOR7_SST revision 3436.
+
+Looks like you are following the same style as the profile daily averaging which is a good idea. However, unless I am mistaken, both your code and the profile code are rather inefficient in that they allocate the vdmean array even if it is not needed. It would be nice if you could avoid allocating this array in obs_surf_def if the night-time averaging is not being used.
+
+ld_sstnight should be set to false by default. Not all SST observation types may be suitable to be compared with the night-time average. I also think the expectation would be that an instantaneous observation would be compared to the model value at the same time (that is the case for all other observation types) so that should be the default and the ld_sstnight option available if you actively set it to true in the namelist. So you will need to add ln_sstnight to the namelist dia_obs_init.
+
+I previously suggested that it be possible to select the SST types which use the night-time average. I'm happy to not worry about this providing the scheme is default to off as stated above. Others can develop it further as and when required.
+
+Can you test that you get the same feedback files for SST and for other obs types (just to be sure), if you run with the NEMO trunk and with your branch but with with ld_sstnight = .false.?
+
+Please indent the code inside the if statement in obs_sst_opt.
+
+Daniel Lea  15/11/2012
 
 === Ticket Details, Documentation and Code changes ===