-
Notifications
You must be signed in to change notification settings - Fork 254
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
2D advection of cellular automata (Combined PR# 1804) #1870
Conversation
Log to RT test intel on Hera: All jobs pass. No anticipated baseline changes. |
@lisa-bengtsson would you be open to combining UFS-WM PR# 1804 into this one? It's just documentation changes. |
Yes no problem. |
@BrianCurtis-NOAA, FYI following fv3 sub pr approval and sync, this pr looks good for testing after we advance through the current queue. |
@lisa-bengtsson Ok, I think we're ready to begin testing against this PR. If you can combine the changes from 1804 in this one and sync up your branch/resolve conflicts, then we should be good to go. |
@zach1221 I have synced up the branch. I wonder how I can easiest combine the changes from 1804 with this PR? A merge? |
I usually just clone the PRs, make the updates, then push back to the branch. There's probably an easier way but that's my method. I'm happy to do it, if that's ok? |
That would be great if you could do it, thank you! |
ORTs passed. Jenkins-ci log attached. |
This link is broken. |
should be fixed. |
Testing is complete. I will follow up on the stochastic physics sub-pr to begin the merging process. |
@lisa-bengtsson Stochastic physics PR #62 has been merged. Please update the submodule pointer and revert the change in .gitmodule url. |
fv3 merged: NOAA-EMC/fv3atm@379ef21 |
Thank you! |
PR Author Checklist:
Description
This development adds a capability to advect CA cells (or - in the future - 2D cloud objects) with a mean level wind. The updated CA/cloud state is then used to inform on convective initiation in saSAS (and -in the future - the C3 convection scheme)
*Add namelist ca_advect as control switch.
*Ensure the same definition for halo sizes are used in the domain construction, as when allocating data to be used in the atmospehre_scalar_field_halo routines.
*Update halo size from 1 to 3.
*Bring in U,V and control flags from FV3 to CA driver routines - compute mean level wind
*2D advection of CA cells
Linked Issues and Pull Requests
Associated UFSWM Issue to close
Closes: #1869
Subcomponent Pull Requests
Blocking Dependencies
None
Subcomponents involved:
Anticipated Changes
Input data
Regression Tests:
Tests effected by changes in this PR:
Libraries
Code Managers Log
Testing Log: