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

Prepare observations for snow DA updates to the ensemble members #998

Merged
merged 10 commits into from
Apr 16, 2024

Conversation

jiaruidong2017
Copy link
Collaborator

@jiaruidong2017 jiaruidong2017 commented Mar 27, 2024

This PR updates several configures to prepare observations for snow DA updates to the ensemble members. Because the snow observations are fetched from the COM_OBS in the gdas deterministic analysis and are saved into the COM_OBS in the enkfgdas analysis for the enkfgdas run, we have to define two different COM_OBS directories (COMIN_OBS vs COMOUT_OBS) and two obs basename heads (OPREFIX_IN vs OPREFIX_OUT) to identify the enkf analysis from the gdas deterministic analysis.

This PR will contribute to NOAA-EMC/global-workflow#2416.

Copy link
Contributor

@CoryMartin-NOAA CoryMartin-NOAA left a comment

Choose a reason for hiding this comment

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

Do you have a draft, corresponding workflow PR? I am not particularly thrilled about these additional variable names and would prefer to come up with a way that is more consistent with other observation types.

EDIT: Never mind, I see you have it linked above. I will comment there.

@CoryMartin-NOAA CoryMartin-NOAA merged commit f5d6da3 into develop Apr 16, 2024
1 of 5 checks passed
@CoryMartin-NOAA CoryMartin-NOAA deleted the feature/prepsnowobs_ens branch April 16, 2024 20:01
danholdaway added a commit that referenced this pull request Apr 17, 2024
* upstream/develop:
  Get SOCA vrfy job working on Hera again (#1045)
  Get test data from a staged location on supported HPC Part Deux (#1052)
  Prepare observations for snow DA updates to the ensemble members (#998)
  Save basic stats in csv at each cycle (#1040)
  Fix bug for datetime in GHRSST Ioda Converter (#1027)
@CoryMartin-NOAA CoryMartin-NOAA restored the feature/prepsnowobs_ens branch April 22, 2024 13:00
@CoryMartin-NOAA
Copy link
Contributor

@jiaruidong2017 I'm going to revert this PR as I think it is holding up other workflow PRs. We can re-merge it when the workflow is ready...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants