UFS-dev PR#111

.github/pull_request_template.md

@@ -1,20 +1,32 @@
+<!-- THE FOLLOWING IS FOR THE PR AUTHOR TO FILL OUT
+PLEASE DO NOT MODIFY THE TEMPLATE BEYOND FILLING OUT THE PROPER SECTIONS -->
+## PR Author Checklist:
+<!--  Please complete all items in list. -->
+- [ ] I have linked PR's from all sub-components involved in section below. <!-- PLEASE DO NOT LINK SUBCOMPONENT ISSUES -->
+- [ ] I am confirming reviews are completed in ALL sub-component PR's.
+- [ ] I have run the full RT suite on either Hera/Cheyenne AND have attached the log to this PR below this line:
+  - LOG: 
+- [ ] I have added the list of all failed regression tests to "Anticipated changes" section.
+- [ ] I have filled out all sections of the template.
+
 ## Description
-<!--
-Provide a detailed description of what this PR does. What bug does it fix, or what feature does it add? Is a change of answers expected from this PR? Are any library updates included in this PR (modulefiles etc.)?
--->
+<!-- Provide a detailed description of what this PR does in the space provided below-->
 
-### Input data additions/changes
-- [ ] No changes are expected to input data.
-- [ ] Changes are expected to input data:
-  - [ ] New input data.
-  - [ ] Updated input data.
 
-### Anticipated changes to regression tests:
-- [ ] No changes are expected to any regression test.
-- [ ] Changes are expected to the following tests:
-<!-- Please insert what RT's change and why you expect them to change -->
+## Linked Issues and Pull Requests
+### Associated UFSWM Issue to close
+<!-- Example: "- Closes #1698" -->
 
-## Subcomponents involved:
+
+### Subcomponent Pull Requests
+<!-- format: - <community>/<repo>/pull/<PR number> i.e.: - NOAA-EMC/fv3atm/pull/33 or "None" -->
+
+
+### Blocking Dependencies
+<!-- Example: "- Depends on #1733" or "None" -->
+
+
+### Subcomponents involved:
 - [ ] AQM
 - [ ] CDEPS
 - [ ] CICE
@@ -29,47 +41,42 @@ Provide a detailed description of what this PR does. What bug does it fix, or wh
 - [ ] stochastic_physics
 - [ ] none
 
-### Library Updates/Changes
+## Anticipated Changes
+### Input data
+- [ ] No changes are expected to input data.
+- [ ] Changes are expected to input data:
+  - [ ] New input data.
+  - [ ] Updated input data.
+
+### Regression Tests:
+- [ ] No changes are expected to any regression test.
+- [ ] Changes are expected to the following tests:
+<!-- Please insert what RT's change and why you expect them to change in the space provided below -->
+<details><summary>Tests effected by changes in this PR:</summary>
+<!-- ADD ITEMS HERE or add "None" -->
+
+</details>
+
+### Libraries
 <!-- Library updates take time. If this PR needs updates to libraries, please make sure to accomplish the following tasks -->
 - [ ] Not Needed
-- [ ] Create separate issue in [JCSDA/spack-stack](https://github.com/JCSDA/spack-stack) asking for update to library. Include library name, library version.
-- [ ] Add issue link from JCSDA/spack-stack following this item
-<!-- for example: "- JCSDA/spack-stack/issue/1757" -->
-
-### Combined with PR's (If Applicable):
-
-## Commit Queue Checklist:
-<!-- 
-Please complete all items in list. Make sure to attach logs from RT testing in comment, not in repository. Once all boxes are checked, please add the label "Ready for Commit Queue".
--->
-- [ ] Link PR's from all sub-components involved in section below
-- [ ] Confirm reviews completed in ALL sub-component PR's
-- [ ] Add all appropriate labels to this PR.
-- [ ] Run full RT suite on either Hera/Cheyenne AND attach log to a PR comment.
-- [ ] Add list of any failed regression tests to "Anticipated changes to regression tests" section.
-
-## Linked PR's and Issues:
-<!--
-Please link dependent pull requests.
-EXAMPLE: "- Depends on NOAA-EMC/fv3atm/pull/<pullrequest_number>"
-
-Please link the related issues to be closed with this PR, whether in this repository, or in another repository.
-EXAMPLE: "- Closes NOAA-EMC/fv3atm/issues/<issue_number>"
-
-PLEASE MAKE SURE TO USE THE - with a space before the "Depends on" or "Closes" as they show up well on github.
--->
-
-## Testing Day Checklist:
-<!--
-Please consult the ufs-weather-model [wiki](https://github.com/ufs-community/ufs-weather-model/wiki/Making-code-changes-in-the-UFS-weather-model-and-its-subcomponents) if you are unsure how to do this.
--->
+- [ ] Needed
+  - [ ] Create separate issue in [JCSDA/spack-stack](https://github.com/JCSDA/spack-stack) asking for update to library. Include library name, library version.
+  - [ ] Add issue link from JCSDA/spack-stack following this item <!-- for example: "- JCSDA/spack-stack/issue/1757" -->
+
+
+<!-- THE FOLLOWING IS FOR CODE MANAGERS ONLY DO NOT FILL OUT -->
+<details><summary>Code Managers Log</summary>
+
 - [ ] This PR is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR.
 - [ ] Move new/updated input data on RDHPCS Hera and propagate input data changes to all supported systems.
+  - [ ] N/A
 
-### Testing Log (for CM's):
+### Testing Log:
 - RDHPCS
   - [ ] Hera
   - [ ] Orion
+  - [ ] Hercules
   - [ ] Jet
   - [ ] Gaea
   - [ ] Cheyenne
@@ -81,3 +88,4 @@ Please consult the ufs-weather-model [wiki](https://github.com/ufs-community/ufs
 - opnReqTest
   - [ ] N/A
   - [ ] Log attached to comment
+</details>

.gitmodules

@@ -1,7 +1,9 @@
 [submodule "FV3"]
   path = FV3
-  url = https://github.com/NCAR/fv3atm
-  branch = main
+  #url = https://github.com/NCAR/fv3atm
+  #branch = main
+  url = https://github.com/grantfirl/fv3atm
+  branch = ufs-dev-PR111
 [submodule "WW3"]
   path = WW3
   url = https://github.com/NOAA-EMC/WW3

.readthedocs.yaml

@@ -4,6 +4,10 @@
 
 # Required
 version: 2
+build: 
+  os: ubuntu-20.04
+  tools: 
+    python: "3.9"
 
 # Build documentation in the docs/ directory with Sphinx
 sphinx:
@@ -18,6 +22,5 @@ formats: all
 
 # Optionally set the version of Python and requirements required to build your docs
 python:
-  version: 3.7
   install:
     - requirements: doc/UsersGuide/requirements.txt

CDEPS-interface/CMakeLists.txt

@@ -1,6 +1,8 @@
 # CDEPS compiler flags
 if(CMAKE_Fortran_COMPILER_ID MATCHES "GNU")
   set(CMAKE_Fortran_FLAGS "-g -fbacktrace -ffree-line-length-none")
+  #Set CPP defintions for the ufs/cdeps_share target library
+  list(APPEND CDEPS_SHARE_DEFS "CPRGNU")
   if(${CMAKE_Fortran_COMPILER_VERSION} VERSION_GREATER_EQUAL 10)
     set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -fallow-argument-mismatch -fallow-invalid-boz")
   endif()
@@ -11,6 +13,8 @@ elseif(CMAKE_Fortran_COMPILER_ID MATCHES "Intel")
   set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -O -assume realloc_lhs")
   set(CMAKE_Fortran_FLAGS_RELEASE "-O2 -fp-model precise")
   set(CMAKE_Fortran_FLAGS_DEBUG "-O0 -check -check noarg_temp_created -check nopointer -fpe0 -ftrapuv -init=snan,arrays")
+  #Set CPP defintions for the ufs/cdeps_share target library
+  list(APPEND CDEPS_SHARE_DEFS "CPRINTEL")
 else()
   message(WARNING "Fortran compiler with ID ${CMAKE_Fortran_COMPILER_ID} will be used with CMake default options")
 endif()
@@ -23,6 +27,7 @@ add_library(share OBJECT ${cdeps_share_files} ${ufs_cdeps_share_files})
 set_target_properties(share PROPERTIES Fortran_MODULE_DIRECTORY ${CMAKE_CURRENT_BINARY_DIR}/mod)
 target_include_directories(share PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/ufs/cdeps_share)
 target_include_directories(share PRIVATE $<BUILD_INTERFACE:${CMAKE_CURRENT_BINARY_DIR}/mod>)
+target_compile_definitions(share PRIVATE ${CDEPS_SHARE_DEFS})
 target_link_libraries(share PRIVATE esmf)
 
 # CDEPS/streams