feat(thermostat): configurable min/max temperature limits#82
Conversation
…ure limit listeners
…ty, and validation Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Expand target_temperature_min/max capability options before calling setCapabilityValue so values outside the built-in 4–35 range are accepted. Also call setCapabilityOptions on target_temperature before setCapabilityValue so the slider validation uses the updated range. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Add ga_temperature_min and ga_temperature_max input fields to the thermostat fieldset in the select_groupaddresses pairing screen, and include their values in the device save handler. Add i18n strings for all 13 supported locales. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ttherbrink
left a comment
ttherbrink
left a comment
There was a problem hiding this comment.
I see the use of having a configurable min/max. However having 3 points where this value can be edited is overkill in my opinion.
I think that only a setting is enough as this is normally not a value that is adjusted regularly (for example daily).
What is the benefit from reading this value from an knx adress?
|
Also i know it has been a while. if you are not interested anymore in finishing this PR let me know |
|
This feature is always required. |
2 participants
Summary
min_temperature(default 4°C) andmax_temperature(default 35°C) number fields, allowing users to configure the target temperature slider rangega_temperature_min/ga_temperature_maxKNX group address fields (DPT9.1, read-only) that dynamically override the static limits at runtime when telegrams arrive — KNX values take precedencetarget_temperature_minandtarget_temperature_maxas Homey capabilities, making current limits visible in the device UI and available in flowsMotivation
Fixes #79 — thermostats used for hot water regulation or other non-HVAC use cases need a wider temperature range (e.g. up to 70°C) than the hardcoded 4–35°C default. KNX devices like the OT-box can also send setpoint limits over the bus.
Test Plan
min=4/max=35defaultsmax_temperatureto70→ confirm target temperature slider extends to 70°Ctarget_temperature_minandtarget_temperature_maxappear as capabilities on the device cardga_temperature_maxwith a valid KNX GA → send a DPT9.1 telegram (e.g. 65°C) → confirm slider max updatesga_temperature_maxand changemax_temperaturein settings → confirm static value is applied again🤖 Generated with Claude Code