Skip to content

FMS2: New interface to set/nullify_domain#394

Merged
Hallberg-NOAA merged 1 commit into
NOAA-GFDL:dev/gfdlfrom
marshallward:fms_io_domain_wrappers
Jun 30, 2023
Merged

FMS2: New interface to set/nullify_domain#394
Hallberg-NOAA merged 1 commit into
NOAA-GFDL:dev/gfdlfrom
marshallward:fms_io_domain_wrappers

Conversation

@marshallward
Copy link
Copy Markdown
Member

This patch adds wrappers to the set_domain and nullify_domain functions used in FMS1 for internal FMS IO operations. These are not used in FMS2, so the wrapper functions are empty.

This is required to eliminate FMS1 IO dependencies in SIS2.

This patch adds wrappers to the set_domain and nullify_domain functions
used in FMS1 for internal FMS IO operations.  These are not used in
FMS2, so the wrapper functions are empty.

This is required to eliminate FMS1 IO dependencies in SIS2.
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 30, 2023

Codecov Report

Merging #394 (b519a4d) into dev/gfdl (b0289fe) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

❗ Current head b519a4d differs from pull request most recent head 3d548b4. Consider uploading reports for the commit 3d548b4 to get more accurate results

@@             Coverage Diff              @@
##           dev/gfdl     #394      +/-   ##
============================================
- Coverage     38.21%   38.21%   -0.01%     
============================================
  Files           269      269              
  Lines         76407    76413       +6     
  Branches      14033    14033              
============================================
  Hits          29201    29201              
- Misses        41963    41969       +6     
  Partials       5243     5243              
Impacted Files Coverage Δ
config_src/infra/FMS1/MOM_domain_infra.F90 36.07% <0.00%> (-0.37%) ⬇️

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

Copy link
Copy Markdown
Member

@Hallberg-NOAA Hallberg-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These are useful changes for helping insulate MOM6 and SIS2 from changes to the FMS interfaces. They are have passed the pipeline testing at https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/19683.

@Hallberg-NOAA Hallberg-NOAA merged commit d44c228 into NOAA-GFDL:dev/gfdl Jun 30, 2023
@marshallward marshallward deleted the fms_io_domain_wrappers branch May 8, 2024 14:56
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