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 inline post #1794

Merged
merged 38 commits into from
Jul 27, 2023
Merged

Update inline post #1794

merged 38 commits into from
Jul 27, 2023

Conversation

WenMeng-NOAA
Copy link
Contributor

@WenMeng-NOAA WenMeng-NOAA commented Jun 14, 2023

Description

Update inline post with:

  • Clean up legacy code in inline post interface

  • Relocate computation of diagnostic variables

  • Sync changes from offline post interface

  • Update upp revision

Input data additions/changes

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

Anticipated changes to regression tests:

  • No changes are expected to any regression test.
  • Changes are expected to the following tests:
    The UPP output grib2 files for global and regional FV3, such as datasets GFSPRS, NATLEV, PRSLEV due to upp upgrade.

RegressionTests_hera.log
RegressionTests_hera-0702.log

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:

  • Link PR's from all sub-components involved in section below
  • Confirm reviews completed in ALL sub-component PR's
  • Add all appropriate labels to this PR.
  • Run full RT suite on either Hera/Cheyenne AND attach log to a PR comment.: attached WCOSS2 test log
  • 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
    • Hera
    • Orion
    • Jet
    • Gaea
    • Cheyenne
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
    • Completed
  • opnReqTest
    • N/A
    • Log attached to comment

@jkbk2004
Copy link
Collaborator

@WenMeng-NOAA Can we check the status of this pr? If good status, can we follow up to attach pre regression test log?

@WenMeng-NOAA
Copy link
Contributor Author

@WenMeng-NOAA Can we check the status of this pr? If good status, can we follow up to attach pre regression test log?

@jkbk2004 I have been waiting for my FV3 PR reviewed. Meanwhile I can sync my branch for regression testing.

@WenMeng-NOAA
Copy link
Contributor Author

WCOSS2 RT test log
RegressionTests_wcoss2.log

@jkbk2004 jkbk2004 added Baseline Updates Current baselines will be updated. Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. labels Jun 29, 2023
@jkbk2004
Copy link
Collaborator

@WenMeng-NOAA All pre test looks good. Can you try to combine in @BrianCurtis-NOAA #1810 ort script update?

@WenMeng-NOAA
Copy link
Contributor Author

@WenMeng-NOAA All pre test looks good. Can you try to combine in @BrianCurtis-NOAA #1810 ort script update?

@jkbk2004 What is the best way to combine my PR #1794 and @BrianCurtis-NOAA 's PR #1810? Manually pick up his changes and commit in my branch?

@jkbk2004
Copy link
Collaborator

@WenMeng-NOAA All pre test looks good. Can you try to combine in @BrianCurtis-NOAA #1810 ort script update?

@jkbk2004 What is the best way to combine my PR #1794 and @BrianCurtis-NOAA 's PR #1810? Manually pick up his changes and commit in my branch?

git clone -b upp_june https://github.com/WenMeng-NOAA/ufs-weather-model
cd ufs-weather-model/
git remote add upstream https://github.com/BrianCurtis-NOAA/ufs-weather-model
git fetch upstream
git merge upstream/fixortdbg
git push origin upp_june

@WenMeng-NOAA
Copy link
Contributor Author

@WenMeng-NOAA All pre test looks good. Can you try to combine in @BrianCurtis-NOAA #1810 ort script update?

@jkbk2004 What is the best way to combine my PR #1794 and @BrianCurtis-NOAA 's PR #1810? Manually pick up his changes and commit in my branch?

git clone -b upp_june https://github.com/WenMeng-NOAA/ufs-weather-model cd ufs-weather-model/ git remote add upstream https://github.com/BrianCurtis-NOAA/ufs-weather-model git fetch upstream git merge upstream/fixortdbg git push origin upp_june

@jkbk2004 Done.

@FernandoAndrade-NOAA
Copy link
Collaborator

Ok, looks like this is ready for testing, getting started with jenkins ci @BrianCurtis-NOAA

@FernandoAndrade-NOAA FernandoAndrade-NOAA added the jenkins-ci Jenkins CI: ORT build/test on docker container label Jun 29, 2023
@BrianCurtis-NOAA
Copy link
Collaborator

@jkbk2004 we're Hera RT's ran? Where is that log?

@jkbk2004
Copy link
Collaborator

@jkbk2004 we're Hera RT's ran? Where is that log?

Attached hera log in the pr description above. No crash on hera. Jenkins-ci underway. I am going to manually submit BL creation on hera now.

@jkbk2004
Copy link
Collaborator

@FernandoAndrade-NOAA can you check jenkins-ci status?

@FernandoAndrade-NOAA
Copy link
Collaborator

@FernandoAndrade-NOAA can you check jenkins-ci status?

regional_control_gnu thr and dcp failed, I'll start a manual run on Hera

@FernandoAndrade-NOAA
Copy link
Collaborator

Apologies for the delay, there were timeout issues with std case. dcp regional_control_gnu is still running at the moment, thr test case failed during the comparisons to NATLEV.GrbF06 and PRSLEV.GrbF06

@jkbk2004 jkbk2004 removed Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. jenkins-ci Jenkins CI: ORT build/test on docker container labels Jun 30, 2023
@zach1221 zach1221 added the Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. label Jul 26, 2023
@github-actions
Copy link

@WenMeng-NOAA please bring these up to date with respective authoritative repositories

  • ufs-weather-model NOT up to date
  • ww3 NOT up to date

@zach1221
Copy link
Collaborator

Good morning, @WenMeng-NOAA . I think we're ready to start over with our testing of this PR. Could you please sync up your branch with ufs-community:develop?

@zach1221 zach1221 added the jenkins-ci Jenkins CI: ORT build/test on docker container label Jul 26, 2023
@WenMeng-NOAA
Copy link
Contributor Author

Good morning, @WenMeng-NOAA . I think we're ready to start over with our testing of this PR. Could you please sync up your branch with ufs-community:develop?

@zach1221 Just synced.

@zach1221
Copy link
Collaborator

Jenkins-ci 1794 log attached. ORTs have passed.
Jenkins-ci 1794.pdf

@zach1221 zach1221 added hera-BL Run Hera baseline creation jet-BL Run Baseline and RT on Jet and removed hera-BL Run Hera baseline creation jet-BL Run Baseline and RT on Jet labels Jul 26, 2023
@zach1221
Copy link
Collaborator

Ok, testing is complete. I'll follow up on the fv3atm sub-pr so we can begin the merging process.

@zach1221
Copy link
Collaborator

@WenMeng-NOAA the fv3atm sub-pr has been merged. Please update the submodule pointer and revert the change in gitmodules.
fv3atm hash: 6d17939836dd5665854a88931552f17e56c1c377

@WenMeng-NOAA
Copy link
Contributor Author

@WenMeng-NOAA the fv3atm sub-pr has been merged. Please update the submodule pointer and revert the change in gitmodules. fv3atm hash: 6d17939836dd5665854a88931552f17e56c1c377

@zach1221 Done.

@zach1221 zach1221 merged commit 506e532 into ufs-community:develop Jul 27, 2023
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.
Projects
None yet
9 participants