-
Notifications
You must be signed in to change notification settings - Fork 6
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 hydrometeors #422
Add hydrometeors #422
Conversation
The PRs from JEDI mentioned above have been merged. I believe the changes here could now make it into SWELL |
@rtodling Can you set the environment |
This will be on-hold until a JEDI build post ~September 20th. |
@Dooruk All tests seem to have passed for this - can we take this in? If not, can we chat? |
Did you run t1tests? I will fire it up in github actions now |
I was looking in the doc how as I user I can fire up the tier-1 test ... but it's not quite clear to me. |
Doesn't this require the new JEDI build by the way? |
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.
So overall, there are issues regarding get_values
and file permissions were changed. These changes are breaking ocean 3dvar and hofx.
src/swell/configuration/jedi/interfaces/geos_atmosphere/model/geometry.yaml
Outdated
Show resolved
Hide resolved
src/swell/configuration/jedi/interfaces/geos_atmosphere/model/geometry_inner.yaml
Outdated
Show resolved
Hide resolved
src/swell/configuration/jedi/interfaces/geos_atmosphere/model/pseudo-model.yaml
Outdated
Show resolved
Hide resolved
src/swell/configuration/jedi/interfaces/geos_atmosphere/model/r2d2.yaml
Outdated
Show resolved
Hide resolved
src/swell/configuration/jedi/interfaces/geos_atmosphere/model/stage.yaml
Outdated
Show resolved
Hide resolved
src/swell/configuration/jedi/interfaces/geos_atmosphere/model/stage_cycle.yaml
Outdated
Show resolved
Hide resolved
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.
all good
Description
This adds hydrometeors and changes to the linear obs operators necessary to account for sensitivity of all-sky MW to such fields.
Dependencies
Impact
Enhanced features and corrections and consistency w/ GSI
Issues
[#418] this is associated w/ the issue marked here.
Required Change in Static Files
This will require a change to the cli_* and hyb_* gsibec 'nml' : all qr/qs references in these files must be uncommented.