Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

User/jpk/cmor named diagnostics #34

Closed
wants to merge 5 commits into from

Conversation

jkrasting
Copy link
Contributor

No description provided.

adcroft and others added 5 commits July 3, 2014 13:37
- Renamed register_diag_field() to register_diag_field_low().
- Added wrapper register_diag_field() to call register_diag_field_low().

TODO:
[ ] Add optional arguments to provide CMOR names.
[ ] Add multiple invocations of register_diag_field_low() with the
    various different names that might appear in a diag_table.
[ ] Do same for static fields.
- Added wrapper to register_diag_field() to store second handle
  for CMOR registration.
- Added wrapper to post_data_*() to call send_data for CMOR handles.
- Alternative handle is stored in diag control structure.
- Attribute fall-back logic for CMOR scalar fields
- Wrapped register_Z_tracer, added standard_name attribute
- Added CMOR names for the following diagnostics
   o  areacello (static)
   o  so (3D)
   o  so_z (3D-remapped to Z)
   o  tosga (scalar - in the style of a CMOR scalar diagnostic)
- Added entries for cmor fields to the diag table (global ALE Z example)
@jkrasting
Copy link
Contributor Author

Closing request ... going to check in something more comprehensive

@jkrasting jkrasting closed this Jul 17, 2014
gustavo-marques pushed a commit to gustavo-marques/MOM6 that referenced this pull request Nov 28, 2017
…_20nov2017

Merge dev/master candidate (2017-11-19)
MJHarrison-GFDL pushed a commit to MJHarrison-GFDL/MOM6 that referenced this pull request Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants