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

Update hash of ufs weather model #146

Merged
merged 13 commits into from
Oct 8, 2024

Conversation

chan-hoo
Copy link
Collaborator

@chan-hoo chan-hoo commented Sep 27, 2024

Description

  • Update the hash of the ufs weather model.
  • Update module files.
  • Update the ctest scripts

To run a new ctest script:

cd sorc/test
./run_ctest_platform.sh

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)
  • none

Linked PR's and Issues:

Resolves Issue #145

Testing (for CM's):

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

@chan-hoo chan-hoo linked an issue Sep 27, 2024 that may be closed by this pull request
@chan-hoo chan-hoo requested a review from jkbk2004 September 27, 2024 17:47
@chan-hoo
Copy link
Collaborator Author

+ ctest
Test project /work/noaa/epic/chjeon/landda_test/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed    2.47 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed    4.80 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    2.43 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   59.31 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) =  69.02 sec

@jkbk2004
Copy link
Collaborator

@chan-hoo can you test with limited resource in ufs.configure as we have develop? It's good to see fast run with more PEs. But I think limited resource option is still good to maintain others: container, run on clouds, etc.

@chan-hoo
Copy link
Collaborator Author

chan-hoo commented Oct 2, 2024

On Hera:

Test project /scratch2/NAGAPE/epic/Chan-hoo.Jeon/landda_test/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed    4.61 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed   16.43 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    1.70 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   42.74 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) =  65.49 sec

@chan-hoo
Copy link
Collaborator Author

chan-hoo commented Oct 2, 2024

On Orion:

Test project /work/noaa/epic/chjeon/landda_test/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed   10.80 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed   31.44 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    2.49 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   70.28 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) = 115.11 sec

@chan-hoo
Copy link
Collaborator Author

chan-hoo commented Oct 2, 2024

On Hercules:

Test project /work/noaa/epic/chjeon/landda_test_hercules/land-DA_workflow/sorc/build
    Start 1: test_create_ens
1/4 Test #1: test_create_ens ..................   Passed   55.87 sec
    Start 2: test_letkfoi_snowda
2/4 Test #2: test_letkfoi_snowda ..............   Passed  280.26 sec
    Start 3: test_apply_jediincr
3/4 Test #3: test_apply_jediincr ..............   Passed    2.50 sec
    Start 4: test_ufs_datm_land
4/4 Test #4: test_ufs_datm_land ...............   Passed   69.36 sec

@chan-hoo
Copy link
Collaborator Author

chan-hoo commented Oct 2, 2024

@jkbk2004, plz review this PR again.

@chan-hoo chan-hoo merged commit d8b7ea8 into ufs-community:develop Oct 8, 2024
2 checks passed
@chan-hoo chan-hoo deleted the feature/uw_hash branch October 28, 2024 12:23
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.

Need to update hash of ufs weather model
2 participants