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

het_diags from mo_chm_diags #251

Merged
merged 9 commits into from
Sep 20, 2023
Merged

het_diags from mo_chm_diags #251

merged 9 commits into from
Sep 20, 2023

Conversation

jaelynlitz
Copy link
Collaborator

isolating this completed function from still in progress chm_diags

@codecov
Copy link

codecov bot commented Sep 19, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (acc7a46) 96.58% compared to head (4e17875) 96.59%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #251   +/-   ##
=======================================
  Coverage   96.58%   96.59%           
=======================================
  Files          31       32    +1     
  Lines        6379     6394   +15     
=======================================
+ Hits         6161     6176   +15     
  Misses        218      218           
Files Changed Coverage Δ
src/mam4xx/mo_chm_diags.hpp 100.00% <100.00%> (ø)

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

Copy link
Collaborator

@jeff-cohere jeff-cohere 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! I asked one question.

Copy link
Contributor

@mjs271 mjs271 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!

@jaelynlitz jaelynlitz merged commit 941930e into main Sep 20, 2023
7 checks passed
@jaelynlitz jaelynlitz deleted the jaelynlitz/het_diags branch September 20, 2023 05:18
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.

3 participants