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

Make parsing Wyoming post-data values more robust #853

Merged
merged 1 commit into from
Dec 12, 2024

Conversation

dopplershift
Copy link
Member

The main goal is to allow for missing station information, which results in '******' for latitude, no longitude, and -9999.0 for elevation. As a result, avoid the fixed index parsing of this information, and instead rely on the text to the left of the ':' to identify appropriate sections.

Description Of Changes

Checklist

Supersedes and close #748.

@dopplershift dopplershift added this to the 0.10 milestone Dec 11, 2024
@dopplershift dopplershift force-pushed the wyoming-missing-station branch from 0a1573c to e404a32 Compare December 11, 2024 23:37
@dopplershift dopplershift changed the title Make parsing Wyoming post-data values more robust (Fixes #749) Make parsing Wyoming post-data values more robust Dec 11, 2024
tests/test_wyoming.py Outdated Show resolved Hide resolved
…#749)

The main goal is to allow for missing station information, which results
in '******' for latitude, no longitude, and -9999.0 for elevation. As a
result, avoid the fixed index parsing of this information, and instead
rely on the text to the left of the ':' to identify appropriate
sections.
@dopplershift dopplershift force-pushed the wyoming-missing-station branch from e404a32 to 7d6b8f3 Compare December 11, 2024 23:54
@lesserwhirls lesserwhirls merged commit 631cc7c into Unidata:main Dec 12, 2024
33 checks passed
@dopplershift dopplershift deleted the wyoming-missing-station branch December 12, 2024 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

WyomingUpperAir Missing Station Information
2 participants