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
73.47k stars 30.69k forks source link

Ebusd ValueError: too many values to unpack (expected 5) #54131

Closed RoSulek1 closed 3 years ago

RoSulek1 commented 3 years ago

The problem

Logger: homeassistant.components.sensor Source: components/ebusd/sensor.py:44 Integration: Sensor

Error while setting up ebusd platform for sensor Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 249, in _async_setup_platform await asyncio.shield(task) File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 52, in run result = self.fn(*self.args, **self.kwargs) File "/usr/src/homeassistant/homeassistant/components/ebusd/sensor.py", line 31, in setup_platform EbusdSensor(ebusd_api, discovery_info["sensor_types"][condition], name) File "/usr/src/homeassistant/homeassistant/components/ebusd/sensor.py", line 44, in __init__ ( ValueError: too many values to unpack (expected 5)

What is version of Home Assistant Core has the issue?

2021.8.1

What was the last working version of Home Assistant Core?

2021.7.2

What type of installation are you running?

Home Assistant Container

Integration causing the issue

ebusd

Link to integration documentation on our website

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

Example YAML snippet

## Vaillant
ebusd:
  host: 192.168.0.204
  port: 8888
  name: 'ebusd'
  circuit: 'bai'
  monitored_conditions:
    - 'HotWaterTemperature'
    - 'StorageTemperature'
    - 'DesiredStorageTemperature'
    - 'OutdoorsTemperature'
    - 'WaterPreasure'
    - 'AverageIgnitionTime'
    - 'MaximumIgnitionTime'
    - 'MinimumIgnitionTime'
    - 'DesiredFlowTemperature'
    - 'CentralHeatingPump'
    - 'HeatingSwitch'
    - 'FlowTemperature'
    - 'Flame'
    - 'PowerEnergyConsumptionHeatingCircuit'
    - 'PowerEnergyConsumptionHotWaterCircuit'
    - 'RoomThermostat'
    - 'HeatingPartLoad'

Anything in the logs that might be useful for us?

No response

Additional information

No response

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

ebusd documentation ebusd source (message by IssueLinks)

Mk4242 commented 3 years ago

Commit #49524 merged recently contains in homeassistant/components/ebusd/const.py on line 226 a 6th attribute (should be 5) which prevents the sensor from initialising. Removing line 226 fixes the issue, tested locally.