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.

Sign up for GitHub

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

Jump to bottom

Refactor setting the source and destination mask values for mapping #64

Closed
DeniseWorthen opened this issue Apr 4, 2022 · 0 comments · Fixed by ESCOMP/CMEPS#279
Closed

Refactor setting the source and destination mask values for mapping #64

DeniseWorthen opened this issue Apr 4, 2022 · 0 comments · Fixed by ESCOMP/CMEPS#279
Assignees
@DeniseWorthen

Comments

@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Apr 4, 2022 •
edited
Loading

Currently a large section of code in med_map_mod.F90 is used to define the src and dst masking for various component couplings.

CMEPS/mediator/med_map_mod.F90

Line 393 in df272f1

if (trim(coupling_mode) == 'cesm') then
Most of these settings are in fact default values for a component and should be set more directly according whether a component is acting as a source or destination.
@DeniseWorthen DeniseWorthen mentioned this issue Apr 4, 2022
Merged
5 tasks
DeniseWorthen added a commit to DeniseWorthen/CMEPS that referenced this issue Apr 23, 2022
@DeniseWorthen
Refactor nems field exchange; set default masks for mapping in med_in…
ef360ea 
…ternalstate (ESCOMP#279)

Refactors esmFldsExchange_nems.F90 to use separate advertise and initialize phases and to check that a component is present before advertising a field to or from that component. Implements default src and dst mask values in place of the code currently in med_map_mod.F90. Fixes NOAA-EMC#63 and NOAA-EMC#64.
This was referenced Apr 25, 2022
Merged
Merged
@DeniseWorthen DeniseWorthen self-assigned this Apr 27, 2022
junwang-noaa pushed a commit that referenced this issue Jun 9, 2022
@uturuncoglu @climbfuji
@fvitt @jedwards4b @mvertens @DeniseWorthen
Update CMEPS to its ESCOMP/master branch 20220607 commit (#67)
424a086 
* add new flux computation for UFS model and add new coupling mode for exchange grid implementation
* fix area field for new flux algorithm
* send fluxes to atmospheric model
* initial implementation for sending fluxes to UFS ATM
* update CCPP host model
* fix latent and sensible heat fluxes and clean code
* add new coupling mode for side by side flux comparison
* fix CCPP host model for latent and sensible heat fluxes
* fix aoflux calculation on agrid and add missing error checks
* add option to write meshes and update code that retrieve area information from xgrid
* update ccpp host based on recent changes in ccpp framework
* fix for providing cell area to CCPP host model
* make ccpp physics options configurable
* Refactor nems field exchange; set default masks for mapping in med_internalstate (ESCOMP#279)
Refactors esmFldsExchange_nems.F90 to use separate advertise and initialize phases and to check that a component is present before advertising a field to or from that component. Implements default src and dst mask values in place of the code currently in med_map_mod.F90. Fixes #63 and #64.
* use mesh file instead of grid name (ESCOMP#285)
This was done so that vertical component used in grid name does not affect tests.

Co-authored-by: Dom Heinzeller <[email protected]>
Co-authored-by: Francis Vitt <[email protected]>
Co-authored-by: Courtney Peverley <[email protected]>
Co-authored-by: Jim Edwards <[email protected]>
Co-authored-by: mvertens <[email protected]>
Co-authored-by: Denise Worthen <[email protected]>
Co-authored-by: Mariana Vertenstein <[email protected]>
Co-authored-by: Ufuk Turuncoglu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DeniseWorthen DeniseWorthen

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor nems field exchange; set default masks for mapping in med_internalstate DeniseWorthen/CMEPS
1 participant
@DeniseWorthen