Skip to content

Fix rescaling in regularize_surface debugging#438

Merged
marshallward merged 2 commits into
NOAA-GFDL:dev/gfdlfrom
Hallberg-NOAA:fix_regularize_layer_debugging
Aug 14, 2023
Merged

Fix rescaling in regularize_surface debugging#438
marshallward merged 2 commits into
NOAA-GFDL:dev/gfdlfrom
Hallberg-NOAA:fix_regularize_layer_debugging

Conversation

@Hallberg-NOAA
Copy link
Copy Markdown
Member

Add missing dimensional rescaling factors for hard-coded temperature and salinity offsets in debugging conservation checks in regularize_surface. All answers are bitwise identical, but without this change some configurations can erroneously give fatal errors for some rescaling settings when DEBUG=True.

  Add missing dimensional rescaling factors for hard-coded temperature and
salinity offsets in debugging conservation checks in regularize_surface.  All
answers are bitwise identical, but without this change some configurations can
erroneously give fatal errors for some rescaling settings when DEBUG=True.
@Hallberg-NOAA Hallberg-NOAA added the bug Something isn't working label Aug 5, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Aug 5, 2023

Codecov Report

Merging #438 (07c80c6) into dev/gfdl (9e756af) will not change coverage.
The diff coverage is 0.00%.

❗ Current head 07c80c6 differs from pull request most recent head b493019. Consider uploading reports for the commit b493019 to get more accurate results

@@            Coverage Diff            @@
##           dev/gfdl     #438   +/-   ##
=========================================
  Coverage     38.06%   38.06%           
=========================================
  Files           269      269           
  Lines         77187    77187           
  Branches      14256    14256           
=========================================
  Hits          29385    29385           
  Misses        42479    42479           
  Partials       5323     5323           
Files Changed Coverage Δ
...rameterizations/vertical/MOM_regularize_layers.F90 4.81% <0.00%> (ø)

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

@marshallward
Copy link
Copy Markdown
Member

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

@marshallward marshallward merged commit 2642c1c into NOAA-GFDL:dev/gfdl Aug 14, 2023
@Hallberg-NOAA Hallberg-NOAA deleted the fix_regularize_layer_debugging branch September 27, 2023 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants