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

CMOR 3.7.0 #668

Merged
merged 34 commits into from
Aug 22, 2022
Merged

CMOR 3.7.0 #668

merged 34 commits into from
Aug 22, 2022

Conversation

mauzey1
Copy link
Collaborator

@mauzey1 mauzey1 commented Aug 18, 2022

CMOR 3.7.0 release

Features

  • CMOR automatically sets attributes in a dataset using the registered data listed in a project CV's source_id entry (e.g. Tables/CMIP6_CVs.json or Tables/input4MIPs.json), reducing the input requirements from a user. This feature can be overridden by setting the attributes in the user input file (e.g. user_input.json).
  • Added ability to write reftime and leadtime coordinates to datasets for forecast data.

Changes

  • End of support for Python 3.6.
  • CMOR now supports Python 3.10.
  • The CCLRC license has been replaced with a BSD-3 license.
  • Updated warning and error messages from retired listserv email address cmor@listserv.llnl.gov to pcmdi-cmip@llnl.gov.
  • Updated CMOR and CF versions listed in the Fortran wrapper.

Bugfixes

  • PrePARE would crash if the experiment_id attribute is not present in a dataset. It will now check for the presence of experiment_id before further processing the dataset.
  • PrePARE was generating errors when processing cell_methods values that contained multiple comments, which are substrings in parentheses embeded in the values. PrePARE can now handle cell_methods values containg mulitple comments.
  • Fixed error caused by an error message exceeding the maximum string size when an attribute value cannot be validated with a list of values from the CV.
  • Corrected typo in udunits initialization message.

Closed Issues

Related issue: #637

mauzey1 and others added 30 commits August 2, 2022 16:18
Remove numpy.distutils.core and unnecessary modules from setup.py.in
Use clang-14 for OSX test environment
Merge miniconda setup into build step to fix clang version issue
Install cdms2 from conda-forge, and then run configure script
Change order of conda channels
Change environment name for cdms2 test
.zenodo.json Outdated Show resolved Hide resolved
include/cmor.h Outdated Show resolved Hide resolved
setup.py.in Outdated Show resolved Hide resolved
@durack1
Copy link
Contributor

durack1 commented Aug 19, 2022

@mauzey1 those release notes look great!

Might I suggest that

  • CMOR will now automatically set attributes in a dataset using the registered data listed in a project CV's source_id entry. This feature can be overridden by setting the attributes in the user input.
    is updated to

  • CMOR automatically sets attributes in a dataset using the registered data listed in a project CV's source_id entry (e.g. Tables/CMIP6_CVs.json or Tables/input4MIPs.json), reducing the input requirements from a user. This feature can be overridden by setting the attributes in the user input file (e.g. user_input.json).

@durack1
Copy link
Contributor

durack1 commented Aug 19, 2022

@mauzey1 one additional release notes catch

from

to

@mauzey1
Copy link
Collaborator Author

mauzey1 commented Aug 19, 2022

@durack1 The mailing list change was from #653, which replaced cmor@listserv.llnl.gov with pcmdi-cmip@llnl.gov.

cmor@llnl.gov will be listed on the documentation site's contact page.

@durack1
Copy link
Contributor

durack1 commented Aug 19, 2022

@mauzey1 got it, this is a within cmor error message update, ala #653 (comment).

Sorry, I had confused this with PCMDI/cmor3_documentation#112

As I now know what you are referring to, let's change the release notes from

to

Copy link
Collaborator

@piotr-florek-mohc piotr-florek-mohc left a comment

Choose a reason for hiding this comment

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

All looking good, approved.

@durack1
Copy link
Contributor

durack1 commented Aug 22, 2022

@mauzey1 if we haven't received a confirmation from folks, we can remove the guesses I made and proceed with the release! Thanks

@mauzey1 mauzey1 merged commit 8ef92d2 into master Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants