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

Add unstructured wave grid to gfsv17 targeted tests #1937

Closed

Conversation

JessicaMeixner-NOAA
Copy link
Collaborator

@JessicaMeixner-NOAA JessicaMeixner-NOAA commented Oct 6, 2023

PR Author Checklist:

  • I have linked PR's from all sub-components involved in section below.

  • I am confirming reviews are completed in ALL sub-component PR's.

  • I have run the full RT suite on either Hera/Cheyenne AND have attached the log to this PR below this line:

  • I have added the list of all failed regression tests to "Anticipated changes" section.

  • I have filled out all sections of the template.

Description

HR3 is moving to unstructured grids for the wave model. This PR builds off another HR3 related PR: #1923

Linked Issues and Pull Requests

Associated UFSWM Issue to close

Closes #1860

Subcomponent Pull Requests

None

Blocking Dependencies

Subcomponents involved:

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

Anticipated Changes

Input data

  • No changes are expected to input data.
  • Changes are expected to input data:
    • New input data.
    • Updated input data.

Regression Tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
    Added additional tests. These tests will require a new baseline. However, there's no anticipation of answer changes to any existing test.
Tests effected by changes in this PR:

Libraries

  • Not Needed
  • Needed
    • Create separate issue in JCSDA/spack-stack asking for update to library. Include library name, library version.
    • Add issue link from JCSDA/spack-stack following this item
Code Managers Log
  • 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.
    • N/A

Testing Log:

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

@JessicaMeixner-NOAA
Copy link
Collaborator Author

@jkbk2004 I ran a full set of regression tests over the weekend against a freshly created baseline. The logs have just been posted. I know this PR will be merged with @mdtoyNOAA PR #1923 should I make a PR to his branch now? Or should I add all the subcomponent details to his PR here? I'm thinking it'll be easier to add this to @mdtoyNOAA's branch as the subcomponent PRs are there, but happy to do either way.

@JessicaMeixner-NOAA
Copy link
Collaborator Author

@junwang-noaa I have removed the extra tests and have only kept 1 GFSv17 test. Please let me know if I've overlooked anything else that should be removed or you'd like additional changes

@junwang-noaa
Copy link
Collaborator

@JessicaMeixner-NOAA May I ask which test will be used in the GFSv17 HR3, cpld_control_gfsv17 or cpld_control_gfsv17_ugwpv1_pdlib? If cpld_control_gfsv17_ugwpv1_pdlib is the one, can we rename cpld_control_gfsv17_ugwpv1_pdlib to cpld_control-gfsv17? Also is there any other use of cpld_control_gfsv17 while cpld_control_gfsv17_ugwpv1_pdlib is used in future HR experiments? I am asking because the IAU test is currently based on cpld_control_gfsv17, I hope these tests are all based on latest GFSv17 configuration. Thanks

@JessicaMeixner-NOAA
Copy link
Collaborator Author

Hi @junwang-noaa we'll be using cpld_control_gfsv17_ugwpv1_pdlib

I can rename this test as desired and update the IAU test accordingly. Should have it done by this afternoon so hopefully this will not delay this PR in the queue.

@lisa-bengtsson
Copy link
Contributor

could the cpld benchmark test be updated to use this prototype? It is currently based on P8 and we don't have a high resolution RT for the later prototypes.

@JessicaMeixner-NOAA
Copy link
Collaborator Author

My plan is to only update the GFSv17 tests, particularly as the wave updates are only valid for GFS not GEFS.

@junwang-noaa
Copy link
Collaborator

It looks to me the FV3 is pointing to a user's branch, would you please add the FV3 PR/CCPP PR dependency info? I'd suggest to confirm all the related PRs are reviewed and tested in order to speed up the code commit. My understanding is that code managers are working on Dom's PR today.

@JessicaMeixner-NOAA
Copy link
Collaborator Author

JessicaMeixner-NOAA commented Oct 19, 2023

This PR builds on #1923 where there is an update to FV3. The updates here will be merged to that PR. That PR has all of the dependency information.

@junwang-noaa
Copy link
Collaborator

I'd appreciate you close this PR and move discussions to PR#1923. Thanks

@JessicaMeixner-NOAA
Copy link
Collaborator Author

Happy to move the discussion to PR #1923, I thought this PR would only be closed when PR#1923 was merged. Apologies for not understanding the current code management practices, I'll close this now. @mdtoyNOAA I'll have PR to your PR branch later today with the requested updates from this thread.

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.

Update cpld_control_gfsv17 to use unstructured grids
5 participants