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.

Sign up for GitHub

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

Jump to bottom

SKC is considered present weather instead of sky cover #453

Closed
hluker opened this issue May 4, 2021 · 3 comments · Fixed by #879
Closed

SKC is considered present weather instead of sky cover #453

hluker opened this issue May 4, 2021 · 3 comments · Fixed by #879

Comments

@hluker
Copy link

hluker commented May 4, 2021

image

This could be by design for all I know, but I wanted to make sure it wasn't a bug. (I guess I'm thinking in METAR logic, ala CLR).
@akrherz
Copy link
Owner

akrherz commented May 4, 2021

Oh, this is a TAF... I will have to debug this some more.

akrherz added a commit that referenced this issue Mar 26, 2024
@akrherz
fix: TAF decoding of SKC
5dc53dd 
fixes #453
@akrherz akrherz mentioned this issue Mar 26, 2024
Merged
@akrherz
Copy link
Owner

akrherz commented Mar 26, 2024 •
edited
Loading

Well @hluker , time flies. Indeed, this was a bug. I've fixed it now, but need to tread carefully pushing this into prod

  • ensure autoplot handles this with a now possible null sky level
  • ensure the download is OK with this
  • correct the archive

@akrherz akrherz reopened this Mar 26, 2024
akrherz added a commit to akrherz/iem that referenced this issue Mar 27, 2024
@akrherz
doc and fix TAF usage per akrherz/pyIEM#453
4beab57 
https://mesonet.agron.iastate.edu/cgi-bin/request/taf.py?help
akrherz added a commit to akrherz/iem that referenced this issue Mar 27, 2024
@akrherz
Merge pull request #719 from akrherz/240326-2
8189ae2 
doc and fix TAF usage per akrherz/pyIEM#453
@akrherz
Copy link
Owner

akrherz commented Mar 27, 2024

code is now running in prod

@akrherz akrherz closed this as completed Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: TAF decoding of SKC akrherz/pyIEM
2 participants
@akrherz @hluker