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

openmeteo: switch from current_weather to current parameter #51

Merged
merged 2 commits into from
Nov 26, 2023

Conversation

dgw
Copy link
Member

@dgw dgw commented Nov 14, 2023

The current_weather=1 parameter no longer appears in Open-Meteo documentation, and is likely deprecated.

Handling that response format also presented us with intermittent bugs, especially around fetching humidity data from an hourly series based on timestamp data that wasn't necessarily aligned with that series.

Using the 'current' parameter, we can directly request exactly the weather properties we need, and nothing else.

Fixes #50.

dgw added 2 commits November 14, 2023 17:33
The 'current_weather=1' parameter no longer appears in Open-Meteo
documentation, and is likely deprecated.

Handling that response format also presented us with intermittent bugs,
especially around fetching humidity data from an hourly series based on
timestamp data that wasn't necessarily aligned with that series.

Using the 'current' parameter, we can directly request exactly the
weather properties we need, and nothing else.
@dgw dgw added enhancement New feature or request bugfix labels Nov 14, 2023
@RustyBower RustyBower merged commit 3506e90 into master Nov 26, 2023
4 checks passed
@RustyBower RustyBower deleted the openmeteo-fixes branch November 26, 2023 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Open-Meteo Intermittently Fails
2 participants