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

Openweather map unable to get wind direction / bearing #18082

Closed
AmiSMB opened this issue Nov 1, 2018 · 1 comment · Fixed by #18089
Closed

Openweather map unable to get wind direction / bearing #18082

AmiSMB opened this issue Nov 1, 2018 · 1 comment · Fixed by #18089

Comments

@AmiSMB
Copy link

AmiSMB commented Nov 1, 2018

Home Assistant release with the issue:

0.81.0

Last working Home Assistant release (if known):

Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io

Component/platform:

openweathermap

Description of problem:
Openweather map throws up errors in the log file when it tries to update due to data returned has changed for wind direction/bearing

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

  - platform: openweathermap
    api_key: !secret openweathermap_api_key
    monitored_conditions:
      - weather
      - temperature
      - wind_speed
      - wind_bearing
      - humidity
      - pressure
      - clouds
      - rain
      - snow

Traceback (if applicable):

  File "/usr/local/lib/python3.6/site-packages/homeassistant/components/sensor/openweathermap.py", line 156, in update
    self._state = round(data.get_wind()['deg'], 1)
KeyError: 'deg'

Additional information:
Looks like they have changed the data payload when the API is called as the 'deg' key no longer exists

@AmiSMB AmiSMB changed the title Openweather map unable to get wind direction Openweather map unable to get wind direction / bearing Nov 1, 2018
@fabaff
Copy link
Member

fabaff commented Nov 1, 2018

No, they haven't but not all data is available all the time. For my location it works fine.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants