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

cmeps integration changes #55

Merged

Conversation

DeniseWorthen
Copy link
Contributor

@DeniseWorthen DeniseWorthen commented Apr 24, 2020

  • issue Field name changes for cmeps integration #48 : Changes two field names for the nems mediator for consistency w/ cmeps (mean_zonal_moment_flx_atm and mean_merid_moment_flux). An additional 3 field names are added to the NUOPC dictionary for the NEMS mediator but these are not currently being used.

Also updates some CMEPS configuration consistent with current master of CMEPS.

Related PRs:
fv3atm PR #104
CCPP physics PR #440
ufs-weather-model PR #108

DeniseWorthen and others added 30 commits October 4, 2019 10:57
merge noaa-emc/nems develop
Change acc# on WCOSS to GFS-DEV (#17)
debug mode for NEMS, additional minor changes in component mk files (…
Code changes to support DEBUG compilation based on appBuilder specifi…
remove ice-ocean merge in prep_atm (#24)
…h mvertens/nems_integration2_update3 that currently has a PR to cmeps master
Update submodule pointer for NCEPLIBS-pyprodutil (NOAA-EMC#51)
@@ -334,6 +356,17 @@ SUBROUTINE EARTH_REGISTER(EARTH_GRID_COMP,RC_REG)
file=__FILE__)) &
return ! bail out
endif
if (.not.NUOPC_FieldDictionaryHasEntry( &
"mean_sensi_heat_flx_atm")) then
Copy link
Collaborator

Choose a reason for hiding this comment

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

Denise, we do not have this change on "mean_sensi_heat_flx_atm" and "mean_laten_heat_flx_atm" in fv3atm, do we need to change them in fv3 too?

@DeniseWorthen
Copy link
Contributor Author

DeniseWorthen commented Apr 24, 2020

We do not need them in FV3. The name used by FV3 (mean_sensi_heat_flx) is still defined in the Dictionary and is used to define the fldsFrAtm in the mediator. Only the momentum fluxes need to be changed in the mediator and FV3 to be consistent w/ the CMEPS exchange fields.

@junwang-noaa
Copy link
Collaborator

  • issue Field name changes for cmeps integration #48 : Changes two field names for the nems mediator for consistency w/ cmeps (mean_zonal_moment_flx_atm and mean_merid_moment_flux). An additional 3 field names are added to the NUOPC dictionary for the NEMS mediator but these are not currently being used.

Also updates some CMEPS configuration consistent with current master of CMEPS.

Denise,
Please add fv3atnm PR #104, CCPP physics PR #440, ufs-weather-model PR #108 to this ticket. Thanks.

@junwang-noaa
Copy link
Collaborator

Final RT passed, code is ready for commit after review. Thanks

@MinsukJi-NOAA MinsukJi-NOAA merged commit 82138ff into NOAA-EMC:develop Apr 28, 2020
DeniseWorthen added a commit to DeniseWorthen/NEMS that referenced this pull request Apr 28, 2020
DeniseWorthen added a commit to DeniseWorthen/NEMS that referenced this pull request May 23, 2020
* support for ufs-s2s-model debug compilaiton (NOAA-EMC#40)

* automatically enable NEMS_BUILDOPT=DEBUG=Y based on coupledFV3_MOM6_CICE_debug.appBuilder
* automatically use esmf/8.0.0g module based on coupledFV3_MOM6_CICE_debug.appBuilder

* Change debug flag check from ifeq to findstring

* Loading machine-dependent esmf debug module

*Use S2S_DEBUB_MODULE env var to load machine-dependent esmf debug module
*Require ufs-s2s-model/modules/${BUILD_TARGET}/fv3_coupled_debug file

* cmeps integration changes (NOAA-EMC#55)

* Bugfix i2a & Add Orion to module-setup (NOAA-EMC#61)

* Add Orion to module-setup.*.inc

* src/incmake/component_CCPP.mk: SUITES argument no longer mandatory for calling ccpp_prebuild.py (NOAA-EMC#62)

Co-authored-by: Minsuk Ji <57227195+MinsukJi-NOAA@users.noreply.github.com>
Co-authored-by: Mariana Vertenstein <mvertens@ucar.edu>
Co-authored-by: Dom Heinzeller <dom.heinzeller@icloud.com>
@DeniseWorthen DeniseWorthen deleted the feature/cmeps_integration_latest branch August 18, 2020 21:12
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.

3 participants