Skip to content

Document core and linear EOS variable units#559

Merged
marshallward merged 2 commits into
NOAA-GFDL:dev/gfdlfrom
Hallberg-NOAA:core_var_units
Feb 13, 2024
Merged

Document core and linear EOS variable units#559
marshallward merged 2 commits into
NOAA-GFDL:dev/gfdlfrom
Hallberg-NOAA:core_var_units

Conversation

@Hallberg-NOAA
Copy link
Copy Markdown
Member

Modified comments to document the units of 17 variables in 5 files in the core directory and another 5 variables in MOM_EOS_linear.F90. Only comments are changed and all answers are bitwise identical.

  Modified comments to document the units of 17 variables in 5 files in the core
directory and another 5 variables in MOM_EOS_linear.F90.  Only comments are
changed and all answers are bitwise identical.
@Hallberg-NOAA Hallberg-NOAA added the documentation Improvements or additions to documentation label Jan 29, 2024
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (de7e5b7) 37.20% compared to head (1935175) 37.20%.

Additional details and impacted files
@@            Coverage Diff            @@
##           dev/gfdl     #559   +/-   ##
=========================================
  Coverage     37.20%   37.20%           
=========================================
  Files           271      271           
  Lines         80438    80438           
  Branches      14999    14999           
=========================================
  Hits          29930    29930           
  Misses        44942    44942           
  Partials       5566     5566           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@marshallward
Copy link
Copy Markdown
Member

Gaea regression: https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/22279 ✔️

@marshallward marshallward merged commit c37a2f0 into NOAA-GFDL:dev/gfdl Feb 13, 2024
@Hallberg-NOAA Hallberg-NOAA deleted the core_var_units branch May 10, 2024 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants