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

PBL, Convection and Microphysics Update for HR2 #1723

Closed
wants to merge 0 commits into from

Conversation

Qingfu-Liu
Copy link
Collaborator

@Qingfu-Liu Qingfu-Liu commented Apr 24, 2023

Description

This PR is created because the ccpp-physics is updated for HR2 (PR#65). The ccpp-physics update in PR#65 includes update for PBL scheme, shallow convection scheme, deep convection scheme and microphysics scheme. The changes of the physics improve hurricane forecast, and CAPE forecast values.

Top of commit queue on: TBD

Input data additions/changes

  • No changes are expected to input data.
  • There will be new input data.
  • Input data will be updated.

Anticipated changes to regression tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
The physics update in PR#65 will improve forecast results

Subcomponents involved:

  • AQM
  • CDEPS
  • CICE
  • CMEPS
  • CMakeModules
  • FV3
  • GOCART
  • HYCOM
  • MOM6
  • NOAHMP
  • WW3
  • stochastic_physics
  • none

Combined with PR's (If Applicable):

Commit Queue Checklist:

Linked PR's and Issues:

None

Testing Day Checklist:

  • This 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.
  • Move new/updated input data on RDHPCS Hera and propagate input data changes to all supported systems.

Testing Log (for CM's):

  • RDHPCS
    • Intel
      • Hera
      • Orion
      • Jet
      • Gaea
      • Cheyenne
    • GNU
      • Hera
      • Cheyenne
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@Qingfu-Liu Qingfu-Liu closed this Apr 25, 2023
@Qingfu-Liu Qingfu-Liu reopened this Apr 26, 2023
@Qingfu-Liu Qingfu-Liu changed the title This PR is created due to the ccpp-physics update for HR2 PBL, Convection and Microphysics Update for HR2 Apr 26, 2023
@jkbk2004
Copy link
Collaborator

@Qingfu-Liu can you sync up branch? then we will try to combine in #1715

@grantfirl
Copy link
Collaborator

@Qingfu-Liu Also, we still need a list of which RTs are expected to fail. If you run all of the tests in rt.conf on Hera and copy/paste the list of failed tests found at the end of your tests/RegressionTest_hera.intel/gnu.log files, that is what we need.

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@grantfirl
Copy link
Collaborator

grantfirl commented Apr 28, 2023

Hi Jong, I saw that If I sync up the branch, I will discard "1 commit", is this what you want? Qingfu

On Fri, Apr 28, 2023 at 3:15 PM JONG KIM @.> wrote: @Qingfu-Liu https://github.com/Qingfu-Liu can you sync up branch? then we will try to combine in #1715 <#1715> — Reply to this email directly, view it on GitHub <#1723 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGTS6UST2EA3E4OXP5OWV5DXDQJLLANCNFSM6AAAAAAXKFB7H4 . You are receiving this because you were mentioned.Message ID: @.>

@Qingfu-Liu There was a PR merged into ufs-weather-model and fv3atm this morning. This branch and your fv3atm branch needs to pull in the latest develop commits.

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@grantfirl
Copy link
Collaborator

grantfirl commented Apr 28, 2023

@grant Firl - NOAA Affiliate @.> , all the runs will be failed since we need to create a baseline data, but the runs do produce the output data. If I rerun the tests, it will take very long time (might be 10+ hours) Qingfu

On Fri, Apr 28, 2023 at 3:22 PM Grant Firl @.
> wrote: @Qingfu-Liu https://github.com/Qingfu-Liu Also, we still need a list of which RTs are expected to fail. If you run all of the tests in rt.conf on Hera and copy/paste the list of failed tests found at the end of your tests/RegressionTest_hera.intel/gnu.log files, that is what we need. — Reply to this email directly, view it on GitHub <#1723 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGTS6UXK43UKC2GWU6LU34DXDQKFNANCNFSM6AAAAAAXKFB7H4 . You are receiving this because you were mentioned.Message ID: @.***>

If you've already run all the tests in rt.conf, then we just need the list of failed tests from the logs that were already produced. We want the list of failed runs to know whether they're all expected. When I go to repeat this procedure for the NCAR:main branch, the list of failed tests is crucial to know if the PR is behaving as expected.

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@grantfirl
Copy link
Collaborator

@grant Firl - NOAA Affiliate @.> , I will start the regression tests now since I did not finish the test last time. I cancelled the run after 5 hours, and late I removed all the data. I hope the tests will be completed sometime tomorrow. Qingfu

On Fri, Apr 28, 2023 at 3:28 PM Grant Firl @.
> wrote: @grant https://github.com/grant Firl - NOAA Affiliate @. > , all the runs will be failed since we need to create a baseline data, but the runs do produce the output data. If I rerun the tests, it will take very long time (might be 10+ hours) Qingfu … <#m_930262221224032925_> On Fri, Apr 28, 2023 at 3:22 PM Grant Firl @.> wrote: @Qingfu-Liu https://github.com/Qingfu-Liu https://github.com/Qingfu-Liu Also, we still need a list of which RTs are expected to fail. If you run all of the tests in rt.conf on Hera and copy/paste the list of failed tests found at the end of your tests/RegressionTest_hera.intel/gnu.log files, that is what we need. — Reply to this email directly, view it on GitHub <#1723 (comment) <#1723 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGTS6UXK43UKC2GWU6LU34DXDQKFNANCNFSM6AAAAAAXKFB7H4 . You are receiving this because you were mentioned.Message ID: @.> If you've already run all the tests in rt.conf, then we just need the list of failed tests from the logs that were already produced. We wan the list of failed runs to know whether they're all expected. When I go to repeat this procedure for the NCAR:main branch, the list of failed tests is crucial to know if the PR is behaving as expected. — Reply to this email directly, view it on GitHub <#1723 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGTS6USLUD7UV747VQHUVGTXDQK33ANCNFSM6AAAAAAXKFB7H4 . You are receiving this because you were mentioned.Message ID: @.>

Are you running with ecflow? It should be much quicker with ecflow.

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@DeniseWorthen
Copy link
Collaborator

@Qingfu-Liu I know you mentioned earlier that the RTs were taking many many hours. If you were using ./rt.sh -l rt.conf, then you were not running with ecflow---meaning each job ran sequentially. That would explain why your tests were taking so long.

On Hera, you'll need to specifically log into the ecflow nodes using ssh hecflow01. Then do

./rt.sh -e >output 2>&1 &

Your tests should go much faster.

@jkbk2004
Copy link
Collaborator

@Qingfu-Liu Do you intend to close pr? fv3 dependency was closed as well. You force pushed, right? , For the this pr, you need to point to your ccpp branch, right? can you fix and re-open?

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@grantfirl
Copy link
Collaborator

@Qingfu-Liu By "sync", Jong means to pull in the latest commits from the develop branches of ufs-weather-model and fv3atm to your PR branches. You still want your PR branch of fv3atm to point to your PR branch of ccpp-physics and your PR branch of ufs-weather-model to point to your PR branch of fv3atm after you've pulled in the latest develop commits.

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@jkbk2004
Copy link
Collaborator

@Qingfu-Liu we may let #1715 got first and come back for this pr next week. would that be ok?

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@Qingfu-Liu
Copy link
Collaborator Author

Qingfu-Liu commented Apr 28, 2023 via email

@grant
Copy link

grant commented Apr 28, 2023

@Qingfu-Liu: I think you mean to ask @grantfirl, not @grant.

@Qingfu-Liu Qingfu-Liu deleted the update_HR2 branch April 29, 2023 01:21
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.

5 participants