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

poc for avwx api fix #25

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

tedder
Copy link
Contributor

@tedder tedder commented Oct 17, 2023

  • remove unneeded files
  • truncate logfile

Lots of duplicated code, in lieu of a fix this is a patch to start to get things running again. the API needs to request XML to get it back, which helps many of them function again.

Still some URLs that need fixing, among other things.

- remove unneeded files
- truncate logfile
@tedder
Copy link
Contributor Author

tedder commented Oct 17, 2023

"rainbow wipe" runs, but the rgb leds aren't coming up yet.

Copy link
Owner

@markyharris markyharris left a comment

Choose a reason for hiding this comment

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

Tedder. Newbie question. Would you mind emailing me at [email protected] so I can ask it?

@markyharris
Copy link
Owner

I haven't had a chance to dive into this yet. But I plan to. If you have any luck please post them so I can fold the fix(es) in. Thanks for all your help.

@tedder
Copy link
Contributor Author

tedder commented Oct 18, 2023

Yeah, the nowx lights are coming on but not the main metar lights, even though the WX data is there.

@e100
Copy link

e100 commented Oct 18, 2023

@markyharris

I forgot to mention in my bug report, if you just change www.aviationweather.gov to aviationweather-cprk.ncep.noaa.gov you can get it working right away!
However, when I contacted NOAA about this being broken after the site upgrade they also informed me to not rely on aviationweather-cprk.ncep.noaa.gov because it will be going away soon.

But seems like it would be a good quick temporary fix until the code is updated to work with www.aviationweather.gov again.
I did that last night on my map and got it working.

@markyharris
Copy link
Owner

markyharris commented Oct 18, 2023 via email

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