Skip to content
This repository has been archived by the owner on Oct 4, 2021. It is now read-only.

V2.2.2 Selected flow temperature is missing #766

Closed
KNXBroker opened this issue May 11, 2021 · 2 comments
Closed

V2.2.2 Selected flow temperature is missing #766

KNXBroker opened this issue May 11, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@KNXBroker
Copy link

Bug description
Since V2.2.2 Selected flow temperatur is missing in Home Assistant.

Steps to reproduce
Home Assistant 2021.5.2
EMS-ESP V2.2.2
Boiler HT3
Device ID 0x08
Product ID 95
Version 18.11

Expected behavior
Selected flow temperatur should be available in HomeAssistant.

Device information

{
  "System": {
    "version": "2.2.2",
    "uptime": "000+00:07:38.403",
    "freemem": 42,
    "fragmem": 12
  },
  "Status": {
    "bus": "connected",
    "bus protocol": "HT3",
    "#telegrams received": 244,
    "#read requests sent": 74,
    "#write requests sent": 0,
    "#incomplete telegrams": 21,
    "#tx fails": 1,
    "rx line quality": 92,
    "tx line quality": 98,
    "#MQTT publish fails": 0,
    "#dallas sensors": 0,
    "#dallas fails": 0
  },
  "Devices": [
    {
      "type": "Boiler",
      "name": "HT3 (DeviceID:0x08, ProductID:95, Version:18.11)",
      "handlers": "0x10 0x11 0x14 0x15 0x1C 0x18 0x19 0x1A 0x35 0x16 0x33 0x34 0x26 0x2A"
    },
    {
      "type": "Thermostat",
      "name": "FW120 (DeviceID:0x10, ProductID:192, Version:53.02)",
      "handlers": "0xA3 0x06 0xA2 0x12 0x16F 0x170 0x171 0x172 0x165 0x166 0x167 0x168"
    },
    {
      "type": "Controller",
      "name": "HT3 (DeviceID:0x09, ProductID:95, Version:18.11)",
      "handlers": ""
    }
  ]
}

Additional context
I assume it is due to the new filtering functionality.

@KNXBroker KNXBroker added the bug Something isn't working label May 11, 2021
@proddy
Copy link
Collaborator

proddy commented May 12, 2021

is it only missing in HA?
Can you see it in the Web and Console?
Is it in the MQTT topic called boiler?
What is in the MQTT topic called homeassistant/sensor/ems-esp/boiler_selflowtemp?
Was it there in 2.2.1?

@MichaelDvP
Copy link
Collaborator

It's a typo in boiler.cpp line 136. I'll fix the dev.

MichaelDvP added a commit that referenced this issue May 12, 2021
@proddy proddy closed this as completed Jun 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants