Merge pull request #139 from Jalle19/ha-sensors

Expose "exhaustAirTemperatureBeforeHeatRecovery" and "returnWaterTemp… …erature" sensors to Home Assistant
Jalle19 · Feb 20, 2025 · 92bab2a · 92bab2a
2 parents 4eabe94 + c5cfd7f
commit 92bab2a
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -165,6 +165,9 @@ in Home Assistant automatically through the MQTT integration. The following enti
   state require a restart of the unit to resume normal operation. However, the unit cannot be restarted via Modbus 
   (at least not officially), so it's best to acknowledge such alarms manually.
 
+* Some readings may be nonsensical depending on the exact ventilation unit used, e.g. "Exhaust air temperature 
+  (before heat recovery)" can erroneously show -40 °C.
+
 ## Troubleshooting
 
 See [docs/CONNECTION.md](./docs/CONNECTION.md)

diff --git a/app/enervent.ts b/app/enervent.ts
@@ -193,7 +193,7 @@ export const ANALOG_INPUT_SENSOR_TYPES: AnalogInput[] = [
 
 type AnalogSensorReadings = Record<string, number>
 
-export enum AutomationHeatingType {
+export enum HeatingType {
     ED = 'ED/MD',
     EDW = 'EDW/MDW',
     EDX = 'EDX/MDX',
@@ -261,10 +261,10 @@ export const getAutomationAndHeatingTypeName = (heatingType: number): string =>
     // M prefix is used for units with MD automation
     return (
         [
-            AutomationHeatingType.ED, // prettier-hack
-            AutomationHeatingType.EDW,
-            AutomationHeatingType.EDX,
-            AutomationHeatingType.EDE,
+            HeatingType.ED, // prettier-hack
+            HeatingType.EDW,
+            HeatingType.EDX,
+            HeatingType.EDE,
         ][heatingType] || 'unknown'
     )
 }

diff --git a/app/homeassistant.ts b/app/homeassistant.ts
@@ -10,6 +10,7 @@ import {
 import { createLogger } from './logger'
 import {
     AlarmDescription,
+    HeatingType,
     AutomationType,
     AVAILABLE_ALARMS,
     createModelNameString,
@@ -28,6 +29,7 @@ export const configureMqttDiscovery = async (modbusClient: ModbusRTU, mqttClient
     const modbusDeviceInformation = await getDeviceInformation(modbusClient)
     const softwareVersion = modbusDeviceInformation.softwareVersion
     const automationType = modbusDeviceInformation.automationType
+    const heatingType = modbusDeviceInformation.heatingTypeInstalled
     const modelName = createModelNameString(modbusDeviceInformation)
     const deviceIdentifier = createDeviceIdentifierString(modbusDeviceInformation)
 
@@ -69,6 +71,18 @@ export const configureMqttDiscovery = async (modbusClient: ModbusRTU, mqttClient
             'exhaustAirTemperature',
             'Exhaust air temperature'
         ),
+        'exhaustAirTemperatureBeforeHeatRecovery': createTemperatureSensorConfiguration(
+            configurationBase,
+            'exhaustAirTemperatureBeforeHeatRecovery',
+            'Exhaust air temperature (before heat recovery)',
+            { 'enabled_by_default': heatingType !== HeatingType.EDW }
+        ),
+        'returnWaterTemperature': createTemperatureSensorConfiguration(
+            configurationBase,
+            'returnWaterTemperature',
+            'Return water temperature',
+            { 'enabled_by_default': heatingType === HeatingType.EDW }
+        ),
         'wasteAirTemperature': createTemperatureSensorConfiguration(
             configurationBase,
             'wasteAirTemperature',

diff --git a/app/modbus.ts b/app/modbus.ts
@@ -22,7 +22,7 @@ import {
     Readings,
     Settings,
     AlarmStatus,
-    AutomationHeatingType,
+    HeatingType,
 } from './enervent'
 import ModbusRTU from 'modbus-serial'
 import { ReadCoilResult, ReadRegisterResult } from 'modbus-serial/ModbusRTU'
@@ -195,7 +195,7 @@ export const getReadings = async (modbusClient: ModbusRTU): Promise<Readings> =>
     // While the value stored in both registers are always identical we publish two separate
     // sensors to avoid confusion.
     result = await mutex.runExclusive(async () => tryReadHoldingRegisters(modbusClient, 11, 2))
-    if (deviceInformation.heatingTypeInstalled === AutomationHeatingType.EDW) {
+    if (deviceInformation.heatingTypeInstalled === HeatingType.EDW) {
         readings = {
             ...readings,
             'returnWaterTemperature': parseTemperature(result.data[1]),