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

Added custom CMOR tables used for EMAC CMORizer #1599

Merged
merged 1 commit into from
May 31, 2022

Conversation

schlunma
Copy link
Contributor

@schlunma schlunma commented May 23, 2022

Description

This PR adds the custom CMOR tables used for the EMAC CMORizer to reduce the size of #1554.

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.


To help with the number pull requests:

@schlunma schlunma added the cmor Related to the CMOR standard label May 23, 2022
@schlunma schlunma added this to the v2.6.0 milestone May 23, 2022
@schlunma schlunma requested a review from axel-lauer May 23, 2022 14:17
@schlunma schlunma self-assigned this May 23, 2022
@codecov
Copy link

codecov bot commented May 23, 2022

Codecov Report

Merging #1599 (2cd9ab2) into main (2f28031) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1599   +/-   ##
=======================================
  Coverage   90.99%   90.99%           
=======================================
  Files         201      201           
  Lines       10676    10676           
=======================================
  Hits         9715     9715           
  Misses        961      961           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f28031...2cd9ab2. Read the comment docs.

@schlunma schlunma mentioned this pull request May 30, 2022
10 tasks
@schlunma schlunma requested a review from remi-kazeroni May 30, 2022 14:49
Copy link
Contributor

@remi-kazeroni remi-kazeroni left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks @schlunma!

Copy link
Contributor

@axel-lauer axel-lauer left a comment

Choose a reason for hiding this comment

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

I looked through the plots produced by the EMAC example recipe recipe_emac_new.yml and most things look good - as far as I can tell. There are, however, a few items that might need to be double-checked:

  • map of precip (given in kg/m2/s): the color scale ranges from 0-10, i.e. the plot is empty and evaluation is not possible.
  • map of zg at 500 hPa: the values over Antarctica and over Greenland look very strange, I would expect the values to range somewhere between 5000 and 6000 m but not between 0 and 3000 m as in the plot.
  • time series of surface pressure: assuming that the values shown are global mean values, the absolute values and the variability look wrong

Copy link
Contributor

@axel-lauer axel-lauer left a comment

Choose a reason for hiding this comment

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

Pasted comments into wrong PR. Ups...

@schlunma schlunma merged commit db7ca97 into main May 31, 2022
@schlunma schlunma deleted the add_custom_CMOR_tables branch May 31, 2022 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmor Related to the CMOR standard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants