Fix "Special Weather" Alert Type Not Found #196
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
MrBartusek
requested changes
Jun 27, 2023
MrBartusek
changed the title
Also fixed comment on typo fix
brendangoldz
force-pushed
the
specialweather-weatheralerts
branch
from
5819826 to
f732ae8
Compare
MrBartusek
approved these changes
Jun 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes: #195
Sample Warning from NWS...
https://forecast.weather.gov/wwamap/wwatxtget.php?cwa=usa&wwa=Special%20Weather%20Statement
For some reason it is not listed on the nws definition page for alerts, https://www.weather.gov/lwx/WarningsDefined
I think this solution can be boiled down to being a thunderstorm but I dont have enough data to support that definitively. Thought about maybe parsing down the statement for what the alerts is stating but that seems overkill.