-
Notifications
You must be signed in to change notification settings - Fork 11
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
clean up variable and subroutine names for Ldas grid #637
Conversation
src/Components/GEOSldas_GridComp/GEOSlandassim_GridComp/clsm_ensupd_enkf_update.F90
Outdated
Show resolved
Hide resolved
Thanks, @weiyuan-jiang, for putting this together. I applied a couple of small commits that should further clean up the code and hope I didn't break anything. I'm not sure the renaming is done in all the right places quite yet. See inline comment above. Additionally, perhaps we need to rename tile_grid -> pert_grid in subroutine get_obs_pred(). I haven't really finished this review and will need to revisit the PR later, but I wanted to write down my thoughts up to this point. |
src/Components/GEOSldas_GridComp/GEOSlandassim_GridComp/GEOS_LandAssimGridComp.F90
Show resolved
Hide resolved
This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged. |
This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged. |
This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged. |
This PR is being prevented from merging because you have added one of our blocking labels: Contingent - DNA, Needs Lead Approval, Contingent -- Do Not Approve. You'll need to remove it before this PR can be merged. |
All Nightly Tests passed on the branch. |
Use pert_grid as the standard names for perturbation and assimilation