Changes between Version 8 and Version 9 of ticket/0937

Timestamp:

2012-10-25T14:22:33+02:00 (12 years ago)

Author:

cbricaud

Comment:

--

ticket/0937

@@ -23 +23 @@
 Testing could consider (where appropriate) other configurations in addition to NVTK].
 
-|| NVTK Tested || !'''YES!''' ||
-|| Other model configurations || !'''NO!''' ||
+|| NVTK Tested || YES ||
+|| Other model configurations || NO ||
 || Processor configurations tested || [ sette layouts ] ||
-|| If adding new functionality please confirm that the [[BR]]New code doesn't change results when it is switched off [[BR]]and !''works!'' when switched on || !'''NO!''' ||
+|| If adding new functionality please confirm that the [[BR]]New code doesn't change results when it is switched off [[BR]]and !''works!'' when switched on || NO ||
 
-(Answering UNSURE is likely to generate further questions from reviewers.)
-
-'Please add further summary details here'
-
- * Processor configurations tested
- * etc----
+Apart from NVTK tests, developments have been checked with AMM12 configuration with a 8 x 4 processor layout.
 
 === Bit Comparability ===
 || Does this change preserve answers in your tested standard configurations (to the last bit) ? || NO ||
-|| Does this change bit compare across various processor configurations. (1xM, Nx1 and MxN are recommended) || !'''YES!''' ||
-|| Is this change expected to preserve answers in all possible model configurations? || !'''NO!''' ||
-|| Is this change expected to preserve all diagnostics? [[BR]]!,,!''Preserving answers in model runs does not necessarily imply preserved diagnostics. !'' || !'''NO!''' ||
+|| Does this change bit compare across various processor configurations. (1xM, Nx1 and MxN are recommended) || YES ||
+|| Is this change expected to preserve answers in all possible model configurations? || NO ||
+|| Is this change expected to preserve all diagnostics? [[BR]]!,,!''Preserving answers in model runs does not necessarily imply preserved diagnostics. !'' || NO ||
 
 __If key_bdy and key_tide are not activated, modifications do not change either answers or results of standard configurations__. Thus results changes only concern AMM12 standard configuration. These changes only come from the completely different tidal module implemented. Thus, there are not induced by the modifications of bdy package alone as long as the same options are used. Keeping new and previous tidal modules would have make the code and namelist interface much heavier so that we preferred to keep the new one.
@@ -45 +40 @@
 Note that some issues with overlapping analytical segments have been fixed in bdy package. It is now possible to obtain identical results with user defined "analytical" open boundary segments bounding the domain (ie identical to the case where a single unstructured segment is read from a netcdf file). As a consequence, diagnostics in AMM12 change although no change in the diagnostics themselves has been introduced.
 
-If you answered !'''NO!''' to any of the above, please provide further details:
-
- * Which routine(s) are causing the difference?
- * Why the changes are not protected by a logical switch or new section-version
- * What is needed to achieve regression with the previous model release (e.g. a regression branch, hand-edits etc). If this is not possible, explain why not.
- * What do you expect to see occur in the test harness jobs?
- * Which diagnostics have you altered and why have they changed?Please add details here........
-
 ----
 === System Changes ===
-|| Does your change alter namelists? || !'''YES!''' ||
-|| Does your change require a change in compiler options? || !'''NO!''' ||
+|| Does your change alter namelists? || YES ||
+|| Does your change require a change in compiler options? || NO ||
 
- * namelist changes:
+ * '''namelist changes:'''
    * nam_tide: Simplifications in the definition of tidal waves. Add a flag and time_scale to activate linear ramping of tidal motions.
    * nambdy_tide: Add a new option to read tidal boundary data from 2d file given on data domain.
@@ -64 +51 @@
    * nambdy_index: Simplify definition of analytical segments.
    * namsbc_apr: Flag to add inverse barometer at open boundaries. Options to set reference pressure to a constant value or to the domain average.
- * About compiler options: Code update does not require any change in compiler options. Yet, we noticed that results are not reproductible unless the lowest level of optimization is used.
+ * '''About compiler options:''' Code update does not require any change in compiler options. Yet, we noticed that results are not reproductible unless the lowest level of optimization is used.
 
 ----