Re-enable EnKF-only for atmosphere#2051
Conversation
This PR works along with the following three dependent PRs to enable the EnKF-only configuration for the atmosphere within the global workflow (see detailed description NOAA-EMC/global-workflow#4345) Dependencies: -NOAA-EMC/global-workflow#4345 -NOAA-EMC/jcb-gdas#219 -NOAA-EMC/jcb#32 Resolve - NOAA-EMC/global-workflow#4339 --------- Co-authored-by: Bo Huang <[email protected]>
|
Keep this PR in draft mode until g-w PR #4472 is merged into g-w |
bhuang95
left a comment
There was a problem hiding this comment.
Looks good to me. Thanks for your update!
|
g-w PR #4472 has been merged into g-w |
|
Automated GW-GDASApp Testing Results: |
|
Automated GW-GDASApp Testing Results: |
|
Automated GW-GDASApp Testing Results: |
|
Automated GW-GDASApp Testing Results: |
|
Automated GW-GDASApp Testing Results: |
|
WCOSS2 GDASApp CI Install All tests Passed on Dogwood. |
|
@CoryMartin-NOAA, @DavidNew-NOAA , and @bhuang95 : This PR is ready for merger into GDASApp |
|
Thanks @CoryMartin-NOAA . Merging now. |
| '${MEMDIR}': 'mem%03d' | format(imem) }) %} | ||
| {% if DOENKFONLY_ATM | default(false) %} | ||
| {% for itile in range(1, ntiles + 1) %} | ||
| - ['{{ DATA }}/anl/{{ 'mem%03d' | format(imem) }}/{{ APREFIX_ENS }}cubed_sphere_grid_atminc.tile{{ itile }}.nc', '{{ COM_ATMOS_ANALYSIS_TMPL | replace_tmpl(tmpl_dict) }}/{{ APREFIX_ENS }}csg_jedi_increment.atm.i006.tile{{ itile }}.nc'] |
There was a problem hiding this comment.
parm/atm/atm_det_config.yaml.j2 copies from DATA to jedi_increment.atm.i006.tile*. There is no csg_ prefix on the deterministic individual tile files.
In contrast, the yaml above copies the ensemble jedi_increment tile from DATA to csg_jedi_increment.atm.i006.tile*. The ensemble DA files have a csg_ prefix on the individual tile files.
We should use the same convention for the deterministic and ensemble jedi_increment tile files.
Since the deterministic does not use the csg_ prefix, it seems we should drop the csg_ prefix for the ensmble jedi_increment tile files.
What do you think @bhuang95 , @DavidNew-NOAA , and @CoryMartin-NOAA ?
This inconsistency is causing failures in select g-w CI cases.
There was a problem hiding this comment.
I'd have to look, but is this the distinction between the increment on the tiles and the increment on a Gaussian grid? Or are they both on the tiles?
EDIT: Scratch that, I can't read :-) If we have tileN in the file name, there is no need for csg in my opinion
There was a problem hiding this comment.
Agreed. Inclusion of tile* in the filename clearly indicates that the file is a tiled file on the cube sphere grid.
There was a problem hiding this comment.
@RussTreadon-NOAA The csg_ prefix was added in the workflow somepoint probaby last December. But my impression is whether to add it or not does not affect other parts. It should be good to remove it. Thanks for the change and merging it.
Description
This PR re-enables the changes from PRs #2010 and #2048.
Companion PRs
Successful GDASApp CI for this PR depend upon g-w PR #4472
Issues
Resolves #2050
Automated CI tests to run in Global Workflow