New URL for NEMO forge!   http://forge.nemo-ocean.eu

Since March 2022 along with NEMO 4.2 release, the code development moved to a self-hosted GitLab.
This present forge is now archived and remained online for history.
2020WP/ENHANCE-04_dford_OBSOP_BGC – NEMO
wiki:2020WP/ENHANCE-04_dford_OBSOP_BGC

Version 5 (modified by dford, 4 years ago) (diff)

--

Name and subject of the action

Last edition: Wikinfo(changed_ts)? by Wikinfo(changed_by)?

The PI is responsible to closely follow the progress of the action, and especially to contact NEMO project manager if the delay on preview (or review) are longer than the 2 weeks expected.

  1. Summary
  2. Preview
  3. Tests
  4. Review

Summary

Action ENHANCE-04_dford_OBSOP_BGC
PI(S) David Ford
Digest Adapt OBS code to make it easier to add new (especially biogeochemical) variables in a generic fashion
Dependencies None
Branch source:/NEMO/branches/2020/dev_r13747_ENHANCE-04_dford_OBSOP_BGC
Previewer(s) Dan Lea
Reviewer(s) Dan Lea
Ticket #2567

Description

Adapt observation operator code so that variables are handled more generically as possible in the code, making it easier to add new variables. This will be particularly useful for adding new biogeochemical profile variables from different models.

Implementation

Error: Failed to load processor box
No macro or processor named 'box' found

Change some of the internal handling of profile observation variables. Currently it's only set up to be able to handle inputs with two variables in, either temperature and salinity, or U&V velocity. This change will remove those restrictions, so that users adding an observation operator for third party biogeochemical model variables will be able to more easily. The changes also add some error checking on the variable names read in from observation files, as currently these aren't checked.

This will require edits to the following modules in OCE/OBS:

  • diaobs.F90
  • obs_oper.F90
  • obs_prep.F90
  • obs_read_prof.F90
  • obs_read_surf.F90
  • obs_write.F90

The following new variables will be added to the &namobs namelist:

  • ln_default_fp_indegs
  • rn_default_avglamscl
  • rn_default_avgphiscl
  • nn_2dint_default [renamed from nn_2dint]

Documentation updates

Error: Failed to load processor box
No macro or processor named 'box' found

namobs to be updated in doc/namelists/namobs to reflect the changes noted above.

Preview

Error: Failed to load processor box
No macro or processor named 'box' found

These are changes which have already been incorporated and used in the branch source:/branches/UKMO/dev_r5518_obs_oper_update. See r9306 and r11546, noting that the changes specific to third party biogeochemical models will not be included here.

Tests

Error: Failed to load processor box
No macro or processor named 'box' found

The SETTE tests have been run with the updated code, and all pass. Note that the fix described in #2568 is required for the OBS and ASM code to be tested correctly.

No changes to results are expected or seen with ln_diaobs either activated or deactivated.

Due to the issue in #2568 the test doesn't seem to run properly in the trunk, so I can't compare feedback files before and after. But all the output feedback files have been created as expected, and the _Hx variables contain expected model values. I've copied the contents of a working directory, including output, to /scratch/frdf/sette/REPRO_4_8 on the Met Office system.

The changes have also been used extensively in source:/branches/UKMO/dev_r5518_obs_oper_update, and shown not to change any results.

Review

Error: Failed to load processor box
No macro or processor named 'box' found

...