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

EFSOI implementation in global-workflow #211

Closed
AndrewEichmann-NOAA opened this issue Dec 11, 2020 · 10 comments
Closed

EFSOI implementation in global-workflow #211

AndrewEichmann-NOAA opened this issue Dec 11, 2020 · 10 comments

Comments

@AndrewEichmann-NOAA
Copy link
Contributor

This issue is to track the implementation of EFSOI in global-workflow.

With the opening of the issue, it will pick up documentation of the development from Redmine task https://vlab.ncep.noaa.gov/redmine/issues/55974

Development is in https://github.com/AndrewEichmann-NOAA/global-workflow under feature/EFSOI

@AndrewEichmann-NOAA
Copy link
Contributor Author

For background, EFSOI runs a parallel GDAS task workflow starting with gdaseupd, called gdaseupdfsoi, followed by gdasecenfsoi, gdasesfcfsoi, gdasefmnfsoi, gdasepmnfsoi, and gdasefsoi (which is specific to EFSOI). All except gdasefsoi are largely the same as the regular gdas tasks, with the following exceptions:

  • gdaseupdfsoi runs with the fsoi flag on so that it outputs the osense file and uninflated fields
  • gdasefmnfsoi and gdasepmnfsoi run and process 30hr ensemble forecasts, specifically to get the 24h and 30h forecasts

The EFSOI-specific tasks have their own scripts in jobs/rocoto, which call EFSOI-specific scripts in jobs. These in turn call task scripts in the scripts directory. These scripts are derived from scripts in the GSI repository, but are currently in the fork of global-workflow. The goal is to have as little duplication as possible of these scripts, but for now they tend to have their EFSOI-specific versions for ease of development.

@AndrewEichmann-NOAA
Copy link
Contributor Author

At present, the EFSOI tasks have been run through gdasesfcfsoi, and is currently stuck on gdasefmnfsoi (the forecast). The latest resolved issue involved files necessary for a warm start not being available - since the EFSOI ensemble runs in its own tree (efsoigdas.YYYYMMDD) and is running there for its first cycle in the experiment (unlike the regular enkfgdas), there was no previous cycle to warm start from. At present the task is using the script exglobal_efsoi_forecast.sh (from exglobal_forecast.sh), and the variables prefix and rprefix were set to point to EFSOI ensemble. rprefix is used to point to the previous cycle for the warm start files, which don't exist in this case. To fix this, rprefix was set back to rprefix=enkf$rCDUMP to use the same warm start files as the regular ensemble, and the only change from the original script is this:
(base) -bash-4.2$ diff exglobal_forecast.sh exglobal_efsoi_forecast.sh
202c202
< prefix=enkf$CDUMP

prefix=efsoi$CDUMP
(base) -bash-4.2$

It will have to be determined whether the regular ensemble's warm start files can used this way as a solution.

@AndrewEichmann-NOAA
Copy link
Contributor Author

Current issue, as emailed to Kate:

For the EFSOI implementation I'm trying to get an ensemble forecast with a set of initial conditions different from the regular GDAS. To do this I'm running in a separate directory tree (efsoigdas instead of enkfgdas). I'm stuck at getting the model to run - it gets to the executable and then stops seemingly without a message:

gfs.x /scratch1/NCEPDEV/stmp2/Andrew.Eichmann/RUNDIRS/hedgehog/2019111706/gdas/efcsfsoi.91157/mem001/.2.657 + OMP_NUM_THREADS=42.657 + export OMP_NUM_THREADS2.657 + srun '--export=ALL' /scratch1/NCEPDEV/stmp2/Andrew.Eichmann/RUNDIRS/hedgehog/2019111706/gdas/efcsfsoi.91157/mem001/global_fv3gfs.x2.661 + 1>& 1 2>& 2

  • . * . * . * . * . * . * . * . * . * . * . * . * . * . * . * . * . * . * . * .     PROGRAM NEMS (Andrew.Eichmann) Thu Nov 19 23:00:58 UTC 2020 r845c887776c6 https://github.com/NOAA-EMC/NEMS HAS BEGUN. COMPILED 2020324.00     ORG: NEMSapplication called MPI_Abort(comm=0x84000004, 1) - process 60application called MPI_Abort(comm=0x84000004, 1) - process 180application called MPI_Abort(comm=0x84000004, 1) - process 10application called MPI_Abort(comm=0x84000004, 1) - process 50application called MPI_Abort(comm=0x84000004, 1) - process 70

The standard GDAS ensemble ran without a problem, and I've tried to make this as identical to that as possible, except for running with a different ensemble. Where should I be looking for messages? 

The above log is/scratch1/NCEPDEV/stmp4/Andrew.Eichmann/hedgehog/logs/2019111706/gdasefcsfsoi01.log
and its successful equivalent is /scratch1/NCEPDEV/stmp4/Andrew.Eichmann/hedgehog/logs/2019111706/gdasefcs01.log

The script stack (expected and verified in the log):
/scratch1/NCEPDEV/da/Andrew.Eichmann/fv3gfs/AFE-fork/global-workflow/jobs/rocoto/efcsfsoi.sh
/scratch1/NCEPDEV/da/Andrew.Eichmann/fv3gfs/AFE-fork/global-workflow/jobs/JGDAS_EFSOI_FCST 
/scratch1/NCEPDEV/da/Andrew.Eichmann/fv3gfs/AFE-fork/global-workflow/scripts/exgdas_enkf_fcst.sh
/scratch1/NCEPDEV/da/Andrew.Eichmann/fv3gfs/AFE-fork/global-workflow/scripts/exglobal_efsoi_forecast.sh

The only variable differences from a regular GDAS forecast should be:
export COMOUT="$ROTDIR/efsoi$CDUMP.$PDY/$cyc/$COMPONENT"

and 
export FORECASTSH=$HOMEgfs/scripts/exglobal_efsoi_forecast.sh

in JGDAS_EFSOI_FCST 

and the following:
(base) -bash-4.2$ diff exglobal_forecast.sh exglobal_efsoi_forecast.sh
202c202
<   prefix=enkf$CDUMP

  prefix=efsoi$CDUMP
(base) -bash-4.2$ 

The experiment directory is /scratch1/NCEPDEV/da/Andrew.Eichmann/para_gfs/hedgehog

@KateFriedman-NOAA
Copy link
Member

So since the error message is lovely and vague I compared the logs to look for a difference (I suspected resource issue based on the error). I found that $NTASKS_FV3 wasn't set in the failed efsoi job:

0.016 + NTASKS_FV3=''

...which led to PE_MEMBER01 also being empty in model_configure:

2.246 + 1> model_configure 0<< \EOF
total_member:            1
print_esmf:              .false.
PE_MEMBER01:
start_year:              2019

The setting of NTASKS_FV3 happens in exglobal_efsoi_forecast.sh (just like in exglobal_forecast.sh)

NTASKS_FV3=${NTASKS_FV3:-$npe_fv3}

Following npe_fv3 backwards...it comes out of config.fcst via npe_fcst:

export npe_fv3=$npe_fcst

...with a value of 424:

npe_fv3=424

...but then I see it getting reset in config.efcsfsoi and an error (line numbers included):

1098 0.061 + echo '  + 1 * 40'
1099 (standard_in) 1: syntax error
1100 0.063 + npe_fv3=''
1101 0.063 + export npe_fv3
1102 0.063 + npe_efcs=''
1103 0.063 + export npe_efcs

/scratch1/NCEPDEV/stmp4/Andrew.Eichmann/hedgehog/logs/2019111706/gdasefcsfsoi01.log

This part of that config is where the error comes from:

 14 export npe_fv3=$npe_efcs
 15
 16 if [ $QUILTING = ".true." ]; then
 17     export npe_fv3=$(echo " $npe_fv3 + $WRITE_GROUP * $WRTTASK_PER_GROUP" | bc)
 18     export npe_efcs=$npe_fv3
 19 fi

The value of $npe_fv3 on line 17 is unknown (cause $npe_efcs on line 14 is unknown) when it enters that block and causes the error that empties npe_fv3. I believe you want $npe_efcsfsoi instead of $npe_efcs on line 14 in config.efcsfsoi since this job isn't efcs so $npe_efcs doesn't come out of config.resources, $npe_efcsfsoi does. If you look a few lines above in the log (1083) you'll see $npe_efcsfsoi being set in config.resources.

So change $npe_efcs to $npe_efcsfsoi in config.efcsfsoi and rewind the failed job. Let me know how it goes.

@AndrewEichmann-NOAA
Copy link
Contributor Author

This worked! Revealing once again the problem with proliferating config files.

@AndrewEichmann-NOAA
Copy link
Contributor Author

Encountered another issue with the regular epos failing, it turning out because ensemble members 11-20 had no forecasts. This was traced back to the problem, previously enountered, with using NMEM_ENS=20 with export NMEM_EFCSGRP=2. Only five tasks of two members each get run. Solved by setting export NMEM_EFCSGRP=4 in config.efcsfsoi. Need to rig some way not get bit by this.

@AndrewEichmann-NOAA
Copy link
Contributor Author

eposfsoi failing early due to $npe_node_eposfsoi apparently not being defined, as discovered in env/HERA.env. Set to $npe_node_epos, as it probably doesn't need to be different. Will have to review all EFSOI-specific environment settings to see if redundant.

@AndrewEichmann-NOAA
Copy link
Contributor Author

Test running new build (experiment ibex) as pushed to repository, flushed out some problems with config files which hadn't been changed from Liaofan's versions. Merged with existing ecen and efcs configs. Edited, committed, and pushed. Also creating a test scripts to clone, build and run overnight.

@KateFriedman-NOAA
Copy link
Member

@AndrewEichmann-NOAA Please update issue, thanks!

@AndrewEichmann-NOAA
Copy link
Contributor Author

Tweaks to config files worked, closing. Thank you for your troubleshooting!

lgannoaa pushed a commit to lgannoaa/global-workflow that referenced this issue Feb 9, 2022
kayeekayee pushed a commit to kayeekayee/global-workflow that referenced this issue May 30, 2024
…OAA-EMC#211)

* updated CMakeLists.txt
* Changes for JEDI linking/control
* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing
* Revert change to .gitmodules and update submodule pointer for ccpp-physics

Co-authored-by: Mark Potts <Mark.Potts@noaa.gov>
kayeekayee pushed a commit to kayeekayee/global-workflow that referenced this issue May 30, 2024
* Update .gitmodules and submodule pointers for ccpp-framework and ccpp-physics for gsl/develop branch

* RUC ice for gsl/develop (replaces NOAA-GSL#54 and NOAA-GSL#56) (NOAA-GSL#60)

Implementation of RUC LSM ice model in CCPP

* Squash-merge climbfuji:rucice_gfsv16dzmin into gsl/develop

* Fix bug in gfsphysics/GFS_layer/GFS_typedefs.F90 from merge

* Remove lsm_ruc_sfc_sice from suite FV3_GSD_v0_unified_ugwp_suite and update submodule pointer for ccpp-physics

* Remove sfc_sice from ccpp/suites/suite_FV3_GSD_v0_unified_ugwp_suite.xml

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

Co-authored-by: Dom Heinzeller <climbfuji@ymail.com>

* Update gsl/develop from develop 2020/12/08 (NOAA-GSL#61)

* Fix for updating stochastic physics on separate time-step. (NOAA-EMC#199)

This bug fix allows the random patterns in the stochastic physics persist the for a period of time (defined as SKEBINT,SPPTINT, etc.) before calculating new patterns.
The fix is to move the allocation of the saved variables into the init section of stochastic_physics_wrapper, and remove the deallocates in the run section.

* Bug fixes in (1) running with frac_grid=T and GFDL MP and (2) restarting with frac_grid=T (NOAA-EMC#204)

* -- Pointing to Moorthi's modifications in ccpp/physics, which fixed the crash when running GFDL MP with frac_grid=T;
-- Not setting fice to zero in order to leave lake ice untouched;
-- Restart in the coupled model with the default physics is reproducible, if bad water temperature is only filtered at initial time;
Co-authored-with: Shrinivas Moorthi <shrinivas.moorthi@noaa.gov>
Co-authored-with: Denise Worthen <Denise.Worthen@noaa.gov>

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

Co-authored-by: Phil Pegion <38869668+pjpegion@users.noreply.github.com>
Co-authored-by: shansun6 <48043606+shansun6@users.noreply.github.com>

* Update submodule pointer for ccpp-physics - MYNN surface layer updates and bugfixes (NOAA-EMC#63)

* Land stochastic perturbations (wrapper PR for NOAA-EMC#65) (NOAA-EMC#68)

* Move initialization of stochastic physics after the physics
initialization in CCPP.
* Add albedo variables to land perturbations with lndp_type=2 option. Change to accommodate soil perturbations with RUC LSM.

* Max/min soil moisture variables are introduced via GFS_Control_type
variables instead of through the use of namelist_soilveg*. This is a
more flexible way for different LSMs.

* Added pores and resid variables for max/min soil moisture to GFS_typedefs.f90.

* Remove tracer_sanitizer from all suites and from CCPP prebuild config

* Add namelist option to apply land surface perturbations at every time step, clean up stochastic_physics/stochastic_physics_wrapper.F90

Co-authored-by: tanyasmirnova <tanya.smirnova@noaa.gov>

* Stochastic land perturbations: add roughness length over land to the perturbed variables (NOAA-EMC#70)

* Added roughness length over land to the perturbed variables.
* Bugfix in gfsphysics/GFS_layer/GFS_typedefs.F90: remove Diag%cldcov, in particular the reset call because the variable is not allocated

Co-authored-by: Dom Heinzeller <climbfuji@ymail.com>

* Update gsl/develop from develop 2021/02/22 (NOAA-EMC#73)

* Fix for updating stochastic physics on separate time-step. (NOAA-EMC#199)

This bug fix allows the random patterns in the stochastic physics persist the for a period of time (defined as SKEBINT,SPPTINT, etc.) before calculating new patterns.
The fix is to move the allocation of the saved variables into the init section of stochastic_physics_wrapper, and remove the deallocates in the run section.

* Bug fixes in (1) running with frac_grid=T and GFDL MP and (2) restarting with frac_grid=T (NOAA-EMC#204)

* -- Pointing to Moorthi's modifications in ccpp/physics, which fixed the crash when running GFDL MP with frac_grid=T;
-- Not setting fice to zero in order to leave lake ice untouched;
-- Restart in the coupled model with the default physics is reproducible, if bad water temperature is only filtered at initial time;
Co-authored-with: Shrinivas Moorthi <shrinivas.moorthi@noaa.gov>
Co-authored-with: Denise Worthen <Denise.Worthen@noaa.gov>

* jedi-ufs with ccpp-physics submodule update (contains NOAA-EMC#201) (NOAA-EMC#211)

* updated CMakeLists.txt
* Changes for JEDI linking/control
* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing
* Revert change to .gitmodules and update submodule pointer for ccpp-physics

Co-authored-by: Mark Potts <Mark.Potts@noaa.gov>

* Final-final GFS v16 updates / restart reproducibility bugfixes (NOAA-EMC#212)

* updates the submodule pointer for ccpp-physics for the final-final (!) GFS v16 physics updates
* fixes a bug in io/FV3GFS_io.F90 to obtain restart reproducibility for uncoupled and coupled runs - contributed by @SMoorthi-emc

Co-authored-by: Shrinivas.Moorthi <Shrinivas.Moorthi@noaa.gov>

* RRTMGP coupling to Thompson MP in ccpp-physics (NOAA-EMC#208)

* Updated physics. Added infrastructure to FV3 to handle new GP capabilities. Thompson MP, LW scattering, Use of LW-jacobian to update surface tendencies.
* Some small changes to work with physics improvements to RRTMGP.
* RRTMGP working with Thompson MP.
* Updated .gitmodules.
* Updated physics. New interstitials.
* Synced with NCAR/master
* Updated ccpp-physics
* Updated physics submodule.
* Cleanup
* Update physics
* Updated physics submodule pointer.
* Updated physics submodule hash.

* Minor bug fixes to CCPP UGWP (update submodule pointer for ccpp-physics) (NOAA-EMC#216)

Update the submodule pointer for ccpp-physics for minor bugfixes in CCPP's Unified Gravity Wave Drag parameterization.

* Remove IPD (step 1) (NOAA-EMC#215)

Remove all IPD source files that are not needed to compile with CCPP. Update gfsphysics/CMakeLists.txt and gfsphysics/GFS_layer/GFS_driver.F90 so that the code compiles after removing IPD sources.

Also: update submodule pointer for GFDL_atmos_cubed_sphere to include PR NOAA-EMC/GFDL_atmos_cubed_sphere#48 ("Removed use of mpp_node for use with FMS 2020.04").

* Update submodule pointers for ccpp-framework (ccpp_prebuild bugfix) and ccpp-physics (RRTMGP GFDL-MP bugfix) (NOAA-EMC#218)

Update the submodule pointers for ccpp-framework and ccpp-physics for the changes described in
NCAR/ccpp-framework#343 and NCAR/ccpp-physics#536.

* Remove unnecessary SIMD instruction sets from ccpp/CMakeLists.txt (NOAA-EMC#220)

* Remove additional/unnecessary SIMD instruction sets from ccpp/CMakeLists.txt

* Implementation of CCPP timestep_init and timestep_final phases (NOAA-EMC#217)

- replace calls to CCPP step `time_vary` with `timestep_init` in `atmos_model.F90`, add call to CCPP step `timestep_final` 
- update `ccpp/driver/CCPP_driver.F90` with calls to CCPP `timestep_init` and `timestep_final`
- add `h2o_def.f` and `ozne_def.` to `ccpp_prebuild_config.py`
- update of `gfsphysics/GFS_layer/GFS_typedefs.F90`: cleanup work for o3 and h2o physics (required by the updates to the CCPP time vary physics)
- update metadata in `gfsphysics/GFS_layer/GFS_typedefs.meta` for the above changes, and clean up the index used for the surface wind enhancement due to convection in the `phy_f2d` array (use proper index, not just the last entry in the array)

* add radiation_clouds_thompson_dependency in ccpp physics  (NOAA-EMC#225)

* using radiation_clouds_thompson_dependency ccpp physics branch
* point to ccpp master branch

* Remove IPD steps 3 and 5 (NOAA-EMC#224)

* Implementation of CCPP timestep_init and timestep_final phases in fv3atm; cleanup work in GFS_typedefs for o3 and h2o physics as a result of the changes to the time vary physics in CCPP
* Use proper index variable for surface wind enhancement due to convection in phy_f2d array in GFS_typedefs.{F90,meta}, move code to clear diagnostic buckets using GFS DDT bound procedures from atmos_model.F90 to CCPP_driver.F90
* First step of cleanup process: remove CCPP preprocessor directives, remove parts of unused IPD code, update cmake build system, Delete IPD source code, Replace IPD DDTs with GFS DDTs, Bugfix in atmos_model.F90; add missing call to GFS_externaldiag_populate, Move contents of gfsphysics/{CCPP_layer,GFS_layer} to ccpp/{data,driver}, entirely deleted IPD typedefs, Rename module GFS_driver to GFS_init
* Fix indentation in ccpp/data/CMakeLists.txt and ccpp/driver/CMakeLists.txt
* Add logic to set DYN32 depending on 32BIT setting
* Pass preprocessor directive GFS_TYPES to dycore to enable use of GFS data types
* Compile GFS_diagnostics.F90 without optimization, this leads to out of memory errors on wcoss_dell_p3

* Add CCPP rrtmgp fix (NOAA-EMC#237)

* ccpp physics merged with top of master
* point back to ccpp master branch

* Update develop from NOAA-GSL: RUC ice, MYNN sfclay, stochastic land perturbations (NOAA-EMC#239)

* Update .gitmodules and submodule pointers for ccpp-framework and ccpp-physics for gsl/develop branch
* RUC ice for gsl/develop (replaces NOAA-GSL#54 and NOAA-GSL#56) (NOAA-GSL#60) Implementation of RUC LSM ice model in CCPP
* Fix bug in gfsphysics/GFS_layer/GFS_typedefs.F90 from merge
* Remove lsm_ruc_sfc_sice from suite FV3_GSD_v0_unified_ugwp_suite and update submodule pointer for ccpp-physics
* Remove sfc_sice from ccpp/suites/suite_FV3_GSD_v0_unified_ugwp_suite.xml
* Update gsl/develop from develop 2020/12/08 (NOAA-GSL#61)
* Fix for updating stochastic physics on separate time-step. (NOAA-EMC#199)
This bug fix allows the random patterns in the stochastic physics persist the for a period of time (defined as SKEBINT,SPPTINT, etc.) before calculating new patterns.
The fix is to move the allocation of the saved variables into the init section of stochastic_physics_wrapper, and remove the deallocates in the run section.
* Bug fixes in (1) running with frac_grid=T and GFDL MP and (2) restarting with frac_grid=T (NOAA-EMC#204)
* -- Pointing to Moorthi's modifications in ccpp/physics, which fixed the crash when running GFDL MP with frac_grid=T;
-- Not setting fice to zero in order to leave lake ice untouched;
-- Restart in the coupled model with the default physics is reproducible, if bad water temperature is only filtered at initial time;
Co-authored-with: Shrinivas Moorthi <shrinivas.moorthi@noaa.gov>
Co-authored-with: Denise Worthen <Denise.Worthen@noaa.gov>
* Revert change to .gitmodules and update submodule pointer for ccpp-physics
* Update submodule pointer for ccpp-physics - MYNN surface layer updates and bugfixes (NOAA-EMC#63)
* Land stochastic perturbations (wrapper PR for NOAA-EMC#65) (NOAA-EMC#68)
* Move initialization of stochastic physics after the physics
initialization in CCPP.
* Add albedo variables to land perturbations with lndp_type=2 option. Change to accommodate soil perturbations with RUC LSM.
* Max/min soil moisture variables are introduced via GFS_Control_type
variables instead of through the use of namelist_soilveg*. This is a
more flexible way for different LSMs.
* Added pores and resid variables for max/min soil moisture to GFS_typedefs.f90.
* Remove tracer_sanitizer from all suites and from CCPP prebuild config
* Add namelist option to apply land surface perturbations at every time step, clean up stochastic_physics/stochastic_physics_wrapper.F90
* Stochastic land perturbations: add roughness length over land to the perturbed variables (NOAA-EMC#70)
* Added roughness length over land to the perturbed variables.
* Bugfix in gfsphysics/GFS_layer/GFS_typedefs.F90: remove Diag%cldcov, in particular the reset call because the variable is not allocated
* Update .gitmodules and submodule pointer for GFDL_atmos_cubed_sphere for code review and testing
* Revert change to .gitmodules for ccpp-physics, update submodule pointer for ccpp-physics
* Revert change to .gitmodules and update submodule pointer for GFDL_atmos_cubed_sphere
Co-authored-by: DomHeinzeller <58610420+DomHeinzeller@users.noreply.github.com>
Co-authored-by: Phil Pegion <38869668+pjpegion@users.noreply.github.com>
Co-authored-by: shansun6 <48043606+shansun6@users.noreply.github.com>
Co-authored-by: tanyasmirnova <tanya.smirnova@noaa.gov>

* Dycore change to add an option to zero-gradient BC for height advection and change dz_min as a namelist input (NOAA-EMC#232)

* Point dycore to a personal branch
* Point to new version of dycore
* Update the dycore version
* Update the dycore version
* Point to NOAA-EMC dev/emc dycore branch

* Remove gnumake build remnants, change v16beta to v16 (NOAA-EMC#234)

* Remove gnumake files
* Replace v16beta with v16 for all CCPP suites
* Remove unused CCPP_INCLUDE_DIRS/CCPP_LIB_DIRS from ccpp/CMakeLists.txt

* Move LSM vegetation lookup tables into CCPP, clean up RUC snow cover on ice initialization  (remove IPD step 2) (NOAA-EMC#244)

* Move LSM vegetation lookup tables into CCPP, clean up RUC snow cover on ice initialization
* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* UGWP v0 v1 combined (NOAA-EMC#241)

* .gitmodules/gsl atmos-cubed cccp-fram cccp/physics
* update fv3atm from NOAA-EMC Jan 9
* new suite_FV3_GFS_v16b_ugwpv1.xml and modifications Jan 11/2021
* Added new logical flag do_ugwp_v0_nst_only which allows non-stationary drag from ugwp_v0 to be run with GSL drag suite
* Fix formatting in ccpp/data/GFS_typedefs.*
* Clean up allocation of arrays in ccpp/data/GFS_typedefs.F90, add active attribute (pass 1) to ccpp/data/GFS_typedefs.meta, strip trailing whitespaces from both
* Update and cleanup of metadata for UGWPv0, UGWPv1, drag suite
* Reorganize UGWP diagnostic variables in data/GFS_typedefs.{F90,meta}
* Bugfix for uninitialized data in ccpp/data/GFS_typedefs.F90
* Update submodule pointer for ccpp-physics
* Allocate 3d diagnostic arrays as dummy arrays if not used

Co-authored-by: valery.yudin <valery.yudin@noaa.gov>
Co-authored-by: Michael Toy <michael.toy@noaa.gov>

* SDF for GFSv16 with Thompson (NOAA-EMC#238)

* SDF for GFSv16 with Thompson
* Delete suite_FV3_GFS_v16beta_thompson.xml
* update to new commit of Thompson MP in ccpp physics

Co-authored-by: Phil Pegion <38869668+pjpegion@users.noreply.github.com>
Co-authored-by: shansun6 <48043606+shansun6@users.noreply.github.com>
Co-authored-by: Mark Potts <Mark.Potts@noaa.gov>
Co-authored-by: Shrinivas.Moorthi <Shrinivas.Moorthi@noaa.gov>
Co-authored-by: dustinswales <dustin.swales@noaa.gov>
Co-authored-by: Jun Wang <37633869+junwang-noaa@users.noreply.github.com>
Co-authored-by: XiaqiongZhou-NOAA <48254930+XiaqiongZhou-NOAA@users.noreply.github.com>
Co-authored-by: valery.yudin <valery.yudin@noaa.gov>
Co-authored-by: Michael Toy <michael.toy@noaa.gov>
Co-authored-by: XiaSun-NOAA <58949533+XiaSun-NOAA@users.noreply.github.com>

* Remove inconsistencies in computation of air density with Thompson MP (NOAA-EMC#74)

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Update thompson mp 20210213 (NOAA-EMC#567) for gsl/develop (NOAA-EMC#75)

* Add logical convert_dry_rho for Thompson MP, hardcoded to false
* Initialize all variables to NaN in ccpp-physics in DEBUG mode
* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-framework and ccpp-physics

* &gfs_physics_nml print_diff_pgr=.true. will print per-timestep pgr change stats

* point to sam's repository for atmos_cubed_sphere

* Add units and lat/lon to per-timestep pgr stats

* Rebase atmos_cubed_sphere to NOAA-GFDL dev/emc branch

* Print mean pgr change as hPa/hr and other pgr data as bars.

* Remove changes from atmos_cubed_sphere and simplify atmos_model.F90 changes.

* print mean abs change instead of mean change in pgr

* Update .gitmodules and submodule pointers for ccpp-framework and ccpp-physics for code review and testing

* Revert change to .gitmodules and update submodule pointers for ccpp-framework and ccpp-physics

* clarify one print

* More 3D diagnostic tendencies (NOAA-EMC#76)

Add diagnostic tendencies for all tracers. Move all d*3dt variables into a 4D sparse array stored as `dtend(i,k,dtidx(tracer,process))` to reduce memory usage.

* Fix incorrect units in dtend metadata (NOAA-EMC#82)

* Fix incorrect units in dtend metadata

* Make the tracer tendency units mandatory.

* Add SO2 tendency units (NOAA-EMC#86)

* Update .gitmodules and submodule pointers for ccpp-framework and ccpp-physics

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-framework and ccpp-physics

* dtend: add missing tendencies and fix bugs in convection tendencies (NOAA-EMC#92)

* Fix incorrect units in dtend metadata

* Point to corresponding ccpp/physics branch

* fixes to dtend support:

1. Store rtg (AKA clw AKA qtr) instead of gq0 in DCNV and SCNV
pre/post for schemes that use convective transport. Tracers handled
solely by convective transport (ones not in rtg) are reported as
convective transport tendencies. Tendencies for variables in rtg are
reported as dcnv and scnv tendencies.

2. Report TKE tendencies from gfs v16 PBL.

3. Add diagnostic tendencies to drag_suite

4. Only report rayleigh damping tendencies if rayleigh damping is enabled

5. List all possible diagnostic tendencies, even if they are not
enabled. In the output, label them "selected" or "disabled."

6. Add phys_tend to the fv3_rap suite.

* Merge gsl/develop into ccpp/physics

* Add phys_tend to suite_FV3_GSD_SAR and remove some debug prints

* Remove some debug prints and commented-out code.

* Correct error in dtend variable availability list print.

* Enable convtrans diagnostic tendencies for number concentrations and qrimef.

* Disable cnvtrans diagnostic tendencies for schemes that do their own diagnostic tendency reporting.

* Point to gsl repositories

* Update .gitmodules for code review and regression testing

* Formatting changes in ccpp/data/GFS_typedefs.F90

* Update submodule pointer for ccpp-physics

* Update submodule pointer for ccpp-physics

* Point update_gsl_develop_from_develop_20210531 to NOAA-GSL branches of ccpp/physics and ccpp/framework

* GF aerosols updates and tunings

* GF aerosol updates and tunings from physics

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Update submodule pointer for ccpp-physics

* Update submodule pointer for ccpp-physics

* Update submodule pointer for ccpp-physics

* Make aod_gf and cactiv_m work appriopriately for restarts
Minor code cleanup

* Add missing metadata for new GF arrays

* Bug fix in ccpp/driver/GFS_restart.F90 for GF

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Add missing active attribute for aod_gf in ccpp/data/GFS_typedefs.meta

* Update submodule pointer for ccpp-physics

* Update submodule pointer for ccpp-physocs

* Change name of sfc_name2(39) from 'weasd' to 'weasdl' in io/FV3GFS_io.F90

* Revert change to .gitmodules and update submodule pointers for ccpp-framework and ccpp-physics

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Changed dimensions of variables tsnow_land and sncovr_ice from
horizontal_dimension to horizontal_loop_extent

* Added Flake to two suites: FV3_GSD_v0 and FV3_HRRR.

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Update to ccpp/data/GFS_typedefs.meta following CCPP standard names update

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointers for ccpp-physics and ccpp-framework

* Revert change to .gitmodules and update submodule pointers for ccpp-framework and ccpp-physics

* Turn on GF aerosol-awareness
Tune clwdet (cloud water detrainment)
Make evfact (evaporation factor) and radiation factor scale-aware

* Update .gitmodules and submodule pointers for ccpp-framework and ccpp-physics for code review and testing

* Switch back to NCAR ccpp-framework main, abandon NOAA-GSL fork

* Update submodule pointer for ccpp-physics

* Update submodule pointer for ccpp/physics

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Update .gitmodules and submodule pointer for ccpp-physics for code review aand testing

* Update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Update submodule pointer for ccpp-physics

* Revert .gitmodules and update submodule pointer for ccpp-physics

* Revert change to .gitmodules and update submodule pointer for ccpp-physics

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Revert .gitmodules and update submodule pointer for ccpp-physics

* Update CCPP metadata and several suite defiition files for new GSL regression tests

* Bug fix for compiling with Intel on macOS, remove clutter from stdout in io/FV3GFS_io.F90

* Update .gitmodules and submodule pointer for ccpp-physics for code review and testing

* Rename GSL suites to RAP suites, add new RAP suites

* rename suite FV3_GSD_v0 to FV3_RAP_noah_sfcdiff_cires_ugwp, delete suite FV3_SAR

* Update submodule pointer for ccpp-physics

* Address reviewer comments

* Update submodule pointer for ccpp-physics

* Revert .gitmodules and update submodule pointer for ccpp-physics

Co-authored-by: DomHeinzeller <58610420+DomHeinzeller@users.noreply.github.com>
Co-authored-by: Phil Pegion <38869668+pjpegion@users.noreply.github.com>
Co-authored-by: shansun6 <48043606+shansun6@users.noreply.github.com>
Co-authored-by: tanyasmirnova <tanya.smirnova@noaa.gov>
Co-authored-by: tanyasmirnova <38667904+tanyasmirnova@users.noreply.github.com>
Co-authored-by: Mark Potts <Mark.Potts@noaa.gov>
Co-authored-by: Shrinivas.Moorthi <Shrinivas.Moorthi@noaa.gov>
Co-authored-by: dustinswales <dustin.swales@noaa.gov>
Co-authored-by: Jun Wang <37633869+junwang-noaa@users.noreply.github.com>
Co-authored-by: XiaqiongZhou-NOAA <48254930+XiaqiongZhou-NOAA@users.noreply.github.com>
Co-authored-by: valery.yudin <valery.yudin@noaa.gov>
Co-authored-by: Michael Toy <michael.toy@noaa.gov>
Co-authored-by: XiaSun-NOAA <58949533+XiaSun-NOAA@users.noreply.github.com>
Co-authored-by: samuel.trahan <Samuel.Trahan@noaa.gov>
Co-authored-by: Samuel Trahan (NOAA contractor) <39415369+SamuelTrahanNOAA@users.noreply.github.com>
Co-authored-by: hannah barnes <hannah.barnes@noaa.gov>
Co-authored-by: Hannah C Barnes <38660891+hannahcbarnes@users.noreply.github.com>
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

No branches or pull requests

2 participants