-
Notifications
You must be signed in to change notification settings - Fork 259
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 excessive evaporation when both innerloop and mraerosol=T #2221
Conversation
@AnningCheng-NOAA I'm running UFS RTs on Hera right now and will upload the test_changes.list file when finished. I'm not sure what happened to the PR template when you edited it, but it looks different than other PRs. |
@grant Firl ***@***.***> are you referring to
NOAA-EMC/fv3atm#816?
…On Thu, Apr 25, 2024 at 1:55 PM Grant Firl ***@***.***> wrote:
@AnningCheng-NOAA <https://github.com/AnningCheng-NOAA> I'm running UFS
RTs on Hera right now and will upload the test_changes.list file when
finished. I'm not sure what happened to the PR template when you edited it,
but it looks different than other PRs.
—
Reply to this email directly, view it on GitHub
<#2221 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALQPMIMCMB5VBYQCN5W6F73Y7E7R5AVCNFSM6AAAAABFSAGCAGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZXHA2DIMRYGU>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
No, this PR: #2221 The description format looks like it is messed up somehow. Also, I fixed .gitmodules in your ufs-weather-model and fv3atm branches (you had that the branches were called innl when they should have been mr2_innl), and the fv3atm PR was pointing to old commits of upp and atmos_cubed_sphere. I think that it should be fixed now. |
@grantfirl, I have just updated the PR description, a little bit better. |
@grantfirl @AnningCheng-NOAA can you please sync up the branch for the PR? |
@grantfirl @AnningCheng-NOAA It was a bit hard to follow what ccpp-framework PR was going into this, but I think i figured it out and linked the correct on in the PR description. Please double check that. What I have not done yet was to make sure that the branch for ccpp-framework PR matched that of the one in the fv3atm PR. Could you please double check that as well? Also make sure you keep your test_changes.list when bumping the branch/fixing conflicts. |
@BrianCurtis-NOAA @AnningCheng-NOAA I can get to this around 4ET today. Please stand by. |
Sorry about the CCPP Framework PR issue. Since this is Anning's PR, I can't edit the description, but you figured out the correct one: NCAR/ccpp-framework#555 |
on-behalf-of @ufs-community <ecc.platform@noaa.gov>
Orion is hanging for me, and I'm unable to run any jobs since last night. I've reached out to MSU to see if there is broader issue. |
…r-model into mr2_innl
…r-model into mr2_innl
There seems to be an ecflow connection issue on Jet. Some tests make it through before connection errors begin to occur. I'm messaging Jet admins to get a support ticket opened and looked into. The RT log will have some of the tests that made it through but it will end without the rest of the tests or the test summary. |
I have not seen this on WCOSS2/Acorn, so it would lean towards a Jet specific issue. We've made some small ecflow changes necessary to Acorn/WCOSS2 that will be brought in when those RT's complete, but if you wanted to see if those changes help/hinder things (i don't think they will be impactful though) then you can git pull using https://github.com/BrianCurtis-NOAA/ufs-weather-model/tree/ecflow_fixes |
I think this is likely a jet specific issue as well, I didn't run into these connection errors with Hera and Gaea. Typically I only see these errors at the beginning when the script needs to start up ecflow, but these are showing up in the middle of testing:
And on the final occurrence, the remaining tasks seem to just be cancelled:
It looks like the sudden log cutoff is from being unable to find the results for the leftover tasks:
|
I'll retry with rocoto while I wait for a reply from the Jet admins. |
Looks like orion is available again, I was able to submit jobs finally. |
Testing complete. We can move to merge the ccpp physics and framework sub-prs |
Commit Queue Requirements:
Description:
NWFA Induced evaporation is turn off , but evaporation not related to aerosol is turned on to prevent excessive evaporation when Inner loop and mraerosol=T
Commit Message:
NWFA Induced evaporation is turn off , but evaporation not related to aerosol is turned on to prevent excessive evaporation when Inner loop and mraerosol=T
Priority:
Git Tracking
Sub component Pull Requests:
ccpp_track_variables.py
, activate "chunked data" CI test, add missing MPI target in src/CMakeLists.txt NCAR/ccpp-framework#555Changes
Regression Test Changes (Please commit test_changes.list):
Input data Changes:
Library Changes/Upgrades:
RegressionTests_hera.log
Testing Log: