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

Möhlenhoff Alpha 2 integration not working properly after upgrading to 2023.12 #105307

Closed
fjeldbo opened this issue Dec 8, 2023 · 3 comments · Fixed by #105437
Closed

Möhlenhoff Alpha 2 integration not working properly after upgrading to 2023.12 #105307

fjeldbo opened this issue Dec 8, 2023 · 3 comments · Fixed by #105437

Comments

@fjeldbo
Copy link

fjeldbo commented Dec 8, 2023

The problem

No devices (Möhlenhoff base stations) are shown (missing 3 of 3).
Most entities are available, but all control valve entities and battery status entities from one of the base stations are missing (EZR01716D in the log data below).
All climate entities are available and temperatures are updating, so there is contact with the base stations.
Scripts for turning heat on and off are not working, because the devices are missing.

What version of Home Assistant Core has the issue?

core-2023.12.0

What was the last working version of Home Assistant Core?

core-2023.10._

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Möhlenhoff Alpha 2

Link to integration documentation on our website

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

Diagnostics information

No response

Example YAML snippet

SCRIPT SHOULD LOOK LIKE THIS:
set_heating_to_day_mode:
  alias: Set heating to day mode
  sequence:
  - device_id: fe404d01be7b11a05e9ccd915c6f21cf
    domain: climate
    entity_id: climate.kjokken
    type: set_preset_mode
    preset_mode: day
  - device_id: fe404d01be7b11a05e9ccd915c6f21cf
    domain: climate
    entity_id: climate.stue
    type: set_preset_mode
    preset_mode: day
  mode: single

SCRIPT NOW LOOKS LIKE THIS:
alias: Set heating to day mode
sequence:
  - device_id: ""
    domain: ""
    entity_id: ""
  - device_id: ""
    domain: ""
    entity_id: ""
mode: single

Anything in the logs that might be useful for us?

2023-12-07 22:35:54.905 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 497, in async_add_entities
    tasks = [
            ^
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 497, in <listcomp>
    tasks = [
            ^
  File "/usr/src/homeassistant/homeassistant/components/moehlenhoff_alpha2/sensor.py", line 25, in <genexpr>
    Alpha2HeatControlValveOpeningSensor(coordinator, heat_control_id)
  File "/usr/src/homeassistant/homeassistant/components/moehlenhoff_alpha2/sensor.py", line 48, in __init__
    heat_area = self.coordinator.data["heat_areas"][heat_control["_HEATAREA_ID"]]
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'EZR01716D:6'
2023-12-07 22:35:54.908 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 497, in async_add_entities
    tasks = [
            ^
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 497, in <listcomp>
    tasks = [
            ^
  File "/usr/src/homeassistant/homeassistant/components/moehlenhoff_alpha2/binary_sensor.py", line 27, in <genexpr>
    Alpha2IODeviceBatterySensor(coordinator, io_device_id)
  File "/usr/src/homeassistant/homeassistant/components/moehlenhoff_alpha2/binary_sensor.py", line 47, in __init__
    heat_area = self.coordinator.data["heat_areas"][io_device["_HEATAREA_ID"]]
                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^
KeyError: 'EZR01716D:6'

Additional information

No response

@home-assistant
Copy link

home-assistant bot commented Dec 8, 2023

Hey there @j-a-n, mind taking a look at this issue as it has been labeled with an integration (moehlenhoff_alpha2) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of moehlenhoff_alpha2 can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign moehlenhoff_alpha2 Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


moehlenhoff_alpha2 documentation
moehlenhoff_alpha2 source
(message by IssueLinks)

@j-a-n
Copy link
Contributor

j-a-n commented Dec 10, 2023

The XML from the alpha2 xml interface appears to contain a valve opening sensor and a battery sensor that refer to a heat area that does not exist.
I will change the integration so that the sensors are ignored in this case.

@fjeldbo
Copy link
Author

fjeldbo commented Dec 11, 2023

Thank you, @j-a-n,
I found out that there is something wrong with the mentioned base station (EZR01716D). One of the room sensors is missing, i.e. only 5 of 6 rooms are showing in EZR manager, even after re-connecting the room control unit. The control is connected and is controlling the actuator, but the room will not show in EZR manager, and consequently causing the error with HA. I will try to re-start the base station and possibly do a factory reset.
Anyway, thank you for including the missing sensor check, and thank you for the integration! Much appreciated.

@github-actions github-actions bot locked and limited conversation to collaborators Jan 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
2 participants