~reducedmodelling/fluidity/ROM_Non-intrusive-ann

Viewing all changes in revision 3909.2.1.

  • Committer: Adam Candy
  • Date: 2012-02-09 22:34:28 UTC
  • mto: (3924.1.6 netcdftests)
  • mto: This revision was merged to the branch mainline in revision 3925.
  • Revision ID: adam.candy@imperial.ac.uk-20120209223428-flh0ntxyfmkpayoe
Fix to bug lp:922025.  This tidies up the manner in which the pressure initial condition is set from a free-surface height field provided in a netCDF file.

Summary of changes:
 - The special case, legacy method has been removed.
 - If the input format is not recognised, the code will abort with an error detailing the problematic field and the schema path of the invalid format.
 - The 'standard' format is now named 'raw' (instead of 'other').  This is the case where no post-processing is performed on the source data, and the field is initialised by the raw data present in the netCDF file.
 - Included logging messages to indicate which field is being populated and the source netCDF file (logging level 1).
 - Logging of the post-processing applied (logging level 3).
 - For the pressure boundary condition set from a free-surface field in particular, logging level 3 will report the min and max values of the field before and after the conversion.

There are currently no tests, examples or longtests that are affected by the above change, but possibly user-generated initialisation flml files that do use the legacy implementation of the pressure initial condition set from free-surface netCDF.  In this case, the additional logging added with this change should make it easy to identify the change required to adopt the improved version of this import.

In short, the following example highlights the change necessary:

The option path:
  /material_phase::Ocean/scalar_field::Pressure/prognostic/initial_condition::WholeMesh/from_netcdf/format/Free-surface height

should become:
  /material_phase::Ocean/scalar_field::Pressure/prognostic/initial_condition::WholeMesh/free_surface/from_netcdf/format/raw

(where the file location is specified under the 'from_netcdf' node in each case).

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: