home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
71.88k stars 30.11k forks source link

Accuweather error - mi/h is not a recognized speed unit #65614

Closed FuzzyMistborn closed 2 years ago

FuzzyMistborn commented 2 years ago

The problem

I get the below error in my log. The wind sensor appears to work however.

What version of Home Assistant Core has the issue?

2022.02.1

What was the last working version of Home Assistant Core?

2021.12.10

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Accuwather

Link to integration documentation on our website

https://www.home-assistant.io/integrations/accuweather/

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant.components.weather
Source: util/speed.py:45
Integration: Weather (documentation, issues)
First occurred: 9:29:19 PM (4 occurrences)
Last logged: 9:30:29 PM

Error adding entities for domain weather with platform accuweather
Error while setting up accuweather platform for weather
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 382, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 614, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 773, in add_to_platform_finish
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 530, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 565, in _async_write_ha_state
    attr.update(self.state_attributes or {})
  File "/usr/src/homeassistant/homeassistant/components/weather/__init__.py", line 237, in state_attributes
    self.hass.config.units.wind_speed(wind_speed, unit),
  File "/usr/src/homeassistant/homeassistant/util/unit_system.py", line 167, in wind_speed
    return speed_util.convert(wind_speed, from_unit, self.wind_speed_unit)  # type: ignore
  File "/usr/src/homeassistant/homeassistant/util/speed.py", line 45, in convert
    raise ValueError(UNIT_NOT_RECOGNIZED_TEMPLATE.format(unit_1, SPEED))
ValueError: mi/h is not a recognized speed unit.

Additional information

I've tried removing/readding the component and still get the same error.

probot-home-assistant[bot] commented 2 years ago

accuweather documentation accuweather source (message by IssueLinks)

probot-home-assistant[bot] commented 2 years ago

Hey there @bieniu, mind taking a look at this issue as it has been labeled with an integration (accuweather) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)

mikesalz commented 2 years ago

Same problem here after upgrading from 2021.12.x to 2022.2.1

Logger: homeassistant.components.weather
Source: util/speed.py:45
Integration: Weather ([documentation](https://www.home-assistant.io/integrations/weather), [issues](https://github.com/home-assistant/home-assistant/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+weather%22))
First occurred: 9:47:29 AM (2 occurrences)
Last logged: 9:47:29 AM

Error adding entities for domain weather with platform accuweather
Error while setting up accuweather platform for weather
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 382, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 614, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 773, in add_to_platform_finish
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 530, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 565, in _async_write_ha_state
    attr.update(self.state_attributes or {})
  File "/usr/src/homeassistant/homeassistant/components/weather/__init__.py", line 237, in state_attributes
    self.hass.config.units.wind_speed(wind_speed, unit),
  File "/usr/src/homeassistant/homeassistant/util/unit_system.py", line 167, in wind_speed
    return speed_util.convert(wind_speed, from_unit, self.wind_speed_unit)  # type: ignore
  File "/usr/src/homeassistant/homeassistant/util/speed.py", line 45, in convert
    raise ValueError(UNIT_NOT_RECOGNIZED_TEMPLATE.format(unit_1, SPEED))
ValueError: mi/h is not a recognized speed unit.
lvcabral commented 2 years ago

Same issue here, after the 2022.2.1 update

FuzzyMistborn commented 2 years ago

Please do not post "same issue here" or like comments that do not provide additional information. Instead, please use the thumbs up emoticon to show that you're having the issue. I'm sure the dev(s) will work to resolve this.

ih8gates commented 2 years ago

When I receive that error, the sensor fails to set up (in my case weather.home). It's state is "unavailable".

Tuxrug commented 2 years ago

Same issue here, after the 2022.2.1 update

For me, I didn't notice the issue until I updated to 2022.2.2.

Sensor went unavailable at 5:55 pm MST last night 2/4/2022. Looking at my updater sensor that's the same time my updater sensor turned off so it was fine on 2022.2.1 until I updated my Docker install with the .2 image.

I doubt a change in .1 likely would have been skipped until .2 in only the Docker image, but maybe that explains the version difference I have on this issue. Or perhaps coincidentally AccuWeather made an API change around that time that changed the responses?

lvcabral commented 2 years ago

The fix was already pushed as a pull request, will probably be on the next release