-
Notifications
You must be signed in to change notification settings - Fork 518
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ME81 Thermostat Valve State is missing #1709
Comments
The valve state is not missing, it is mapped to hvac_action. |
If you attach the diagnostics for the device, I might be able to see more clearly why the hvac_action is not showing as expected. |
Hello make-all, Thanks! `{
}, |
OK, I see now. There is an error in specifying the conditions for hvac_action - it has a default case, but conditions do not actually support that. It seems to be a common error affecting quite a few other conditional mappings in other device configs too. |
Defaults for conditions have to be set one level up in the mapping, as the conditions only look for matches, not keep track of defaults. There may be some side-effect to changing this, and setting one level up is an acceptable workaround. In read-write situations that need to control which options are set for a specific value, it is sometimes better to list all conditions rather than use a default. Main effect is on climate hvac_action, swing_mode where more than just two options for vertical and horizontal are available, cover actions where opened/opening, closed/closing need to distinguished based on position, and complex logic to handle optional vacuum activate and pause switches. Issue #1709
Hello, i just installed the new version, but i can´t font the Thermostat Valve State |
Sorry, but I cannot tell anything from that. Please attach diagnostics as before. |
Hello Horst, The Valve State is visible, but only when valve is open. Then you will see hvac_action: heating. When valve is closed it will disappear. When you use the entity in a mushroom climate card, it works. The card shows you when the valve is open an extra symbol. I got an other issue, in my log I have this error: 2024-04-05 14:35:47.706 WARNING (MainThread) [homeassistant.components.climate] Entity None (<class 'custom_components.tuya_local.climate.TuyaLocalClimate'>) implements HVACMode(s): off, auto, heat and therefore implicitly supports the turn_on/turn_off methods without setting the proper ClimateEntityFeature. Please create a bug report at https://github.com/make-all/tuya-local/issues This perhaps have to to with errors made for id:1 and id:2 id:1 switches the thermostat on or off, the display is switched on or off. hvac_mode should actually be on or off id:2 allows you to choose between manual setting or automatic. With automatic, the programmed program is followed. Auto should therefore not be in hvac_mode, but should be a separate selector switch, such as: childlock So: manual/auto |
Defaults for conditions have to be set one level up in the mapping, as the conditions only look for matches, not keep track of defaults. There may be some side-effect to changing this, and setting one level up is an acceptable workaround. In read-write situations that need to control which options are set for a specific value, it is sometimes better to list all conditions rather than use a default. Main effect is on climate hvac_action, swing_mode where more than just two options for vertical and horizontal are available, cover actions where opened/opening, closed/closing need to distinguished based on position, and complex logic to handle optional vacuum activate and pause switches. Issue make-all#1709
The ME81 Thermostat is working fine, but the valve state is missing
thanks
{ "code": "valve_state", "name": "阀门状态", "type": "Enum", "values": "{\"range\":[\"open\",\"close\"]}"
The text was updated successfully, but these errors were encountered: