-
Notifications
You must be signed in to change notification settings - Fork 254
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
Update to esmf-bs47 across all platforms; add export of openwater_frac_in_atm from FV3 when coupled #493
Update to esmf-bs47 across all platforms; add export of openwater_frac_in_atm from FV3 when coupled #493
Conversation
*merge changes required for export of Sa_ofrac from FV3
* use branch with addition of field to fd_nems.yaml
@junwang-noaa esmf/8.1.0bs47 does not seem to be available on wcoss-cray I tried:
|
Hang installed it at NCEPLIBS location:
The esmf 810bs47 has been added to the NCEPLIBS-v1.3.0 on wcoss-cray.
module load
/gpfs/hps/usrx/local/nceplibs/NCEPLIBS/cmake/install/NCEPLIBS-v1.3.0/modules/esmf/810bs47
…On Tue, Mar 30, 2021 at 3:52 PM Denise Worthen ***@***.***> wrote:
@junwang-noaa <https://github.com/junwang-noaa> esmf/8.1.0bs47 does not
seem to be available on wcoss-cray
I tried:
module use /gpfs/hps3/emc/nems/noscrub/emc.nemspara/soft/modulefiles
module avail
[image: Screen Shot 2021-03-30 at 3 50 57 PM]
<https://user-images.githubusercontent.com/40498404/113047833-ce17dd80-916f-11eb-960a-aa1dac1c0e94.png>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#493 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TNLZKFZ5GYFN3ZDRZ3TGITWVANCNFSM4Z7PZQ4Q>
.
|
Thanks. It looks like it is compiling now. |
@MinsukJi-NOAA Testing this branch on wcoss-cray gave failures for the alt compare, for example:
Can you take a look? My run directory is /gpfs/hps3/emc/marine/noscrub/Denise.Worthen/ufs_bs47 |
ALT CHECK kicks in when CMP fails. Taking a look at your log file, there are 44 failed test cases of phyf000.tile.1.nc.
|
Thanks Minsuk. I was expecting that the forecast files would need alt check but I was surprised it turned up as "error". The comparison doesn't continue past that so the tests are flagged as failing. |
I reran the wcoss-cray tests with the forecast files moved to the end of the compare list. All non-forecast files compare "OK" against the current baseline. |
Merged into PR #475; closing. |
…add export of openwater_frac_in_atm from fv3ATM when coupled (#475) * update CICE in ufs-weather for the latest emc/develop * fix ice_in_template; sw_redist has been moved to shortwave_nml and removed two unused settings * update fv3ATM to use esmf-bs47 and add the export of open-water fraction from fv3ATM * update module files to use bs47 (PR #493) * fix wcoss-cray for alt-compare functionality * update CMEPS and NEMS for added export field * update Dockerfile
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
Issue(s) addressed
Testing
ESMF changes: Testing on hera.intel and wcoss-dell-p3 show comparison failures for these tests and files (all other files pass either alt or direct comparison):
fv3_regional_quilt: NATLEV.GrbF00 and NATLEV.GrbF24
fv3_esg_HAFS_v0_hwrf_thompson and fv3_esg_HAFS_v0_hwrf_thompson_debug : phyf000 and dynf000
On dell-p3, some tests failed because of compile failures for 016 and 017 (job killed after reaching LSF run time limit)
Dependencies
CMEPS #39
FV3-ATM #266
This PR is to be combined with PR #475