Skip to content

ADD: KILX NEXRAD Level 2 sample file (2023-06-29 15:44 UTC)#98

Merged
kmuehlbauer merged 1 commit into
openradar:mainfrom
aladinor:add-kilx-nexrad-level2
May 8, 2026
Merged

ADD: KILX NEXRAD Level 2 sample file (2023-06-29 15:44 UTC)#98
kmuehlbauer merged 1 commit into
openradar:mainfrom
aladinor:add-kilx-nexrad-level2

Conversation

@aladinor

@aladinor aladinor commented May 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds data/KILX20230629_154426_V06 (~10.4 MB), a single NEXRAD Level 2 archive file for the KILX (Lincoln, IL) WSR-88D, fetched from s3://unidata-nexrad-level2/2023/06/29/KILX/KILX20230629_154426_V06.
  • Updates open_radar_data/registry.txt with the corresponding pooch SHA256 hash.

Test plan

  • CI passes on Python 3.9–3.12
  • DATASETS.fetch('KILX20230629_154426_V06') resolves and verifies the hash after merge

🤖 Generated with Claude Code

Source: s3://unidata-nexrad-level2/2023/06/29/KILX/KILX20230629_154426_V06
@kmuehlbauer kmuehlbauer merged commit 8727533 into openradar:main May 8, 2026
6 checks passed
aladinor added a commit to aladinor/xradar that referenced this pull request May 8, 2026
Address @kmuehlbauer's review on openradar#377. KILX20230629_154426_V06 has
landed in open-radar-data 0.8.0 (openradar/open-radar-data#98), so
we can now ship a targeted regression test against the exact file
that exposed the bug.

LDM 49 of this file contains 120 MSG_31 + 2 MSG_2 = 122 messages.
Pre-fix the mod-120 stride dropped the trailing 2 MSG_31s, so
sweep_10 reported 358 rays (on-wire-correct is 360) and the volume
total was 6838 instead of 6840.

- Add ``nexradlevel2_kilx_ldm_stride_file`` session fixture in
  ``tests/conftest.py``.
- Add ``test_kilx_over_120_ldm_decodes_all_msg31`` asserting
  per-sweep ray counts == [720]*6 + [360]*7, total == 6840, and
  sweep_10 == 360 against the file.
- Bump open-radar-data minimum version to 0.8.0 across
  ``requirements_dev.txt``, ``environment.yml``, ``ci/unittests.yml``,
  and ``ci/notebooktests.yml``.
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.

2 participants