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

Hash of UFS Weather Model is updated #176

Merged
merged 4 commits into from
Jan 15, 2025

Conversation

chan-hoo
Copy link
Collaborator

@chan-hoo chan-hoo commented Jan 9, 2025

Description

The hash of the ufs weather model is updated with the latest one as of 01/07/25.

Anticipated changes to regression tests:

  • Is baseline change expected ?

Subcomponents involved:

  • apply_incr.fd (NOAA-PSL/land-apply_jedi_incr)
  • ufs_model.fd (ufs-community/ufs-weather-model)
  • UFS_UTILS.fd (ufs-community/UFS_UTILS)
  • none

Linked PR's and Issues:

Resolves Issue #175

Testing (for CM's):

  • RDHPCS
    • Hera
    • Orion
    • Hercules
  • CI
    • Completed
  • WE2E
    • Completed
  • PW-Clouds
    • AWS
    • AZURE
    • GCP

@chan-hoo chan-hoo linked an issue Jan 9, 2025 that may be closed by this pull request
@chan-hoo chan-hoo requested a review from jkbk2004 January 9, 2025 15:04
@chan-hoo
Copy link
Collaborator Author

chan-hoo commented Jan 9, 2025

On Hercules:

+ ctest
Test project /work/noaa/epic/chjeon/landda_hash_hercules/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed    8.80 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed   28.93 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    2.35 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   29.80 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) =  69.93 sec

@chan-hoo
Copy link
Collaborator Author

chan-hoo commented Jan 9, 2025

On Orion:

+ ctest
Test project /work/noaa/epic/chjeon/landda_hash_orion/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed   10.51 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed   30.13 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    2.60 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   50.75 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) =  94.03 sec

@chan-hoo
Copy link
Collaborator Author

Hera:

+ ctest
Test project /scratch2/NAGAPE/epic/Chan-hoo.Jeon/landda_hash/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed    2.38 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed   16.69 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    2.40 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   30.76 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) =  52.24 sec

@chan-hoo
Copy link
Collaborator Author

@jkbk2004 , please review this pr.

@chan-hoo chan-hoo merged commit c48cf7b into ufs-community:develop Jan 15, 2025
2 checks passed
@chan-hoo chan-hoo deleted the feature/hash_uwm branch January 22, 2025 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Hash of UFS Weather Model needs to be updated
2 participants