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

+Eliminated 3 unused elements of the surface type #145

Merged

Conversation

Hallberg-NOAA
Copy link
Member

Eliminated the salt_deficit, TempxPmE and internal heat elements of the
surface type. None of these particular elements are used outside of MOM6, and
they are not used again after they are set, and appear not to have been used in
many years. All answers are bitwise identical.

  Eliminated the salt_deficit, TempxPmE and internal heat elements of the
surface type.  None of these particular elements are used outside of MOM6, and
they are not used again after they are set, and appear not to have been used in
many years.  All answers are bitwise identical.
@codecov
Copy link

codecov bot commented Jun 13, 2022

Codecov Report

Merging #145 (5e41d06) into dev/gfdl (79eb807) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##           dev/gfdl     #145      +/-   ##
============================================
- Coverage     34.26%   34.25%   -0.01%     
============================================
  Files           257      257              
  Lines         69744    69721      -23     
  Branches      12913    12909       -4     
============================================
- Hits          23897    23885      -12     
+ Misses        41373    41365       -8     
+ Partials       4474     4471       -3     
Impacted Files Coverage Δ
src/core/MOM.F90 45.23% <ø> (+<0.01%) ⬆️
src/core/MOM_checksum_packages.F90 25.00% <ø> (ø)
src/core/MOM_forcing_type.F90 31.63% <0.00%> (-0.40%) ⬇️
src/core/MOM_variables.F90 16.93% <ø> (+0.78%) ⬆️

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 79eb807...5e41d06. Read the comment docs.

@marshallward
Copy link
Member

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

@marshallward marshallward merged commit 371c8bb into NOAA-GFDL:dev/gfdl Jun 20, 2022
@Hallberg-NOAA Hallberg-NOAA deleted the remove_unused_sfc_elements branch July 16, 2022 09:42
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