-
Notifications
You must be signed in to change notification settings - Fork 261
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
fixed restart non-reproducing issue 2588 when using MERRA2 climatology + Add output of ocean scalar variables for quick diagnostics #2632 + Fix CICD Jenkinsfile change default RT suite to full #2628 #2625
Conversation
@AnningCheng-NOAA Thanks for the fix! I've confirmed also that your fix passes the restart tests at day 15, hour 00 in my reproducer branch. |
…IS2/OM4_025/diag_table.MOM6
@jkbk2004 This is a critical bug fix. |
Sure! If possible, I like to schedule today with some other no baseline change PRs. |
…s more informative
@AnningCheng-NOAA can you sync up branches? |
@jong Kim - NOAA Affiliate ***@***.***> done.
…On Tue, Mar 4, 2025 at 4:32 PM JONG KIM ***@***.***> wrote:
@AnningCheng-NOAA <https://github.com/AnningCheng-NOAA> can you sync up
branches?
—
Reply to this email directly, view it on GitHub
<#2625 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALQPMIMJU7WKTURIFC2HMV32SYLXLAVCNFSM6AAAAABX775SQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMOJYHE4DAMZTGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
[image: jkbk2004]*jkbk2004* left a comment
(ufs-community/ufs-weather-model#2625)
<#2625 (comment)>
@AnningCheng-NOAA <https://github.com/AnningCheng-NOAA> can you sync up
branches?
—
Reply to this email directly, view it on GitHub
<#2625 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALQPMIMJU7WKTURIFC2HMV32SYLXLAVCNFSM6AAAAABX775SQGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMOJYHE4DAMZTGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@AnningCheng-NOAA can you sync up branches one more time? We can start working on this pr. |
@jkbk2004 Please be sure to commit a clean test change list for this PR. I see you posted the log, but Anning's branch still shows the previous list I believe. |
on-behalf-of @ufs-community <[email protected]>
on-behalf-of @ufs-community <[email protected]>
on-behalf-of @ufs-community <[email protected]>
@BrianCurtis-NOAA skip acorn for this pr? |
Nope, Acorn almost finished. |
OK! we can start merging. |
Commit Queue Requirements:
Description:
Commit Message:
Priority:
Git Tracking
UFSWM:
Sub component Pull Requests:
UFSWM Blocking Dependencies:
Changes
Regression Test Changes (Please commit test_changes.list):
Input data Changes:
Library Changes/Upgrades:
Testing Log: