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

Add ch4 surface custom cmor table entry #2168

Merged
merged 4 commits into from
Aug 10, 2023

Conversation

hb326
Copy link
Contributor

@hb326 hb326 commented Aug 10, 2023

Description

This pull request adds a custom cmor table entry for surface methane data (monthly mean values), and a test for this new entry.

Closes #2164

Link to documentation:


Before you get started

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.

hb326 added 3 commits August 1, 2023 15:58
New file to define the surface methane measurements.
Adding a test for the new custom cmor table "ch4s"
@hb326 hb326 requested a review from a team August 10, 2023 09:04
@codecov
Copy link

codecov bot commented Aug 10, 2023

Codecov Report

Merging #2168 (16585a3) into main (6b43ca2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2168   +/-   ##
=======================================
  Coverage   93.10%   93.10%           
=======================================
  Files         237      237           
  Lines       12822    12822           
=======================================
  Hits        11938    11938           
  Misses        884      884           

@remi-kazeroni remi-kazeroni added the cmor Related to the CMOR standard label Aug 10, 2023
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.

Nice job, @hb326! Looks good to me 👍 Just a very minor suggestion before the merge

esmvalcore/cmor/tables/custom/CMOR_ch4s.dat Outdated Show resolved Hide resolved
Co-authored-by: Rémi Kazeroni <remi.kazeroni@dlr.de>
@hb326
Copy link
Contributor Author

hb326 commented Aug 10, 2023

Nice job, @hb326! Looks good to me 👍 Just a very minor suggestion before the merge

Thanks, @remi-kazeroni!
I have made your suggested changes.

@hb326 hb326 requested a review from remi-kazeroni August 10, 2023 13:45
Copy link
Contributor

@valeriupredoi valeriupredoi left a comment

Choose a reason for hiding this comment

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

danke @hb326 - Remi, let me know when you happy so I merges 🍺

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.

New custom variable for surface methane data
3 participants