Evohome integration reporting temperatures of 128 degrees celsius

[scstraus]

The problem

When batteries are low in TRV's, the evohome component is reporting 128 degree C temperatures

Environment

hass 107.7, Ubuntu server, hass.io

• Home Assistant Core release with the issue:
  107.7
• Last working Home Assistant Core release (if known):
  ?
• Operating environment (Home Assistant/Supervised/Docker/venv):
  Supervised
• Integration causing this issue:
  evohome
• Link to integration documentation on our website:
  https://www.home-assistant.io/integrations/evohome/

Problem-relevant configuration.yaml

evohome:

  username: !secret my_email
  password: !secret hass_pass
  location_idx: 0
  scan_interval: 360 

Traceback/Error logs

2020-04-21 12:39:37 DEBUG (MainThread) [homeassistant.components.evohome] Temperatures = {'4217766': 23.01, '4217767': 22.21, '4217774': 17.81, '4217771': 20.78, '4217768': 21.21, '4187036': 23.73, '4217776': 23.61, '4217773': 22.76, '4217772': 22.69, '4217775': 21.97, '4217769': 23.34, '4217770': 22.68}

2020-04-21 12:45:38 DEBUG (MainThread) [homeassistant.components.evohome] Temperatures = {'4217766': 22.92, '4217767': 21.64, '4217774': 17.81, '4217771': 20.78, '4217768': 128.0, '4187036': 23.63, '4217776': 23.61, '4217773': 22.76, '4217772': 22.69, '4217775': 21.97, '4217769': 23.34, '4217770': 22.68}

2020-04-21 13:09:41 DEBUG (MainThread) [homeassistant.components.evohome] Status = {'locationId': '3184115', 'gateways': [{'gatewayId': '2937956', 'temperatureControlSystems': [{'systemId':  '4187037', 'zones': [{'zoneId': '4187036', 'temperatureStatus': {'temperature': 23.5, 'isAvailable': True}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 20.0, 'setpointM
ode': 'TemporaryOverride', 'until': '2020-04-21T22:29:00Z'}, 'name': 'Living Room'}, {'zoneId': '4217766', 'temperatureStatus': {'temperature': 23.0, 'isAvailable': True}, 'activeFaults': [
{'faultType': 'TempZoneActuatorLowBattery', 'since': '2020-04-15T03:11:03'}], 'setpointStatus': {'targetHeatTemperature': 23.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Dining Room'}, {'
zoneId': '4217767', 'temperatureStatus': {'temperature': 21.5, 'isAvailable': True}, 'activeFaults': [{'faultType': 'TempZoneSensorLowBattery', 'since': '2020-04-14T22:47:28'}, {'faultType'
: 'TempZoneActuatorLowBattery', 'since': '2020-04-15T01:46:57'}], 'setpointStatus': {'targetHeatTemperature': 21.5, 'setpointMode': 'FollowSchedule'}, 'name': 'Entrance'}, {'zoneId': '4217768', 'temperatureStatus': {'isAvailable': False}, 'activeFaults': [{'faultType': 'TempZoneSensorCommunicationLost', 'since': '2020-04-21T10:41:38'}, {'faultType': 'TempZoneActuatorCommunicationLost', 'since': '2020-04-21T10:58:39'}], 'setpointStatus': {'targetHeatTemperature': 22.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Laundry room'}, {'zoneId': '4217769', 'temperatureStatus': {'isAvailable': False}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 22.0, 'setpointMode': 'FollowSchedule'}, 'name': "Sebastian's Room"}, {'zoneId': '4217770','temperatureStatus': {'isAvailable': False}, 'activeFaults': [{'faultType': 'TempZoneSensorCommunicationLost', 'since': '2020-04-21T10:52:40'}], 'setpointStatus': {'targetHeatTemperature':23.0, 'se tpointMode': 'TemporaryOverride', 'until': '2020-04-21T14:10:00Z'}, 'name': "Sophie's Room"}, {'zoneId': '4217771', 'temperatureStatus': {'temperature': 21.0, 'isAvailable': True}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 21.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Guest Bedroom'}, {'zoneId': '4217772', 'temperatureStatus': {'temperature': 23.0, 'isAvailable': True}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 23.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Master Closet'}, {'zoneId': '4217773', 'temperatureStatus': {'temperature': 22.5, 'isAvailable': True}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 21.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Master Bedroom'}, {'zoneId': '4217774', 'temperatureStatus': {'temperature': 18.0, 'isAvailable': True}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 20.0, 'setpointMode': 'FollowSchedul
e'}, 'name': 'Garage'}, {'zoneId': '4217775', 'temperatureStatus': {'temperature': 22.0, 'isAvailable': True}, 'activeFaults': [{'faultType': 'TempZoneActuatorLowBattery', 'since': '2020-04-08T09:21:12'}, {'faultType': 'TempZoneSensorLowBattery', 'since': '2020-04-15T09:21:07'}], 'setpointStatus': {'targetHeatTemperature': 23.0, 'setpointMode': 'TemporaryOverride', 'until': '2020-04-21T19:40:00Z'}, 'name': 'Office & Library'}, {'zoneId': '4217776', 'temperatureStatus': {'temperature': 23.5, 'isAvailable': True}, 'activeFaults': [], 'setpointStatus': {'targetHeatTemperature': 22.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Master Bath'}], 'activeFaults': [], 'systemModeStatus': {'mode': 'DayOff', 'isPermanent': True}}], 'activeFaults': []}]}

Additional information

As discussed here

[probot-home-assistant]

Hey there @zxdavb, mind taking a look at this issue as its been labeled with a integration (evohome) you are listed as a codeowner for? Thanks!

[zxdavb]

Well, the answer is in the logs, firstly:

2020-04-21 12:45:38 DEBUG (MainThread) [homeassistant.components.evohome] 
Temperatures = 
{
  ...
  '4217768': 128.0, 
  ...
}

... and also:

{
  'zoneId': '4217768', 
  'temperatureStatus': {'isAvailable': False}, 
  'activeFaults': [
    {'faultType': 'TempZoneSensorCommunicationLost',   'since': '2020-04-21T10:41:38'}, 
    {'faultType': 'TempZoneActuatorCommunicationLost', 'since': '2020-04-21T10:58:39'}
  ], 
  'setpointStatus': {'targetHeatTemperature': 22.0, 'setpointMode': 'FollowSchedule'}, 'name': 'Laundry room'
},

A proposed solution (not best-practice code, tho'):

    def current_temperature(self) -> Optional[float]:
        """Return the current temperature of a Zone."""
        if self._evo_broker.temps:
            if self._evo_broker.temps[self._evo_device.zoneId] != 128:
                return self._evo_broker.temps[self._evo_device.zoneId]

        if not self._evo_device.temperatureStatus["isAvailable"]:
            return None

        return self._evo_device.temperatureStatus["temperature"]

[zxdavb]

This is a bit of an edge-case, I (am a bit busy, but) will submit a PR ASAP.

[stale]

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

[scstraus]

@zxdavb did this one ever make it in? I still see the problem sometimes..

[zxdavb]

OK, the issue is when the temp is valid for the v2 API, but the v1 API temp is used, but it isn't valid, so is set to 128.