Skip to content
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.

Sign up for GitHub

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 C384mx025_3DVarAOWCDA yamls #2625

Merged
merged 10 commits into from
May 29, 2024
Merged

Add C384mx025_3DVarAOWCDA yamls #2625

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
2c76cb6
add C384mx025_3DVarAOWCDA yamls
JessicaMeixner-NOAA May 24, 2024
42629e6
make v17 folder
JessicaMeixner-NOAA May 24, 2024
e8ad803
Merge branch 'NOAA-EMC:develop' into addyamlexp
JessicaMeixner-NOAA May 29, 2024
db2390c
Update ocnanal.yaml
JessicaMeixner-NOAA May 29, 2024
2b451f8
make account and hpss more generic
JessicaMeixner-NOAA May 29, 2024
619ede3
be consistent with PR yamls
JessicaMeixner-NOAA May 29, 2024
c3a455e
remove user-defined stmp/ptmp/tmp/etc to provide more consistency wit…
JessicaMeixner-NOAA May 29, 2024
6ed1d3c
remove hpss_project for consistency with ci tests
JessicaMeixner-NOAA May 29, 2024
62f4616
update obslist
JessicaMeixner-NOAA May 29, 2024
3928942
Update ci/cases/gfsv17/ocnanal.yaml
JessicaMeixner-NOAA May 29, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions ci/cases/gfsv17/C384mx025_3DVarAOWCDA.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
experiment:
system: gfs
mode: cycled

arguments:
pslot: {{ 'pslot' | getenv }}
app: S2S
resdetatmos: 384
resdetocean: 0.25
nens: 0
gfs_cyc: 4
start: cold
comroot: {{ 'PWD' | getenv }}/{{ 'pslot' | getenv }}/COMROOT
expdir: {{ 'PWD' | getenv }}/{{ 'pslot' | getenv }}/EXPDIR
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this should be PWD, because then it has to be executed within the RUNTESTS/.
This should be kept as is the other yamls

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed.

idate: 2021063000
edate: 2021070306
icsdir: /scratch1/NCEPDEV/climate/Jessica.Meixner/cycling/IC_2021063000_V2
yaml: {{ HOMEgfs }}/ci/cases/gfsv17/ocnanal.yaml
17 changes: 17 additions & 0 deletions ci/cases/gfsv17/obs_list.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
observers:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this obslist in global-workflow?
This is part of GDASApp.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guillaumevernieres is there an obs_list in gadsapp this can point to instead?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could, but we'd have to edit it before running. What's wrong with having the obs list here @aerorahul ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no need to edit anything.

Replace

SOCA_OBS_LIST: {{ 'PWD' | getenv }}/obs_list.yaml

with

SOCA_OBS_LIST: {{ HOMEgfs }}/sorc/gdas.cd/PATH_TO_ocean_obs_list.yaml

in the yaml

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant editing the obs_list.yaml @aerorahul , but that's not relevant to the proposed ci test, sorry.
That should do it @Jeeixner-NOAA :

SOCA_OBS_LIST: {{ HOMEgfs }}/sorc/gdas.cd/parm/soca/obs/obs_list.yaml

# ADT
- !INC ${OBS_YAML_DIR}/adt_rads_all.yaml

# SST
- !INC ${OBS_YAML_DIR}/sst_avhrr_ma_l3u.yaml
- !INC ${OBS_YAML_DIR}/sst_avhrr_mb_l3u.yaml
- !INC ${OBS_YAML_DIR}/sst_avhrr_mc_l3u.yaml
- !INC ${OBS_YAML_DIR}/sst_viirs_npp_l3u.yaml
- !INC ${OBS_YAML_DIR}/sst_viirs_n20_l3u.yaml
- !INC ${OBS_YAML_DIR}/sst_abi_g16_l3c.yaml
- !INC ${OBS_YAML_DIR}/sst_abi_g17_l3c.yaml
- !INC ${OBS_YAML_DIR}/sst_ahi_h08_l3c.yaml

# Ice concentration
- !INC ${OBS_YAML_DIR}/icec_amsr2_north.yaml
- !INC ${OBS_YAML_DIR}/icec_amsr2_south.yaml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just an FYI, the !INC tag is going away. I think its only in the ocean left.
There is no rush, I just wanted to drop a note.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the FYI! @guillaumevernieres want to point this out to you

Quick question, we also use this to include the defaults, how should this change to continue to include those?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes. that is also on a TODO list. I am not worried about the ocean stuff as it will be assembled using JCB

36 changes: 36 additions & 0 deletions ci/cases/gfsv17/ocnanal.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
defaults:
!INC {{ HOMEgfs }}/parm/config/gfs/yaml/defaults.yaml

base:
DOIAU: "YES"
DO_JEDIATMVAR: "NO"
DO_JEDIATMENS: "NO"
DO_JEDIOCNVAR: "YES"
DO_JEDISNOWDA: "NO"
DO_MERGENSST: "NO"
DOHYBVAR: "NO"
DO_FIT2OBS: "NO"
DO_VERFOZN: "NO"
DO_VERFRAD: "NO"
DO_VRFY_OCEANDA: "NO"
ACCOUNT: "marine-cpu"
FHMAX_GFS: 240
HPSS_PROJECT: emc-climate
HOMEDIR: {{ 'PWD' | getenv }}/HOMEDIR
STMP: {{ 'PWD' | getenv }}/TMP
PTMP: {{ 'PWD' | getenv }}/TMP
NOSCRUB: {{ 'PWD' | getenv }}/TMP
Copy link
Contributor

@aerorahul aerorahul May 29, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While this may work, this is not the level of configuration we are built for.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because stmp is always full.... I can remove this from this test though. It might be nice in the future to have like a "user" type yaml that updates all the configs that a person might need to do, I don't know how that would work so this is just spit-balling an idea here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes. stmp being full is a problem.
The feature request is noted. We will do what we can.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe replace with this @JessicaMeixner-NOAA :

{{ 'RUNTESTS' | getenv }}

for stmp and ptmp ?


ocnanal:
SOCA_INPUT_FIX_DIR: /scratch2/NCEPDEV/ocean/Guillaume.Vernieres/data/static/1440x1080x75/soca
CASE_ANL: 'C24'
SOCA_OBS_LIST: {{ 'PWD' | getenv }}/obs_list.yaml
SOCA_NINNER: 100
SABER_BLOCKS_YAML: ''
NICAS_RESOL: 1
NICAS_GRID_SIZE: 15000

prepoceanobs:
SOCA_OBS_LIST: {{ HOMEgfs }}/ci/cases/gfsv17/obs_list.yaml
OBSPREP_YAML: {{ HOMEgfs }}/sorc/gdas.cd/parm/soca/obsprep/obsprep_config.yaml
DMPDIR: /scratch1/NCEPDEV/da/common/
Loading