[Technical Support] General discussion regarding Tuya TRVs - for people implementing the quirks

Hi,

As @MattWestb and @jacekk015 suggested I open a technical topic for all the tech questions regarding the implementation of the quirks for Tuya TRV.

And to start off here is a first one: Changing the range of available temperature settings. For current Saswell quirk (https://github.com/zigpy/zha-device-handlers/blob/dev/zhaquirks/tuya/ts0601_trv_sas.py) the temp range is set to 7-30 (default) however the device uses 5 -30. The setting used in quirk is not being applied.

The way I managed to change this is:

class SaswellThermostatCluster(TuyaThermostatCluster):
    """Thermostat cluster for Tuya thermostats."""

    #    cluster_id = Thermostat.cluster_id

    def __init__(self, *args, **kwargs):
        """Init."""
        super().__init__(*args, **kwargs)
        self._update_attribute(self.attridx["min_heat_setpoint_limit"], 500)
        self._update_attribute(self.attridx["max_heat_setpoint_limit"], 3000)

this is not working:

 _CONSTANT_ATTRIBUTES = {
        MIN_HEAT_SETPOINT_ATTR: 500,
        MAX_HEAT_SETPOINT_ATTR: 3000,
        CTRL_SEQ_OF_OPER_ATTR: Thermostat.ControlSequenceOfOperation.Heating_Only,  # the device supports heating mode
    }

Do you know of any other/better way of doing this?

PS. Check if this is working correctly for Moes/Beca TRV as those seem to read those values from TRV but, are those values implemented in HA?

[jacekk015]

All tuya TRVs is being looked on only the original Moes is not completely rewritten but is also in the pipe if all is going well.

That’s actually wrong since 20 minutes or so: https://github.com/zigpy/zha-device-handlers/issues/1178#issuecomment-980294492 Rewritten Moes is in above comment. Next is Maxsmart cosmetics + schedule. Next in queue is: TS0601 _TZE200_a4bpgplm tuya trv - this weekend for sure. https://github.com/zigpy/zha-device-handlers/issues/1159#

[jacekk015]

@PLTorrent Nicely coded, nice code re-usage. I feel a bit lazy looking now 😉 I’m tuning my Maxsmart with 126 schedule attributes. TRV -> HA already working. Need to create reverse communication and it should be the end.