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

sync of dev/gfdl with master #130

Merged
merged 49 commits into from
Sep 9, 2021
Merged

Conversation

bensonr
Copy link
Contributor

@bensonr bensonr commented Aug 17, 2021

Description
Synchronize the latest public release in master with dev/gfdl branch. This update fixes nesting bugs, introduces regional capabilities, as well as advanced nesting capabilities (multiple and telescoping nests).

Includes a fix for #134

How Has This Been Tested?
This release has been extensively tested in the SHiELD models and with the UFS. Prior to merge, tests will be completed with GFDL models.

Checklist:

Please check all whether they apply or not

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules

laurenchilutti and others added 30 commits March 5, 2021 14:38
…cing the necessary functions with fms2_io_mod functions
…name when needed and removing unneccessary code in fv_io_mod
…ter_axis in fv_ada_nudge. Reintroducing flush function and updating namelist read in fv_diag_column
Adding Issue Templates to Repository
Dev/gfdl updates being brought back to master
Formatted PDF and source files for FV3 documentation.
Updated defaults for hord options to use 8 and 10, and removal of mention of hord = 9, which is now a different (experimental, unsupported) scheme.
Adding Code Style guide to the repository.
*Corrected* update to FV3 documentation v1.0a
Documentation and defaults changes
DusanJovic-NOAA and others added 16 commits June 25, 2021 16:13
…ion (NOAA-GFDL#114)

* merge of latest dev work from GFDL Weather and Climate Dynamics Division

* remove trailing whitespace from source files

* release notes updates in RELEASE.md

* Remove blocks of debug code and commented code
* read ak/bk from user specified files

* add input.nml parameter fv_eta_file for user specified ak/bk; change ks calculation when npz_type=input; use newunit to replace fixed file unit for npz_type=input
(cherry picked from commit 3a0d35a)

Conflicts:
	model/fv_control.F90
	tools/fv_eta.F90
* Create examples README.md

* Reorganizing files in docs/ directory

* Added advection notebook

* Fixed subplot spacing

* New 3D case notebooks

* Added notebooks

* Update README.md
* add check on eta levels to ensure their monotonicity

* update Jili Dong's ak/bk external input to
  - use the FMS ascii_read (single read/broadcast)
  - error check input to ensure the proper number of levels present

* added a format description for the external eta file and ensured a correct the file length check for FMS 2021.03 and greater
…n (20210804) (NOAA-GFDL#127)

* output conversion error warning in debug mode w/ Intel
* incorrect argument to mpp_max in fv_restart.F90 affecting nesting
@bensonr bensonr changed the title Master2gfdl sync sync of dev/gfdl with master Aug 17, 2021
Copy link
Contributor

@lharris4 lharris4 left a comment

Choose a reason for hiding this comment

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

This looks good. Glad to see a lot of the old junk getting cleaned out.

@@ -878,6 +879,11 @@ module fv_arrays_mod

integer :: bc_update_interval = 3 !< Default setting for interval (hours) between external regional BC data files.

integer :: nrows_blend = 0 !< # of blending rows in the outer integration domain.
Copy link
Contributor

Choose a reason for hiding this comment

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

These updates allow the climate models to be run as RCMs.

@@ -203,16 +206,12 @@ subroutine get_external_ic( Atm, fv_domain, cold_start )
call prt_maxmin('graupel', Atm%q(:,:,:,graupel), is, ie, js, je, ng, Atm%npz, 1.)
if ( o3mr > 0 ) &
call prt_maxmin('O3MR', Atm%q(:,:,:,o3mr), is, ie, js, je, ng, Atm%npz, 1.)
if ( sgs_tke > 0 ) &
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe later we should just cycle through all the tracer species instead of hard-coding each one.

tools/fv_grid_tools.F90 Show resolved Hide resolved
tools/fv_io.F90 Show resolved Hide resolved
@bensonr bensonr merged commit a032380 into NOAA-GFDL:dev/gfdl Sep 9, 2021
@bensonr bensonr deleted the master2gfdl_sync branch September 10, 2021 19:05
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.

FV3 runs fine in debug mode (-O0) , but gives forrtl: error (63): output conversion error, unit -1
6 participants