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

Modify prepqc script to check for pre-qc data and skip if no data is present #184

Merged
merged 3 commits into from
Jul 25, 2022

Conversation

gmao-msienkie
Copy link
Contributor

Originally the prepqc script determined the templates for the 'pre-qc' files to be combined and passed through 'prepqc' by checking the output of the 'acquire_obsys' command. A few years ago the DAS was modified to obtain the templates for the 'pre-qc' files by checking the OBSCLASS and obtaining the templates from the obsys.rc. This was necessary because the 'acquire_obsys' was being run several times and the obsys.acq output from e.g. the acquire of aerosol data was overwriting the the acquire of the GSI observations.
The problem with this is the 'obsys.acq' had previously only had filename templates for data that was actually acquired for the segment while the newer method obtained templates for all of the files with pre-qc OBSCLASS entries even if the data was not being obtained.
This script adds a check that each pre-qc file exists before the filename is added to the input to the 'combfrd.x' program that combines the prepbufr file. If there are no matching pre-qc files present, the rest of the prepqc related processing is skipped.
Among other things, this allows a DAS run to transition between MERRA2/GEOSIT/R21c reanalysis data streams to the real-time GDAS stream (which does not need and should not have 'prepqc' processing performed on it) without having to stop and reconfigure. The obsys.rc file can be configured to use the 'pre-qc' files up to a certain date and the real-time files after that date.
The GEOSdas.csm modifications have been tested. There are similar modifications in 'prepqc_daemon.pl' that have not been tested since it doesn't look like that script is being used in current runs.

I have also made a small change that will hopefully remove the dependency on having the unneeded ana_satbang_rst file in the restart files. A test has been set up running with the old VarBC to verify that the ana_satbang_rst file will be handled correctly for that case.

Modify code to check for presence of pre-qc files before running 'combfrd.x'
 and skip running prepqc if no pre-qc files are present
and skip combfrd.x and prepqc if no pre-qc files are present
@gmao-msienkie gmao-msienkie requested a review from a team as a code owner June 3, 2022 02:28
@gmao-msienkie gmao-msienkie added the Non 0-diff The changes in this pull request are non-zero-diff label Jun 3, 2022
@gmao-msienkie
Copy link
Contributor Author

The test with unified radianceVarBC showed that the satbang rs was no longer needed. The test with the old bias correction (and $ANGLBC=0) required and acquired the satbang rs as needed

Copy link
Collaborator

@rtodling rtodling left a comment

Choose a reason for hiding this comment

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

I get the sense that your changes in GEOSadas associated w/ satbang need similar changes elsewhere ... I take this in and see what I get. Thanks

@rtodling rtodling merged commit b648de9 into develop Jul 25, 2022
@gmao-msienkie gmao-msienkie deleted the feature/msienkie/pre-qc_transition branch January 31, 2023 17:20
@gmao-msienkie gmao-msienkie restored the feature/msienkie/pre-qc_transition branch January 31, 2023 17:21
@gmao-msienkie gmao-msienkie deleted the feature/msienkie/pre-qc_transition branch January 31, 2023 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Non 0-diff The changes in this pull request are non-zero-diff
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants