Skip to content

CMIP7 requirements test #790

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

Merged
merged 24 commits into from
Apr 7, 2025
Merged

CMIP7 requirements test #790

merged 24 commits into from
Apr 7, 2025

Conversation

mauzey1
Copy link
Collaborator

@mauzey1 mauzey1 commented Apr 4, 2025

Changes to enable CMOR to write CMIP7 datasets.

  • The branding_suffix of variables will be tested against the branding_suffix template provided by the CV. Its components temporal_label, vertical_label, horizontal_label, and area_label will have their values validated by their corresponding sections in the CV.
  • mip_era and data_specs_version will be set from the CV.
  • frequency can be defined in the user input JSON or by calling cmor_set_cur_dataset_attribute prior to creating the time axis and variable that uses the axis.
  • Interval values for the frequencies are acquired from the CV if the current dataset has a frequency. If the interval value used for the time axis doesn't match the interval values of the frequency used by the variable, then CMOR will throw an error.
  • region can be defined in the user input JSON or by calling cmor_set_cur_dataset_attribute prior to calling cmor_write.
  • Table attributes type, positive, valid_max, valid_min, ok_max_mean_abs, and ok_min_mean_abs are now acquired from the header of tables and used as the default for a table's variable entries.
  • variable_title and brand_description can now be variable attributes if defined in the variable entry.
  • _cmip7_option should be set in the user input JSON to enable CMIP7 global attribute checking similar to CMIP6 but without subexperiment tests. parent_mip_era will be set to CMIP7 for all datasets that are written with _cmip7_option set.
  • data_archive_id has been changed to archive_id in the CV.
  • host_collection has been added to the experiment_id entries of the CV.

mauzey1 added 24 commits March 12, 2025 17:49
@mauzey1 mauzey1 requested a review from durack1 April 7, 2025 16:01
@mauzey1 mauzey1 marked this pull request as ready for review April 7, 2025 16:05
Copy link
Contributor

@durack1 durack1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mauzey1 looks great

@mauzey1 mauzey1 merged commit 752530b into main Apr 7, 2025
15 of 25 checks passed
@mauzey1 mauzey1 deleted the 762-cmip7-requirements branch April 7, 2025 17:25
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.

2 participants