Skip to content

Aeris/Xweather unavailable fixes #974

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

Closed
wants to merge 2 commits into from

Conversation

uajqq
Copy link
Contributor

@uajqq uajqq commented Jan 14, 2025

This is a fix for #904 and #959, both errors related to Aeris/Xweather. I have no idea why the diff looks so insane -- if you know how to fix this please tell me -- but all I did is wrap the entire forecast block in a try block, so exceptions can be handled gracefully and rendering can continue if any step fails. I also added a block (line 1412) that specifically catches the IOError that only happens on the first time the skin is run, so an informational message can be logged instead.

@uajqq uajqq force-pushed the aeris_unavailable_fix branch from 5af3433 to 139c74e Compare January 15, 2025 00:33
Copy link

stale bot commented Apr 25, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Issue has not had activity label Apr 25, 2025
@roe-dl
Copy link
Contributor

roe-dl commented Apr 25, 2025

What about this?

@stale stale bot closed this Jun 26, 2025
@uajqq uajqq deleted the aeris_unavailable_fix branch June 27, 2025 13:51
@uajqq
Copy link
Contributor Author

uajqq commented Jun 27, 2025

Since this repo is no longer being actively maintained, I've merged this fix into my own branch, which I'll maintain moving forward. Feel free to use it if you need this bug fix: https://github.com/uajqq/weewx-belchertown-new

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stale Issue has not had activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants