Skip to content

Offline patch for sonde timeOffset#538

Merged
ShunLiu-NOAA merged 1 commit intoNOAA-EMC:developfrom
delippi:feature/sonde_time_logic
Feb 18, 2026
Merged

Offline patch for sonde timeOffset#538
ShunLiu-NOAA merged 1 commit intoNOAA-EMC:developfrom
delippi:feature/sonde_time_logic

Conversation

@delippi
Copy link
Collaborator

@delippi delippi commented Feb 18, 2026

Description

In #496, it was shown that GSI applies special logic in read_prepbufr.F90 that overrides the twindow specified in the convinfo table for obs with balloon drift information. Essentially, GSI will attempt a fallback time (-3600s) if the original time falls outside the allowed window (5400s), and rejects the observation if the fallback still fails the new window check. This doesn't usually impact the 00/12z cycles since those are launch times. The impact from this change is usually noted in 01/13z cycles where the timeOffset can be -7500s (outside the -5400s time window) in which case, those obs gets snapped to -3600s.

This PR adds this capability to change the timeOffset of the data via the offline_ioda_patch.py utility. Here is an example call to use this new function:

./offline_ioda_patch.py -o ioda_adpupa.nc --patch-timeoffset

There are hard coded kx values so it is technically safe to run on any ioda file and should only change types 120/220 timeOffsets, but it will add a new metaData variable called origtimeOffset.

Issue(s) addressed

Resolves/Results are documented in Issue #496

Checklist

  • I have performed a self-review of my own code.
  • I have run rrfs tests before creating the PR (if applicable).
  • Unit tests added/updated (if applicable).

Copy link
Contributor

@SamuelDegelia-NOAA SamuelDegelia-NOAA left a comment

Choose a reason for hiding this comment

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

LGTM!

@ShunLiu-NOAA ShunLiu-NOAA merged commit 63bcf24 into NOAA-EMC:develop Feb 18, 2026
1 check passed
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.

3 participants