Skip to content

Feature/allowmaskchanges#1

Merged
jiandewang merged 19 commits into
jiandewang:feature/update-MOM6-20210402from
DeniseWorthen:feature/allowmaskchanges
Apr 9, 2021
Merged

Feature/allowmaskchanges#1
jiandewang merged 19 commits into
jiandewang:feature/update-MOM6-20210402from
DeniseWorthen:feature/allowmaskchanges

Conversation

@DeniseWorthen
Copy link
Copy Markdown

Changes to add allow_landmask_changes as variable in default_vars and MOM_templates.

@DeniseWorthen
Copy link
Copy Markdown
Author

@jiandewang Here are the changes required to add the parameter to default_vars and the templates. I tested:

cpld_controlfrac
cpld_controlfrac_c192
cpld_controlfrac_c384
datm_control_cfs
datm_control_gefs
datm_mx025_cfsr
datm_mx025_gefs

and all passed current baseline

@jiandewang
Copy link
Copy Markdown
Owner

@jiandewang Here are the changes required to add the parameter to default_vars and the templates. I tested:

cpld_controlfrac
cpld_controlfrac_c192
cpld_controlfrac_c384
datm_control_cfs
datm_control_gefs
datm_mx025_cfsr
datm_mx025_gefs

and all passed current baseline

look good

@jiandewang jiandewang merged commit fc66762 into jiandewang:feature/update-MOM6-20210402 Apr 9, 2021
@DeniseWorthen DeniseWorthen deleted the feature/allowmaskchanges branch April 10, 2021 18:19
jiandewang pushed a commit that referenced this pull request Apr 22, 2021
…y#524)

* point to Jun fv3atm personal fork
* add CDEPS submodule
* add building instructions
* fix issues that came from building
* builds CDEPS targets
* missed w3nco dep
* cleanup use of FoX and remove redundant library links in components
* Add 3 files generated by genf90.pl to the CDEPS-interface/ufs directory. (#1)
* remove dependency on genf90.pl for CDEPS/share
* remove FoX (#2)
remove FoX dependency building and use ESMF Config instead.
Co-authored-by: Jun Wang <junwang-noaa@users.noreply.github.com>
Co-authored-by: BinLi-NOAA <bin.li@noaa.gov>
Co-authored-by: Brian Curtis <brian.curtis@noaa.gov>
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