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.

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 Aerosol DA Tasks to Rocoto Workflow #981

Closed
3 tasks
CoryMartin-NOAA opened this issue Aug 17, 2022 · 1 comment · Fixed by #990
Closed
3 tasks

Add Aerosol DA Tasks to Rocoto Workflow #981

CoryMartin-NOAA opened this issue Aug 17, 2022 · 1 comment · Fixed by #990
Assignees

Comments

@CoryMartin-NOAA
Copy link
Contributor

CoryMartin-NOAA commented Aug 17, 2022

Description
To support global aerosol initialization for future GEFS/GFS implementations, we need to add to global-workflow experiment setup and rocoto XML generation scripts support for including aerosol DA related tasks.

Requirements
I believe we probably want 3 new tasks/jobs

  • aeroanalprep
  • aeroanalrun
  • aeroanalpost

Eventually, I think we will want the aerosol_init job removed, but for prototyping/testing, I would like an option that keeps existing capabilities (for coupled model team) as well as cycling for DA experiments.

Acceptance Criteria (Definition of Done)
Stub Aerosol DA tasks are able to be included in the workflow.

Dependencies
Issue #984

@CoryMartin-NOAA
Copy link
Contributor Author

As part of this work, a bug was found in the develop branch of global-workflow when setting up a cycled experiment. This bugfix will be included as part of the PR that adds stub aerosol DA tasks.

WalterKolczynski-NOAA pushed a commit that referenced this issue Aug 23, 2022
…d and ATMA are used (#990)

This PR adds in stubs for aerosol DA tasks (jobs/rocoto shell scripts), the ability for setup_expt and setup_xml to include three aerosol DA related tasks, and in the process of adding this capability, fixes a bug in which the combination of ATMA and cycled revealed that @Property was being used incorrectly.

Fixes #981
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment