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

Lightning threat indexes #1642

Merged
merged 94 commits into from
Mar 24, 2023
Merged

Conversation

SamuelTrahanNOAA
Copy link
Collaborator

@SamuelTrahanNOAA SamuelTrahanNOAA commented Mar 6, 2023

Description

Calculates the lightning threat index product from HRRR, inside CCPP and sends it to UPP for output.

The algorithm was originally ported from HRRR to the GFDL_atmos_cubed_sphere NGGPS diagnostics, using the w variable in both implementations. I've moved it to CCPP's maximum hourly diagnostics, but it still needs w in the formulation, so I've added the dynamical core's w variable as wgrs to match the u & v winds which are ugrs and vgrs.

Top of commit queue on: TBD

Input data additions/changes

  • No changes are expected to input data.

Anticipated changes to regression tests:

  • Many regression test results may change due to updating to the latest UPP and adding fields to regional output.

Subcomponents involved:

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

Combined with PR's (If Applicable):

I suggest combining this with my PR #1361 which adds other UPP diagnostic output.

Commit Queue Checklist:

  • Link PR's from all sub-components involved
  • Confirm reviews completed in sub-component PR's
  • Add all appropriate labels to this PR.
  • Run full RT suite on either Hera/Cheyenne with both Intel/GNU compilers
  • Add list of any failed regression tests to "Anticipated changes to regression tests" section.

Linked PR's and Issues:

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

… since it is, in fact, the new postxconfig-NT-fv3lam.txt
@SamuelTrahanNOAA SamuelTrahanNOAA mentioned this pull request Mar 23, 2023
37 tasks
@jkbk2004
Copy link
Collaborator

OK! all tests are done. We can start merging process.

@jkbk2004
Copy link
Collaborator

@SamuelTrahanNOAA I will go to ask to merge upstream dependencies.

@jkbk2004
Copy link
Collaborator

BTW, Gaea C4 jobs are not moving at all. An email sent out to Gaea admin.

@jkbk2004
Copy link
Collaborator

Automated RT Failure Notification
Machine: gaea
Compiler: intel
Job: BL
[BL] Repo location: /lustre/f2/pdata/ncep/emc.nemspara/autort/pr/1265303346/20230323031507/ufs-weather-model
Please make changes and add the following label back: gaea-intel-BL

@jkbk2004
Copy link
Collaborator

@SamuelTrahanNOAA all set! can you update fv3 pointer and revert changes in gitmodules?

@SamuelTrahanNOAA
Copy link
Collaborator Author

The fv3atm should be right now.

modulefiles/ufs_acorn.intel.lua Outdated Show resolved Hide resolved
modulefiles/ufs_acorn.intel.lua Outdated Show resolved Hide resolved
@SamuelTrahanNOAA
Copy link
Collaborator Author

I had forgotten to push that change. Now that the acorn modulefiles are updated, I suggest you rerun the regression tests there, to make sure it runs in your account and reproduces the baseline.

@BrianCurtis-NOAA
Copy link
Collaborator

I'm fairly certain I ran with the updated modulefiles. I tried a pull and it did not pick up any modulefiles changes and the change shows in my clone. No need to re-run on Acorn.

@jkbk2004
Copy link
Collaborator

@BrianCurtis-NOAA I agree. PR is ready to merge.

@jkbk2004
Copy link
Collaborator

Please, go ahead to approve the PR.

@jkbk2004 jkbk2004 merged commit 36d6e16 into ufs-community:develop Mar 24, 2023
@grantfirl
Copy link
Collaborator

@SamuelTrahanNOAA Do you have any guidance to help us understand which RTs are expected to fail with this? We're moving these changes to the NCAR fork (NCAR#92) and re-running RTs there and we'd like to understand which tests should fail with the UPP and regional output changes.

@SamuelTrahanNOAA
Copy link
Collaborator Author

@SamuelTrahanNOAA Do you have any guidance to help us understand which RTs are expected to fail with this? We're moving these changes to the NCAR fork (NCAR#92) and re-running RTs there and we'd like to understand which tests should fail with the UPP and regional output changes.

When one updates UPP, any GRIB files may change output. The regional* tests will also change due to the new diag_table. Put together, that's nearly all of the tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Baseline Updates Current baselines will be updated. jenkins-ci Jenkins CI: ORT build/test on docker container Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. Waiting for Reviews The PR is waiting for reviews from associated component PR's.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lightning Threat Indexes
10 participants