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

CLM Lake pressure is not density and GFS Diagnostics: weasdi is not snodi + #2047, #2053, and #2056 #2044

Merged
merged 41 commits into from
Dec 21, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
8635abd
clm lake unit fix & wrong variable sent for snodi
SamuelTrahanNOAA Dec 15, 2023
21ee461
point to sam's branch
SamuelTrahanNOAA Dec 15, 2023
7d2ce38
hera tests passed
SamuelTrahanNOAA Dec 15, 2023
a1dba0a
revisions to slowfall accumulation from @tanyasmirnova
SamuelTrahanNOAA Dec 18, 2023
0ec8f84
hera tests passed
SamuelTrahanNOAA Dec 18, 2023
d1ece2c
point to sam's branch
SamuelTrahanNOAA Dec 18, 2023
3747544
clm_lake.f90: explain lakedepth corruption safeguards
SamuelTrahanNOAA Dec 18, 2023
ea272d2
ccpp-physics module_sf_ruclsm.f90: explain the snow_mosaic=0 line
SamuelTrahanNOAA Dec 18, 2023
10f30e0
point FV3 to personal fork
JiliDong-NOAA Dec 18, 2023
256322f
update FV3
JiliDong-NOAA Dec 18, 2023
5173503
point CMEPS to personal fork
JiliDong-NOAA Dec 19, 2023
315f27d
Initial Updates to template
BrianCurtis-NOAA Dec 19, 2023
d2e1e08
Fix type for checkboxes and add instructions section to pr template
BrianCurtis-NOAA Dec 19, 2023
243690f
Cheyenne->Derecho, update GNU machines in template
BrianCurtis-NOAA Dec 19, 2023
29c0fa0
update FV3
JiliDong-NOAA Dec 19, 2023
8945208
change name of lake_freeze to flag_lakefreeze
JiliDong-NOAA Dec 19, 2023
803e54a
fv3 updated
JiliDong-NOAA Dec 19, 2023
0ca12c0
Updates taking into account most suggestions up to this point
BrianCurtis-NOAA Dec 19, 2023
4779ed4
More changes from PR discussion
BrianCurtis-NOAA Dec 19, 2023
24236dc
merge #2047
SamuelTrahanNOAA Dec 19, 2023
b1b6484
merge develop
SamuelTrahanNOAA Dec 19, 2023
b0b76e3
merge fv3atm#742 into FV3
SamuelTrahanNOAA Dec 19, 2023
c330e79
merge #2056
SamuelTrahanNOAA Dec 19, 2023
d7f81b7
CCPP: fix mistake in merge: extra rho0 argument
SamuelTrahanNOAA Dec 20, 2023
127f6c9
hera tests passed
SamuelTrahanNOAA Dec 20, 2023
1cf0edd
Update bl_date.conf
jkbk2004 Dec 20, 2023
a6180c4
Merge branch 'ufs-community:develop' into template_tuneup
BrianCurtis-NOAA Dec 20, 2023
55f22fa
WCOSS2 RT Log
BrianCurtis-NOAA Dec 20, 2023
b85c396
add gaea c5 RT log: passed
jkbk2004 Dec 20, 2023
11f969d
add derecho RT log: passed
jkbk2004 Dec 20, 2023
fa96215
add orion RT logs: passed
zach1221 Dec 21, 2023
97c5d7a
add hera RT log: passed
jkbk2004 Dec 21, 2023
69769cd
add hercules RT logs: passed
zach1221 Dec 21, 2023
f1a9177
Update rt.conf enable wrtGauss cases for hercules
zach1221 Dec 21, 2023
e587d29
Update fv3_slurm.IN_hercules export additional mpi variable
zach1221 Dec 21, 2023
5479e0f
add jet RT log: passed
jkbk2004 Dec 21, 2023
d64d98d
Acorn RT Log
BrianCurtis-NOAA Dec 21, 2023
4391a46
add gaea c4 RT log: passed
jkbk2004 Dec 21, 2023
abf6360
Merge remote-tracking branch 'upstream/template_tuneup' into clm-unit…
jkbk2004 Dec 21, 2023
626dc93
point FV3 to NOAA-EMC/fv3atm
SamuelTrahanNOAA Dec 21, 2023
f91ed44
point to NOAA-EMC emc/develop for CMEPS-interface/CMEPS
SamuelTrahanNOAA Dec 21, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
82 changes: 48 additions & 34 deletions .github/pull_request_template.md
Original file line number Diff line number Diff line change
@@ -1,32 +1,56 @@
<!-- 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/Hercules 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
<!-- INSTRUCTIONS:
- Please fill out all sections of this PR and complete the checklist below
- Please be as descriptive as possible, this is really important.
- Please "fill in" checkboxes. Use [X] for a filled in checkbox or leave it [ ] for an empty checkbox
- Please use github markup as much as possible in linking
i.e.:
* Linking to UFSWM PR's and issues add - #<pr/issue number>
* Linking to a subcomponent PR and issues add - <Group>/<Fork>/pull/<number> or - <Group>/<Fork>/issues/<number>
-->
## Commit Queue Requirements:
<!-- Please note: PRs will be scheduled in the Commit Queue in the order received and only after all
pre-requisite testing is complete and all PR requirements (e.g. Issues created and noted, Subcomponent
PRs reviewed and accepted) are met. -->
- [ ] Fill out all sections of this template.
- [ ] All sub component pull requests have been reviewed by their code managers.
- [ ] Run the full RT suite (compared to current baselines) on either Hera/Derecho/Hercules AND have committed the log to my PR branch.
- [ ] Add list of all failed regression tests in "Regression Tests" section.

## PR Information

### Description
<!-- Provide a detailed description of what this PR does in the space provided below-->

### Commit Message
<!--
Please provide the following concise information:
Description of all changes - 1 line
Please list all individual issue titles addressed with github links at the end in parenthesis (using #<number> or <group>/<fork>/issues/<number>).
-->

## Linked Issues and Pull Requests
### Associated UFSWM Issue to close
<!-- Example: "- Closes #1698" -->

### Priority
- [ ] Critical Bugfix (This PR contains a critical bug fix and should be prioritized.)
- [ ] High (This PR contains a feature or fix needed for a time-sensitive project (eg, retrospectives, implementations))
- [ ] Normal

### Subcomponent Pull Requests
<!-- format: - <community>/<repo>/pull/<PR number> i.e.: - NOAA-EMC/fv3atm/pull/33 or "None" -->
### Blocking Dependencies
<!-- If there are any PR's that are needed to be completed before this one, please add links
to them here -->

### Git Issues Fixed By This PR
<!-- Example: - Closes #1698 or - Closes NOAA-EMC/fv3atm/issues/729 -->

### Blocking Dependencies
<!-- Example: "- Depends on #1733" or "None" -->

## Changes

### Subcomponents involved:
### Subcomponent (with links)
<!-- (add links to subcomponent PR's here) -->
<!-- Example:
[X] FV3
- NOAA-EMC/fv3atm/pull/734
- NOAA-EMC/fv3atm/pull/735
-->
- [ ] AQM
- [ ] CDEPS
- [ ] CICE
Expand All @@ -41,7 +65,6 @@ PLEASE DO NOT MODIFY THE TEMPLATE BEYOND FILLING OUT THE PROPER SECTIONS -->
- [ ] stochastic_physics
- [ ] none

## Anticipated Changes
### Input data
- [ ] No changes are expected to input data.
- [ ] Changes are expected to input data:
Expand All @@ -51,9 +74,8 @@ PLEASE DO NOT MODIFY THE TEMPLATE BEYOND FILLING OUT THE PROPER SECTIONS -->
### 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><summary>FAILED REGRESSION TESTS</summary>
<!-- List failed regression tests here or add "None" -->

</details>

Expand All @@ -64,14 +86,7 @@ PLEASE DO NOT MODIFY THE TEMPLATE BEYOND FILLING OUT THE PROPER SECTIONS -->
- [ ] 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

<!-- STOP!!! THE FOLLOWING IS FOR CODE MANAGERS ONLY. PLEASE DO NOT FILL OUT -->
### Testing Log:
- RDHPCS
- [ ] Hera
Expand All @@ -87,5 +102,4 @@ PLEASE DO NOT MODIFY THE TEMPLATE BEYOND FILLING OUT THE PROPER SECTIONS -->
- [ ] Completed
- opnReqTest
- [ ] N/A
- [ ] Log attached to comment
</details>
- [ ] Log attached to comment
2 changes: 1 addition & 1 deletion CMEPS-interface/CMEPS
2 changes: 1 addition & 1 deletion FV3
2 changes: 1 addition & 1 deletion tests/bl_date.conf
Original file line number Diff line number Diff line change
@@ -1 +1 @@
export BL_DATE=20231207
export BL_DATE=20231220
6 changes: 4 additions & 2 deletions tests/fv3_conf/fv3_slurm.IN_hercules
Original file line number Diff line number Diff line change
Expand Up @@ -36,8 +36,10 @@ export OMP_NUM_THREADS=@[THRD]
export ESMF_RUNTIME_PROFILE=ON
export ESMF_RUNTIME_PROFILE_OUTPUT="SUMMARY"

# For mvapich2
if [[ @[RT_COMPILER] == gnu ]]; then
if [[ @[RT_COMPILER] == intel ]]; then
export I_MPI_EXTRA_FILESYSTEM=ON
elif [[ @[RT_COMPILER] == gnu ]]; then
# For mvapich2
export MV2_SHMEM_COLL_NUM_COMM=128
fi

Expand Down
Loading