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 units to 11 log_param calls #262

Merged
merged 2 commits into from
Dec 14, 2022

Conversation

Hallberg-NOAA
Copy link
Member

Add appropriate units arguments to 11 log_param calls that were missing them.
All answers are bitwise identical, but the logged output can change in some cases.

@codecov
Copy link

codecov bot commented Dec 2, 2022

Codecov Report

Merging #262 (135c60d) into dev/gfdl (c80db67) will increase coverage by 0.00%.
The diff coverage is 80.00%.

❗ Current head 135c60d differs from pull request most recent head 8228029. Consider uploading reports for the commit 8228029 to get more accurate results

@@            Coverage Diff            @@
##           dev/gfdl     #262   +/-   ##
=========================================
  Coverage     37.12%   37.13%           
=========================================
  Files           263      263           
  Lines         73444    73444           
  Branches      13675    13675           
=========================================
+ Hits          27268    27270    +2     
+ Misses        41151    41148    -3     
- Partials       5025     5026    +1     
Impacted Files Coverage Δ
src/core/MOM_PressureForce_Montgomery.F90 10.71% <0.00%> (ø)
src/tracer/oil_tracer.F90 0.00% <0.00%> (ø)
src/core/MOM_PressureForce_FV.F90 42.10% <100.00%> (ø)
src/core/MOM_barotropic.F90 58.98% <100.00%> (ø)
src/core/MOM_verticalGrid.F90 63.72% <100.00%> (ø)
src/framework/testing/MOM_file_parser_tests.F90 94.15% <100.00%> (ø)
src/framework/MOM_document.F90 74.33% <0.00%> (+0.44%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

  Add appropriate units arguments to 11 log_param calls that were missing them.
All answers are bitwise identical, but the logged output can change in some
cases.
@marshallward
Copy link
Member

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

@marshallward marshallward merged commit c6c84c8 into NOAA-GFDL:dev/gfdl Dec 14, 2022
@Hallberg-NOAA Hallberg-NOAA deleted the log_param_units branch February 2, 2023 13:30
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.

2 participants