Update input data consistent with P6#537
Conversation
DeniseWorthen
added
Baseline Updates
|
Machine: jet |
|
There appears to be a problem with the new input data directory. Some files are missing, some directories I can't |
|
There is not enough space to rsync the input directory from my local directory (where I transferred from Globus) to the baseline area. I've clean out (removed the rt_*) in @binli2337 please delete what you can. |
I deleted my files. |
|
Machine: orion |
|
Still waiting for the two failed orion jobs. Since the restart jobs were the ones that failed, this will actually require 4 jobs to complete. |
|
At this point since we think it's just a system problem and not a code problem, we should just move on without those two tests? |
* repeated two tests after initial failure; both passed * cpld_restart_bmark, cpld_restart_bmarkfrac_v16
|
All RTs are complete and pass. My FV3 branch now points to updated dycore (3720aea). After fv3 is merged I will revert my submodules and run-ci. |
|
FV3 is merged
…On Thu, May 13, 2021 at 12:54 PM Denise Worthen ***@***.***> wrote:
All RTs are complete and pass. My FV3 branch now points to updated dycore
(3720aea). After fv3 is merged I will revert my submodules and run-ci.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#537 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TN6SFW2RKXLSRKF6N3TNP733ANCNFSM43IZFLJA>
.
|
climbfuji
left a comment
climbfuji
left a comment
There was a problem hiding this comment.
fv3atm has is correct, changes look good
* WE2E test for MET validation. * Add a header to the config file. * Update config.MET_validation.sh * Change 'validation' to 'verification' Co-authored-by: JeffBeck-NOAA <55201531+JeffBeck-NOAA@users.noreply.github.com>
7 participants
PR Checklist
Ths 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. Please consult the ufs-weather-model wiki if you are unsure how to do this.
This PR has been tested using a branch which is up-to-date with the top of all sub-component repositories except for those sub-components which are the subject of this PR
An Issue describing the work contained in this PR has been created either in the subcomponent(s) or in the ufs-weather-model. The Issue should be created in the repository that is most relevant to the changes in contained in the PR. The Issue and the dependent sub-component PR
are specified below.
If new or updated input data is required by this PR, it is clearly stated in the text of the PR.
Instructions: All subsequent sections of text should be filled in as appropriate.
The information provided below allows the code managers to understand the changes relevant to this PR, whether those changes are in the ufs-weather-model repository or in a subcomponent repository. Ufs-weather-model code managers will use the information provided to add any applicable labels, assign reviewers and place it in the Commit Queue. Once the PR is in the Commit Queue, it is the PR owner's responsiblity to keep the PR up-to-date with the develop branch of ufs-weather-model.
Description
Updates
FV3_input_fracoro data and IC for full lakes only, consistent with input used for P6: Source directory:/scratch2/BMC/gsd-fv3-dev/FV3-MOM6-CICE5/inputdata_lakeint_p6/FV3_input_fracSets
nst_anl=.truefor benchmark v16 testsSwitches to using a mesh for MOM6 in the cap (use_mommesh is set
true). See original issue #398adds 2 additional suites: suite_FV3_GFS_v16_coupled_noahmp.xml, suite_FV3_GFS_v16_coupled_nsstNoahmp.xml
A new
input-data-20210518has been staged on on platforms. This was created by first rsync'ing the currentinput-data-20210428and then updating the files in theFV3_input_fracand themesh.mx100.ncfiles in theCICE_FIXdirectories.Issue(s) addressed
Link the issues to be closed with this PR, whether in this repository, or in another repository.
(Remember, issues must always be created before starting work on a PR branch!)
Related issue #564. When
na_init=1, the bottom fields exported by ATM are expected to depend on dt. The fact that changing the timestep to 300s in P6 appeared to fix many of the cases which failed on startup is most likely due to chance. When dt was changed, the change in the bottom fields happened to remove the negative zbot which was sent to ICE.Testing
How were these changes tested? What compilers / HPCs was it tested with? Are the changes covered by regression tests? (If not, why? Do new tests need to be added?) Have regression tests and unit tests (utests) been run? On which platforms and with which compilers? (Note that unit tests can only be run on tier-1 platforms)
Pre-commit testing on Gaea: a temporary baseline was created using the updated data and then verified against that baseline. All coupled and DATM tests passed. Standalone ATM tests were not run.
Dependencies
GFDL_atmos_cubed_sphere PR #102
fv3atm PR#294
co-author: @shansun6
co-author: @XiaqiongZhou-NOAA