fix(sensor): 🐛 hide exhaust air temperature when the sensor is unwired - #808
Merged
Merged
Conversation
- gate VENTILATION_EXHAUST_AIR_TEMPERATURE on entity_active_formula "!= 5.0" - two LWC407s (#729, #807) run a ventilation module without an exhaust sensor; the register sits on the controller's 5.0 placeholder and the controller's own display never shows it - supply air deliberately not gated: 5.0 is a plausible reading there and no unwired supply channel has been reported - test covers both channel assignments Resolves #807 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
- the entity table listed the sensor unconditionally; it is now gated on the register not reading the 5 °C unwired-sensor placeholder (#807) Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔍 What this fixes
A ventilation module can run without an exhaust-air sensor. Two LWC407s (#729, #807) then hold
ID_WEB_Temp_Lueftung_Abluft(C0160) on the controller's unwired-sensor placeholder 5.0 — constant for days while supply air drifts normally (#729: one state in 18 h of recorder history against 229 on supply; #807: 5.0 in both dumps while Zuluft went 25.1 → 22.4), and never shown on the controller's own display. The integration published it as a real 5 °C temperature.✨ Changes
sensor_entities_predefined.py:VENTILATION_EXHAUST_AIR_TEMPERATUREgetsentity_active_formula="!= 5.0", so the sensor is not created while the register reads the placeholder. Same shape as the!= -50.0gate on the heat-source outlet temperature.has_ventilation), and 75.0 has never been seen on this channel.has_ventilationalready treats 5.0 as absent and latches on the live supply channel).README.md: entity table now says the sensor only exists when an exhaust sensor is wired.🧪 Tests
test_exhaust_air_temperature_needs_a_wired_sensor: on the Parameter Ablufttemperatur in der Lüftung hat keinen Sensor #807 shape (Zuluft 22.4 / Abluft 5.0) the ventilation device exists, the exhaust sensor does not, the supply sensor does; on the mirror (Zuluft 5.0 / Abluft 21.2) both exist. Runs through the real coordinator and descriptions.If your ventilation module has no exhaust-air sensor,
sensor.<prefix>_ventilation_exhaust_air_temperature(which showed a constant 5 °C) is no longer created. The existing entity stays in the entity registry as "no longer provided by the integration" — delete it there. Units with a wired exhaust sensor are unaffected.Resolves #807
🤖 Generated with Claude Code