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

Overlapping obs windows for insitu and altimeter obs #837

Closed
guillaumevernieres opened this issue Jan 5, 2024 · 3 comments
Closed

Overlapping obs windows for insitu and altimeter obs #837

guillaumevernieres opened this issue Jan 5, 2024 · 3 comments
Assignees
Labels

Comments

@guillaumevernieres
Copy link
Contributor

guillaumevernieres commented Jan 5, 2024

- obs space:
    name: adt_rads_all
    dmpdir subdir: ADT
    dmpdir regex: rads_adt_??_???????.nc
    provider: RADS
    output file: adt_rads_all.nc4
    window:
       back: 8         # look back 8 six-hourly obs dumps
       forward: 1    #  look forward 1 six-hourly bin 
guillaumevernieres pushed a commit that referenced this issue Jan 27, 2024
#882)

Number of cycles forward and back are set in new entries in
`parm/soca/obsprep/obsprep_config.yaml`, with a hardcoded default of 0.

More cycles of obs (adt and metop b sst) have been added to the ctests
and `test/soca/testdata/` had been reorganized to a structure more
resembling the `dmpdir` in order to keep everything straight.

Most of the "Files changed" are cdl files added to the testdata.

At least partially addresses
#837

Now also fixes #881

Tested with `DMPDIR: /scratch1/NCEPDEV/stmp4/Shastri.Paturi/forAndrew`
as well - `JGLOBAL_PREP_OCEAN_OBS` take 16 minutes
@guillaumevernieres
Copy link
Contributor Author

Closing this, we'll open an other issue for the work that @apchoiCMD is doing, if not done yet.

@apchoiCMD
Copy link
Collaborator

Closing this, we'll open an other issue for the work that @apchoiCMD is doing, if not done yet.

Well noted @guillaumevernieres

@guillaumevernieres
Copy link
Contributor Author

I just checked, there's already an issue open for you @apchoiCMD ...

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

No branches or pull requests

4 participants