Skip to content

Fix restart-reproduce if using a restart from 15th of month using MERRA2 climatology#933

Merged
jkbk2004 merged 8 commits into
NOAA-EMC:developfrom
AnningCheng-NOAA:mr2_rst
Mar 10, 2025
Merged

Fix restart-reproduce if using a restart from 15th of month using MERRA2 climatology#933
jkbk2004 merged 8 commits into
NOAA-EMC:developfrom
AnningCheng-NOAA:mr2_rst

Conversation

@AnningCheng-NOAA
Copy link
Copy Markdown
Contributor

@AnningCheng-NOAA AnningCheng-NOAA commented Feb 27, 2025

Description

(Instructions: this, and all subsequent sections of text should be removed and filled in as appropriate.)
Provide a detailed description of what this PR does.
What bug does it fix, or what feature does it add? fixed restart non reproducing issue 2588 from MERRA2 climatology
Is a change of answers expected from this PR?
No

Issue(s) addressed

Link the issues to be closed with this PR, whether in this repository, or in another repository.
(Remember, issues should always be created before starting work on a PR branch!)

Testing

How were these changes tested?
What compilers / HPCs was it tested with? hera
Are the changes covered by regression tests? (If not, why? Do new tests need to be added?) yes, no new tests is needed
Have the ufs-weather-model regression test been run? On what platform?

  • Will the code updates change regression test baseline? If yes, why? Please show the baseline directory below.
    No
  • Please commit the regression test log files in your ufs-weather-model branch
    I have only tested two tests related to restart because ecflow cannot connect the server

Dependencies

If testing this branch requires non-default branches in other repositories, list them.
Those branches should have matching names (ideally)

Do PRs in upstream repositories need to be merged first?
If so add the "waiting for other repos" label and list the upstream PRs

@DusanJovic-NOAA DusanJovic-NOAA changed the title fixed restart non reproducing issue 2588 Fix restart-reproduce if using a restart from 15th of month Mar 7, 2025
@DusanJovic-NOAA DusanJovic-NOAA changed the title Fix restart-reproduce if using a restart from 15th of month Fix restart-reproduce if using a restart from 15th of month using MERRA2 climatology Mar 7, 2025
@grantfirl
Copy link
Copy Markdown
Collaborator

@AnningCheng-NOAA ufs-community/ccpp-physics#256 was merged. Please make sure that the ccpp-physics commit hash is ufs-community/ccpp-physics@38268cf and revert .gitmodules.

@AnningCheng-NOAA
Copy link
Copy Markdown
Contributor Author

AnningCheng-NOAA commented Mar 10, 2025 via email

@grantfirl
Copy link
Copy Markdown
Collaborator

Firl, I got the following message /scratch1/NCEPDEV/global/Anning.Cheng/ufs-weather-model/FV3/ccpp/physics$ git branch mr2_rst * ufs/dev /scratch1/NCEPDEV/global/Anning.Cheng/ufs-weather-model/FV3/ccpp/physics$ git checkout 38268f error: pathspec '38268f' did not match any file(s) known to git

On Mon, Mar 10, 2025 at 11:39 AM Grant Firl @.> wrote: @AnningCheng-NOAA https://github.com/AnningCheng-NOAA ufs-community/ccpp-physics#256 <ufs-community/ccpp-physics#256> was merged. Please make sure that the ccpp-physics commit hash is @. <ufs-community/ccpp-physics@38268cf> and revert .gitmodules. — Reply to this email directly, view it on GitHub <#933 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALQPMINIJ4A62WDYBLDBP6L2TW55RAVCNFSM6AAAAABYABMHJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMJRGAYTSMBZGQ . You are receiving this because you were mentioned.Message ID: @.> [image: grantfirl]grantfirl left a comment (NOAA-EMC/fv3atm#933) <#933 (comment)> @AnningCheng-NOAA https://github.com/AnningCheng-NOAA ufs-community/ccpp-physics#256 <ufs-community/ccpp-physics#256> was merged. Please make sure that the ccpp-physics commit hash is @. <ufs-community/ccpp-physics@38268cf> and revert .gitmodules. — Reply to this email directly, view it on GitHub <#933 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALQPMINIJ4A62WDYBLDBP6L2TW55RAVCNFSM6AAAAABYABMHJWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMJRGAYTSMBZGQ . You are receiving this because you were mentioned.Message ID: @.***>

@AnningCheng-NOAA Don't worry about it. It looks like @jkbk2004 took care of this for you.

@jkbk2004 jkbk2004 merged commit a46b8cb into NOAA-EMC:develop Mar 10, 2025
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.

4 participants