-
Notifications
You must be signed in to change notification settings - Fork 10
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
add checks on snow states after application of LDAS increments #676
add checks on snow states after application of LDAS increments #676
Conversation
…eck_after_apply_LDAS_incr
…ain 0-diff of SMAP Tb DA test case)
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. |
components.yaml
Outdated
@@ -41,6 +41,6 @@ MAPL: | |||
GEOSgcm_GridComp: | |||
local: ./src/Components/GEOSldas_GridComp/@GEOSgcm_GridComp | |||
remote: ../GEOSgcm_GridComp.git | |||
branch: develop | |||
branch: feature/rreichle/snow_check_after_apply_LDAS_incr |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@biljanaorescanin: Make sure to restore components.yaml from the develop branch before merging this PR.
PR restores 0-diff of LDAS assim tests after merger of eponymous GEOS-ESM/GEOSgcm_GridComp#834.
0-diff is maintained by turning off the new snow checks in the 3d Tb-based soil moisture analysis for now.
#834 also introduces new exports (FICE[n]). They have not yet been added into GEOS_EnsGridComp.F90. It is unclear if that should be done. (The ens avg of FICE may not be meaningful.) This can be left for a later PR.
Successfully tested by @gmao-rreichle some time ago. Needs retesting as of 6 Oct 2023
Retested successfully for GEOSldas Intel-only on Oct 15 by @gmao-rreichle
Fully tested by @biljanaorescanin on 16/17 Oct 2023
GEOSldas tests passed except for "aggressive" tests, which failed the comparison with roundoff differences, as expected.