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

the coupled model requires FV3 and CMEPS updates for frac grid normalization #366

Closed
DeniseWorthen opened this issue Jan 13, 2021 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Jan 13, 2021

Description

Both FV3 and CMEPS will have an update to normalize the fluxes being sent from FV3 through CMEPS for the ocean and ice. When these are ready, the coupled model will need to be updated and tested.

Related Issues

FV3 #230
CMEPS #29

@DeniseWorthen
Copy link
Collaborator Author

This change will involve normalizing the fluxes from ATM to the OCN by the open water fraction that ATM used to calculate those fluxes. In CMEPS, the field imported from the ATM will have the name Sa_ofrac and will be carried in the fractional FieldBundle with the name aofrac.

The normalization change will be implemented in esmFieldsExchange_nems when the fluxes are mapped from ATM to OCN by changing the normalization from none to aofrac.

Because the packed field bundles can have only one normalization type, with this change I needed to change the maptype for the sea level pressure from consf to bilinear. This is actually the correct mapping type for sea level pressure since it is a global unmasked field. (I believe that consf mapping type was carried over from the NEMS mediator, when all fields mapped from ATM to OCN used nearest neighbor conservative.)

When the mapping type is changed from consf to bilinear for sea level pressure, the fields of sea level pressure exported to the OCN are in fact different (by small amounts). Two effects are seen. First, in the 1deg ocean configuration, the answers were bit identical to mapping sea level pressure conservatively. This can only be true if the 1deg configuration is not actually using this field. See MOM6 issue #58.

Secondly, the creation of the bilinear routehandle resulted in a small number of points now being mapped from OCN to land points (on the ATM grid) when those fields were mapped with conservative destination. This indicates some cross-contamination of the routehandles, either at the ESMF or CMEPS level. Following a suggestion by @rsdunlapiv, I added the retrieval of the dstStatusField from the RegridStore in CMEPS. This field contains a flag for reporting the type of masking which is done at each point in the destination grid when the routehandle is being created. The values contained in this field are described here. In the case where a bilinear routehandle is being created, the points which now appear as OCN->ATM land are all associated with a dstStatusField of 5. When no bilinear routehandle is created, these points all contain a dstStatusField of 0. Since these points are in fact land, the correct value in this case is in fact 0.

We're continuing to debug this issue now, with additional help from the ESMF team.

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Mar 2, 2021 via email

@DeniseWorthen
Copy link
Collaborator Author

@junwang-noaa I would need to understand more before giving a certain answer. I only have a very vague memory of this from very early days when we were coupling GSM and HYCOM.

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Mar 2, 2021 via email

@DeniseWorthen
Copy link
Collaborator Author

CMEPS has this mapping type available:

mappatch_uv3d = 9 ! rotate u,v to 3d cartesian space, map from src->dest, then rotate back

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Mar 3, 2021 via email

epic-cicd-jenkins pushed a commit that referenced this issue Apr 17, 2023
…on for geogrid files (fix_am directory) (#366)

* Point exregional_make_ics.sh script to the new location for geogrid files (fix_am directory).
Set the config.community.sh external model staging user-defined variable to "false."

* Change path for geogrid files on Hera.

* Remove if statements for geogrid files in exregional_make_ics.sh, and update paths to fix_am/fix_sfc_climo in setup.sh to be consistent across platforms and remove paths to personal directories.

* Remove GRID_GEN_METHOD and reset EXPT_SUBDIR value. Keep USE_USER_STAGED_EXTRN_FILES set to "TRUE" but comment it out and add additional information for the user to uncomment this line if they wish to use this feature.

* Reset numerous variables to correct values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants