Use v2.0alpha01 tag of https://github.com/JeffBeck-NOAA/UFS_UTILS #29
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tested out-of-the-box case on Cheyenne, runs to completion.
JulieSchramm
requested review from
gsketefian,
JeffBeck-NOAA and
mkavulich
as code owners
|
Julie - Could you also add the update to use the release branch of regional_workflow? Might as well do these together, and since it's the same hash (for develop and release/public-v1) shouldn't make any actual difference at this point. |
|
@JulieSchramm Did you run any tests? I've been finding that with the new UFS_UTILS changes, bug fixes sometimes introduce new bugs. |
|
@gerard Ketefian - NOAA Affiliate <gerard.ketefian@noaa.gov> I ran the
out-of-the-box case on Cheyenne to test the v2.0alpha01 tag, no WE2E
tests. I didn't run any tests with the release/public-v1 branch of
regional_workflow since it is identical to develop at this point.
…On Tue, Oct 6, 2020 at 8:52 AM gsketefian ***@***.***> wrote:
@JulieSchramm <https://github.com/JulieSchramm> Did you run any tests?
I've been finding that with the new UFS_UTILS changes, bug fixes sometimes
introduce new bugs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#29 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3WNU5TL3GZOJZ5GDEB67TSJMVLTANCNFSM4SE65QUA>
.
|
|
@JulieSchramm Probably one test is not enough at this point. I suggest running those WE2E tests for the capabilities the release is going to support. We can still merge the PR after running the tests, but we should at least get a status report on what capabilities currently work/fail. |
|
@gerard Ketefian - NOAA Affiliate <gerard.ketefian@noaa.gov> Should I test
on Hera or Cheyenne?
…On Tue, Oct 6, 2020 at 9:37 AM gsketefian ***@***.***> wrote:
@JulieSchramm <https://github.com/JulieSchramm> Probably one test is not
enough at this point. I suggest running those WE2E tests for the
capabilities the release is going to support. We can still merge the PR
after running the tests, but we should at least get a status report on what
capabilities currently work/fail.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#29 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3WNUYUOF6QZBX54APWCFDSJM2TTANCNFSM4SE65QUA>
.
|
|
@JulieSchramm Let's start with hera. |
|
hera is down for maintenance today!
…On Tue, Oct 6, 2020 at 10:27 AM gsketefian ***@***.***> wrote:
@JulieSchramm <https://github.com/JulieSchramm> Let's start with hera.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#29 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFN5ZDH2LZU3FTITY6CNSJTSJNAQXANCNFSM4SE65QUA>
.
--
Jamie K. Wolff
National Center for Atmospheric Research (NCAR)
Research Applications Laboratory (RAL)
Phone: 303.497.2812 Email: jwolff@ucar.edu
*My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.*
|
|
Oh right! That's why I'm working on Orion today. Cheyenne it is then. @JulieSchramm Let me know if you need help running the WE2E tests on cheyenne. @jwolff-ncar |
|
I just started the run_experiments.sh on Cheyenne, running all of the
tests. I'm expecting some to fail.
…On Tue, Oct 6, 2020 at 10:36 AM gsketefian ***@***.***> wrote:
Oh right! That's why I'm working on Orion today. Cheyenne it is then.
@JulieSchramm <https://github.com/JulieSchramm> Let me know if you need
help running the WE2E tests on cheyenne. @jwolff-ncar
<https://github.com/jwolff-ncar>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#29 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AA3WNU2PTANIENEBGBTXO3TSJNBPTANCNFSM4SE65QUA>
.
|
|
The tests took a while to run on Cheyenne. Here is the summary:
community_ensemble_008mems FAILURE
community_ensemble_2mems FAILURE
DOT_OR_USCORE SUCCESS
get_extrn_files_from_hpss_FV3GFS_FV3GFS FAILURE
get_extrn_files_from_hpss_GSMGFS_GSMGFS FAILURE
get_extrn_files_from_hpss_HRRRX_RAPX FAILURE
get_extrn_files_from_hpss_RAPX_RAPX FAILURE
grid_GSD_HRRR_AK_50km FAILURE
grid_GSD_RAP13km FAILURE
new_ESGgrid SUCCESS
new_GFDLgrid__GFDLgrid_USE_GFDLgrid_RES_IN_FILENAMES_eq_FALSE SUCCESS
new_GFDLgrid__GFDLgrid_USE_GFDLgrid_RES_IN_FILENAMES_eq_TRUE SUCCESS
new_GFDLgrid SUCCESS
pregen_grid_orog_sfc_climo SUCCESS
regional_001 SUCCESS
regional_002 SUCCESS
suite_FV3_CPT_v0 FAILURE
suite_FV3_GFS_v15p2 SUCCESS
suite_FV3_GFS_v16beta SUCCESS
suite_FV3_GSD_SAR FAILURE
suite_FV3_GSD_v0 FAILURE
suite_FV3_RRFS_v1beta FAILURE
…On Tue, Oct 6, 2020 at 10:37 AM Julie Schramm ***@***.***> wrote:
I just started the run_experiments.sh on Cheyenne, running all of the
tests. I'm expecting some to fail.
On Tue, Oct 6, 2020 at 10:36 AM gsketefian ***@***.***>
wrote:
> Oh right! That's why I'm working on Orion today. Cheyenne it is then.
> @JulieSchramm <https://github.com/JulieSchramm> Let me know if you need
> help running the WE2E tests on cheyenne. @jwolff-ncar
> <https://github.com/jwolff-ncar>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#29 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AA3WNU2PTANIENEBGBTXO3TSJNBPTANCNFSM4SE65QUA>
> .
>
|
|
Running the WE2E tests on Cheyenne uses 3541 core hours or 8% of the allocation, from what I could tell. |
|
Jeff's tag is in Externals.cfg, not sure when we should use release/public-v1 of regional_workflow. |
christinaholtNOAA
pushed a commit
to christinaholtNOAA/ufs-srweather-app
that referenced
this pull request
May 20, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DESCRIPTION OF CHANGES:
Change Externals.cfg to point to tag v2.0alpha01 of https://github.com/JeffBeck-NOAA/UFS_UTILS
TESTS CONDUCTED:
Tested out-of-the-box case on Cheyenne, runs to completion. Results look reasonable.