New device: Moes , Thermoestatic Valve

Koenkk / zigbee2mqtt

Zigbee 🐝 to MQTT bridge 🌉, get rid of your proprietary Zigbee bridges 🔨

https://www.zigbee2mqtt.io

GNU General Public License v3.0

12.23k stars 1.69k forks source link

New device: Moes , Thermoestatic Valve #23206

Open egarijo opened 5 months ago

egarijo commented 5 months ago

Link

{"id":21,"type":"Router","ieeeAddr":"0xa4c138b618017288","nwkAddr":4156,"manufId":4417,"manufName":"_TZE204_qasjif9e","powerSource":"Mains (single phase)","modelId":"TS0601","epList":[1,242],"endpoints":{"1":{"profId":260,"epId":1,"devId":81,"inClusterList":[4,5,61184,0],"outClusterList":[25,10],"clusters":{"genBasic":{"attributes":{"65503":"\u0006\u0000\u0000\u0000e\u0007\u0000\u0000\u0000\u0012\u0006q�-\u0013\u0006q�-e\u0007q�-\u0012\u0005\u0000\u0000\u0000e\u0006\u0000\u0000\u0000\u0012","65506":56,"65508":1,"modelId":"TS0601","manufacturerName":"_TZE204_qasjif9e","powerSource":1,"zclVersion":3,"appVersion":74,"stackVersion":0,"hwVersion":1,"dateCode":""}}},"binds":[],"configuredReportings":[],"meta":{}},"242":{"profId":41440,"epId":242,"devId":97,"inClusterList":[],"outClusterList":[33],"clusters":{},"binds":[],"configuredReportings":[],"meta":{}}},"appVersion":74,"stackVersion":0,"hwVersion":1,"dateCode":"","zclVersion":3,"interviewCompleted":true,"meta":{},"lastSeen":1718784631633}

Database entry

Comments

I created the file T0601.js file following the instructions: the file is the following:

const {} = require('zigbee-herdsman-converters/lib/modernExtend'); // Add the lines below const fz = require('zigbee-herdsman-converters/converters/fromZigbee'); const tz = require('zigbee-herdsman-converters/converters/toZigbee'); const exposes = require('zigbee-herdsman-converters/lib/exposes'); const reporting = require('zigbee-herdsman-converters/lib/reporting'); const ota = require('zigbee-herdsman-converters/lib/ota'); const utils = require('zigbee-herdsman-converters/lib/utils'); const globalStore = require('zigbee-herdsman-converters/lib/store'); const e = exposes.presets; const ea = exposes.access;

const definition = { zigbeeModel: ['TS0601'], model: 'TRV FSZPJYDC', // Update this with the real model of the device (written on the device itself or product page) vendor: 'Moes', // Update this with the real vendor of the device (written on the device itself or product page) description: 'THERMOSTATE VALVE', // Description of the device, copy from vendor site. (only used for documentation and startup logging) extend: [], fromZigbee: [], // We will add this later toZigbee: [], // Should be empty, unless device can be controlled (e.g. lights, switches). exposes: [e.battery(), e.temperature(), e.valve_possition(), e.state()] // Defines what this device exposes, used for e.g. Home Assistant discovery and in the frontend };

module.exports = definition;

After that I have added the following lines on the configuration.yml

advanced: log_level: debug external_converters:

TS0601.js But still the same. Nothing happens.

External definition

const definition = {
    zigbeeModel: ['TS0601'],
    model: 'TS0601',
    vendor: '_TZE204_9mjy74mp',
    description: 'Automatically generated definition',
    extend: [],
    meta: {},
};

module.exports = definition;

LintHart commented 4 months ago

Hi @egarijo , did you ever get anywhere with these? I unfortunately bought a bunch of them without realising they're not supported. Also having trouble creating an external definition

egarijo commented 4 months ago

Hi. At the moment I can't do anything with that. I'm Waiting if with the time, somebody solves the problem. I have time to the next winter. Regards

El sáb, 27 jul 2024, 12:12, LintHart @.***> escribió:

Hi @egarijo https://github.com/egarijo , did you ever get anywhere with these? I unfortunately bought a bunch of them without realising they're not supported. Also having trouble creating an external definition

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2254102233, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6GO7IN7WTASAKWIA7TZONW7LAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJUGEYDEMRTGM . You are receiving this because you were mentioned.Message ID: @.***>

LintHart commented 4 months ago

I had some luck with learning how to work with DDFs in Phoscon as it has a builder and managed to adjust a template to read and control the temperature. I did figure out how to start an external definition but my skills aren't there yet to finish it.

I bought a Tuya hub and extracted all of the DPIDs in hope that we might be able to get some assistance :)

"1":"Mode", "2":"Target temperature", "3":"Current temperature", "6":"Working status", "7":"Window status", "8":"Open window", "12":"Child lock", "13":"Battery", "14":"Fault alarm", "15":"Min. limit temperature", "16":"Max. limit temperature", "17":"Heating schedule", "18":"Week program Tuesday", "19":"Week program Wednesday", "20":"Week program Thursday", "21":"Week program Friday", "22":"Week program Saturday", "23":"Week program Sunday", "101":"Room sensor calibration", "108":"Valve", "109":"Models", "110":"Motor thrust", "111":"Display brightness", "112":"Software version", "113":"Screen orientation", "114":"System mode", "115":"Switch deviation (energy-saving mode only)", "116":"Motor data"

micheltol commented 2 months ago

@egarijo @LintHart any luck? I'm in the same boat and pretty much a newbee in zigbee2mqtt-space 😦

hideintheclouds commented 2 months ago

I think that the device is a Tuya type, so a different converter should be created, according to the documentation here: https://www.zigbee2mqtt.io/advanced/support-new-devices/02_support_new_tuya_devices.html

I had the same issue with a Moes TRV (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee');                                                                                          
const tz = require('zigbee-herdsman-converters/converters/toZigbee');                                                                                            
const exposes = require('zigbee-herdsman-converters/lib/exposes');                                                                                               
const reporting = require('zigbee-herdsman-converters/lib/reporting');                                                                                           
const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend');                                                                                     
const e = exposes.presets;                                                                                                                                       
const ea = exposes.access;                                                                                                                                       
const tuya = require('zigbee-herdsman-converters/lib/tuya');                                                                                                     

const definition = {                                                                                                                                             
    // Since a lot of Tuya devices use the same modelID, but use different datapoints                                                                            
    // it's necessary to provide a fingerprint instead of a zigbeeModel                                                                                          
    fingerprint: [                                                                                                                                               
        {                                                                                                                                                        
            // The model ID from: Device with modelID 'TS0601' is not supported                                                                                  
            // You may need to add \u0000 at the end of the name in some cases                                                                                   
            modelID: 'TS0601',                                                                                                                                   
            // The manufacturer name from: Device with modelID 'TS0601' is not supported.                                                                        
            manufacturerName: '_TZE204_9mjy74mp',                                                                                                                
        },                                                                                                                                                       
    ],                                                                                                                                                           
    model: 'TRV801Z',                                                                                                                                            
    vendor: 'Moes',                                                                                                                                              
    description: 'Moes old TRV/zigbee model',                                                                                                                    
    fromZigbee: [tuya.fz.datapoints],                                                                                                                            
    toZigbee: [tuya.tz.datapoints],                                                                                                                              
    onEvent: tuya.onEventSetLocalTime,                                                                                                                           
    configure: tuya.configureMagicPacket,                                                                                                                        
exposes: [                                                                                                                                                       
            e.battery(),                                                                                                                                         
            e.child_lock(),                                                                                                                                      
            e.max_temperature(),                                                                                                                                 
            e.min_temperature(),                                                                                                                                 
            e.position(),                                                                                                                                        
            e.window_detection(),                                                                                                                                
            e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '),                                                      
            e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'),                                                        
            e                                                                                                                                                    
                .climate()                                                                                                                                       
                .withLocalTemperature(ea.STATE)                                                                                                                  
                .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET)                                                                              
                .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET)                                                                                     
                .withPreset(                                                                                                                                     
                    ['auto', 'manual', 'off', 'on'],                                                                                                             
                    'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' +                                                           
                        'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' +                              
                        'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' +
                        'ON - In this mode, the thermostat stays open ' +
                        'OFF - In this mode, the thermostat stays closed',
                )
                .withSystemMode(['auto', 'heat', 'off'], ea.STATE)
                .withRunningState(['idle', 'heat'], ea.STATE),
            ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'),
            e
                .enum('mode', ea.STATE_SET, ['comfort', 'eco'])
                .withDescription(
                    'Hysteresis - comfort > switches off/on exactly at reached ' +
                        'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%',
                ),
        ],
        meta: {
            tuyaDatapoints: [
                [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)],
                [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')],
                [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')],
                [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10],
                [3, 'local_temperature', tuya.valueConverter.divideBy10],
                [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})],
                [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})],
                [8, 'window_detection', tuya.valueConverter.onOff],
                [12, 'child_lock', tuya.valueConverter.lockUnlock],
                [13, 'battery', tuya.valueConverter.raw],
                [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})],
                [15, 'min_temperature', tuya.valueConverter.divideBy10],
                [16, 'max_temperature', tuya.valueConverter.divideBy10],
                [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)],
                [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)],
                [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)],
                [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)],
                [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)],
                [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)],
                [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)],
                [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1],
                [108, 'position', tuya.valueConverter.divideBy10],
                [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})],
            ],
        },

    extend: [
        // A preferred new way of extending functionality.
    ],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

egarijo commented 1 month ago

Thanks for your info. I am not an expert on that matter, but I will try to test it. For your informatin, I bougth the valve to Moes. But it seams must be manufacturing by Tuya.

Thanks again.

El mar, 1 oct 2024 a las 14:10, Adrian M @.***>) escribió:

I think that the device is a Tuya type, so a different converter should be created, according to the documentation here:

https://www.zigbee2mqtt.io/advanced/support-new-devices/02_support_new_tuya_devices.html

I had the same issue with a Moes TRV https://www.aliexpress.com/item/1005006068601747.html?spm=a2g0o.order_list.order_list_main.5.187518022PIULc (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart https://github.com/LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here https://github.com/Koenkk/zigbee-herdsman-converters/blob/13ca5a04abfb6e38a03aee0d3eb4737a18e33071/src/devices/tuya.ts#L4933C1-L4934C1

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee'); const tz = require('zigbee-herdsman-converters/converters/toZigbee'); const exposes = require('zigbee-herdsman-converters/lib/exposes'); const reporting = require('zigbee-herdsman-converters/lib/reporting'); const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend'); const e = exposes.presets; const ea = exposes.access; const tuya = require('zigbee-herdsman-converters/lib/tuya');

const definition = { // Since a lot of Tuya devices use the same modelID, but use different datapoints // it's necessary to provide a fingerprint instead of a zigbeeModel fingerprint: [ { // The model ID from: Device with modelID 'TS0601' is not supported // You may need to add \u0000 at the end of the name in some cases modelID: 'TS0601', // The manufacturer name from: Device with modelID 'TS0601' is not supported. manufacturerName: '_TZE204_9mjy74mp', }, ], model: 'TRV801Z', vendor: 'Moes', description: 'Moes old TRV/zigbee model', fromZigbee: [tuya.fz.datapoints], toZigbee: [tuya.tz.datapoints], onEvent: tuya.onEventSetLocalTime, configure: tuya.configureMagicPacket, exposes: [ e.battery(), e.child_lock(), e.max_temperature(), e.min_temperature(), e.position(), e.window_detection(), e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '), e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'), e .climate() .withLocalTemperature(ea.STATE) .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET) .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET) .withPreset( ['auto', 'manual', 'off', 'on'], 'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' + 'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' + 'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' + 'ON - In this mode, the thermostat stays open ' + 'OFF - In this mode, the thermostat stays closed', ) .withSystemMode(['auto', 'heat', 'off'], ea.STATE) .withRunningState(['idle', 'heat'], ea.STATE), ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'), e .enum('mode', ea.STATE_SET, ['comfort', 'eco']) .withDescription( 'Hysteresis - comfort > switches off/on exactly at reached ' + 'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%', ), ], meta: { tuyaDatapoints: [ [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)], [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')], [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')], [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10], [3, 'local_temperature', tuya.valueConverter.divideBy10], [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})], [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})], [8, 'window_detection', tuya.valueConverter.onOff], [12, 'child_lock', tuya.valueConverter.lockUnlock], [13, 'battery', tuya.valueConverter.raw], [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})], [15, 'min_temperature', tuya.valueConverter.divideBy10], [16, 'max_temperature', tuya.valueConverter.divideBy10], [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)], [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)], [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)], [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)], [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)], [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)], [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)], [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1], [108, 'position', tuya.valueConverter.divideBy10], [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})], ], },
extend: [
    // A preferred new way of extending functionality.
],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2385609418, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6DZUTZSABZHAC4SDLLZZKGLPAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGYYDSNBRHA . You are receiving this because you were mentioned.Message ID: @.***>

-- Eugenio Garijo Azpeitia Tfn. 623214313

egarijo commented 1 month ago

Hi, at the moment I have not chance. I will try to test the propossal by @Adrian M and test it. But I will need time. Regards

El mar, 17 sept 2024 a las 21:48, Michel Tol @.***>) escribió:

@egarijo https://github.com/egarijo @LintHart https://github.com/LintHart any luck? I'm in the same boat and pretty much a newbee in zigbee2mqtt-space 😦

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2356762545, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6D34MC7BWXP3DVKLO3ZXCBRHAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJWG43DENJUGU . You are receiving this because you were mentioned.Message ID: @.***>

-- Eugenio Garijo Azpeitia Tfn. 623214313

monteship commented 1 month ago

I had the same issue with a Moes TRV (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee');                                                                                          
const tz = require('zigbee-herdsman-converters/converters/toZigbee');                                                                                            
const exposes = require('zigbee-herdsman-converters/lib/exposes');                                                                                               
const reporting = require('zigbee-herdsman-converters/lib/reporting');                                                                                           
const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend');                                                                                     
const e = exposes.presets;                                                                                                                                       
const ea = exposes.access;                                                                                                                                       
const tuya = require('zigbee-herdsman-converters/lib/tuya');                                                                                                     

const definition = {                                                                                                                                             
    // Since a lot of Tuya devices use the same modelID, but use different datapoints                                                                            
    // it's necessary to provide a fingerprint instead of a zigbeeModel                                                                                          
    fingerprint: [                                                                                                                                               
        {                                                                                                                                                        
            // The model ID from: Device with modelID 'TS0601' is not supported                                                                                  
            // You may need to add \u0000 at the end of the name in some cases                                                                                   
            modelID: 'TS0601',                                                                                                                                   
            // The manufacturer name from: Device with modelID 'TS0601' is not supported.                                                                        
            manufacturerName: '_TZE204_9mjy74mp',                                                                                                                
        },                                                                                                                                                       
    ],                                                                                                                                                           
    model: 'TRV801Z',                                                                                                                                            
    vendor: 'Moes',                                                                                                                                              
    description: 'Moes old TRV/zigbee model',                                                                                                                    
    fromZigbee: [tuya.fz.datapoints],                                                                                                                            
    toZigbee: [tuya.tz.datapoints],                                                                                                                              
    onEvent: tuya.onEventSetLocalTime,                                                                                                                           
    configure: tuya.configureMagicPacket,                                                                                                                        
exposes: [                                                                                                                                                       
            e.battery(),                                                                                                                                         
            e.child_lock(),                                                                                                                                      
            e.max_temperature(),                                                                                                                                 
            e.min_temperature(),                                                                                                                                 
            e.position(),                                                                                                                                        
            e.window_detection(),                                                                                                                                
            e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '),                                                      
            e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'),                                                        
            e                                                                                                                                                    
                .climate()                                                                                                                                       
                .withLocalTemperature(ea.STATE)                                                                                                                  
                .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET)                                                                              
                .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET)                                                                                     
                .withPreset(                                                                                                                                     
                    ['auto', 'manual', 'off', 'on'],                                                                                                             
                    'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' +                                                           
                        'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' +                              
                        'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' +
                        'ON - In this mode, the thermostat stays open ' +
                        'OFF - In this mode, the thermostat stays closed',
                )
                .withSystemMode(['auto', 'heat', 'off'], ea.STATE)
                .withRunningState(['idle', 'heat'], ea.STATE),
            ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'),
            e
                .enum('mode', ea.STATE_SET, ['comfort', 'eco'])
                .withDescription(
                    'Hysteresis - comfort > switches off/on exactly at reached ' +
                        'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%',
                ),
        ],
        meta: {
            tuyaDatapoints: [
                [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)],
                [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')],
                [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')],
                [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10],
                [3, 'local_temperature', tuya.valueConverter.divideBy10],
                [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})],
                [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})],
                [8, 'window_detection', tuya.valueConverter.onOff],
                [12, 'child_lock', tuya.valueConverter.lockUnlock],
                [13, 'battery', tuya.valueConverter.raw],
                [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})],
                [15, 'min_temperature', tuya.valueConverter.divideBy10],
                [16, 'max_temperature', tuya.valueConverter.divideBy10],
                [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)],
                [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)],
                [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)],
                [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)],
                [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)],
                [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)],
                [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)],
                [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1],
                [108, 'position', tuya.valueConverter.divideBy10],
                [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})],
            ],
        },

    extend: [
        // A preferred new way of extending functionality.
    ],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

Confirm as working solution

egarijo commented 1 month ago

Hi, I can't confirm anything at the moment.

El vie, 4 oct 2024, 18:07, Misha Ship @.***> escribió:

I think that the device is a Tuya type, so a different converter should be created, according to the documentation here: https://www.zigbee2mqtt.io/advanced/support-new-devices/02_support_new_tuya_devices.html

I had the same issue with a Moes TRV https://www.aliexpress.com/item/1005006068601747.html?spm=a2g0o.order_list.order_list_main.5.187518022PIULc (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart https://github.com/LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here https://github.com/Koenkk/zigbee-herdsman-converters/blob/13ca5a04abfb6e38a03aee0d3eb4737a18e33071/src/devices/tuya.ts#L4933C1-L4934C1

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee'); const tz = require('zigbee-herdsman-converters/converters/toZigbee'); const exposes = require('zigbee-herdsman-converters/lib/exposes'); const reporting = require('zigbee-herdsman-converters/lib/reporting'); const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend'); const e = exposes.presets; const ea = exposes.access; const tuya = require('zigbee-herdsman-converters/lib/tuya');

const definition = { // Since a lot of Tuya devices use the same modelID, but use different datapoints // it's necessary to provide a fingerprint instead of a zigbeeModel fingerprint: [ { // The model ID from: Device with modelID 'TS0601' is not supported // You may need to add \u0000 at the end of the name in some cases modelID: 'TS0601', // The manufacturer name from: Device with modelID 'TS0601' is not supported. manufacturerName: '_TZE204_9mjy74mp', }, ], model: 'TRV801Z', vendor: 'Moes', description: 'Moes old TRV/zigbee model', fromZigbee: [tuya.fz.datapoints], toZigbee: [tuya.tz.datapoints], onEvent: tuya.onEventSetLocalTime, configure: tuya.configureMagicPacket, exposes: [ e.battery(), e.child_lock(), e.max_temperature(), e.min_temperature(), e.position(), e.window_detection(), e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '), e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'), e .climate() .withLocalTemperature(ea.STATE) .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET) .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET) .withPreset( ['auto', 'manual', 'off', 'on'], 'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' + 'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' + 'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' + 'ON - In this mode, the thermostat stays open ' + 'OFF - In this mode, the thermostat stays closed', ) .withSystemMode(['auto', 'heat', 'off'], ea.STATE) .withRunningState(['idle', 'heat'], ea.STATE), ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'), e .enum('mode', ea.STATE_SET, ['comfort', 'eco']) .withDescription( 'Hysteresis - comfort > switches off/on exactly at reached ' + 'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%', ), ], meta: { tuyaDatapoints: [ [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)], [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')], [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')], [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10], [3, 'local_temperature', tuya.valueConverter.divideBy10], [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})], [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})], [8, 'window_detection', tuya.valueConverter.onOff], [12, 'child_lock', tuya.valueConverter.lockUnlock], [13, 'battery', tuya.valueConverter.raw], [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})], [15, 'min_temperature', tuya.valueConverter.divideBy10], [16, 'max_temperature', tuya.valueConverter.divideBy10], [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)], [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)], [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)], [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)], [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)], [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)], [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)], [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1], [108, 'position', tuya.valueConverter.divideBy10], [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})], ], },
extend: [
    // A preferred new way of extending functionality.
],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

Confirm as working solution

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2394030874, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6HIOC5V6JAS3N52KTLZZ24NPAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUGAZTAOBXGQ . You are receiving this because you were mentioned.Message ID: @.***>

egarijo commented 1 month ago

Hi Adrian. Today I had time to test. Now the device appears as "supported". [image: image.png]

But if I check the values exposed, all of them appears without any information.

[image: image.png]

I follow all the steps, but someone must be missing, and I can't find it. ¿Do you have any suggestion? Regards

El mar, 1 oct 2024 a las 14:10, Adrian M @.***>) escribió:

I think that the device is a Tuya type, so a different converter should be created, according to the documentation here:

https://www.zigbee2mqtt.io/advanced/support-new-devices/02_support_new_tuya_devices.html

I had the same issue with a Moes TRV https://www.aliexpress.com/item/1005006068601747.html?spm=a2g0o.order_list.order_list_main.5.187518022PIULc (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart https://github.com/LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here https://github.com/Koenkk/zigbee-herdsman-converters/blob/13ca5a04abfb6e38a03aee0d3eb4737a18e33071/src/devices/tuya.ts#L4933C1-L4934C1

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee'); const tz = require('zigbee-herdsman-converters/converters/toZigbee'); const exposes = require('zigbee-herdsman-converters/lib/exposes'); const reporting = require('zigbee-herdsman-converters/lib/reporting'); const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend'); const e = exposes.presets; const ea = exposes.access; const tuya = require('zigbee-herdsman-converters/lib/tuya');

const definition = { // Since a lot of Tuya devices use the same modelID, but use different datapoints // it's necessary to provide a fingerprint instead of a zigbeeModel fingerprint: [ { // The model ID from: Device with modelID 'TS0601' is not supported // You may need to add \u0000 at the end of the name in some cases modelID: 'TS0601', // The manufacturer name from: Device with modelID 'TS0601' is not supported. manufacturerName: '_TZE204_9mjy74mp', }, ], model: 'TRV801Z', vendor: 'Moes', description: 'Moes old TRV/zigbee model', fromZigbee: [tuya.fz.datapoints], toZigbee: [tuya.tz.datapoints], onEvent: tuya.onEventSetLocalTime, configure: tuya.configureMagicPacket, exposes: [ e.battery(), e.child_lock(), e.max_temperature(), e.min_temperature(), e.position(), e.window_detection(), e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '), e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'), e .climate() .withLocalTemperature(ea.STATE) .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET) .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET) .withPreset( ['auto', 'manual', 'off', 'on'], 'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' + 'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' + 'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' + 'ON - In this mode, the thermostat stays open ' + 'OFF - In this mode, the thermostat stays closed', ) .withSystemMode(['auto', 'heat', 'off'], ea.STATE) .withRunningState(['idle', 'heat'], ea.STATE), ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'), e .enum('mode', ea.STATE_SET, ['comfort', 'eco']) .withDescription( 'Hysteresis - comfort > switches off/on exactly at reached ' + 'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%', ), ], meta: { tuyaDatapoints: [ [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)], [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')], [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')], [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10], [3, 'local_temperature', tuya.valueConverter.divideBy10], [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})], [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})], [8, 'window_detection', tuya.valueConverter.onOff], [12, 'child_lock', tuya.valueConverter.lockUnlock], [13, 'battery', tuya.valueConverter.raw], [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})], [15, 'min_temperature', tuya.valueConverter.divideBy10], [16, 'max_temperature', tuya.valueConverter.divideBy10], [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)], [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)], [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)], [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)], [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)], [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)], [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)], [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1], [108, 'position', tuya.valueConverter.divideBy10], [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})], ], },
extend: [
    // A preferred new way of extending functionality.
],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2385609418, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6DZUTZSABZHAC4SDLLZZKGLPAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBVGYYDSNBRHA . You are receiving this because you were mentioned.Message ID: @.***>

-- Eugenio Garijo Azpeitia Tfn. 623214313

egarijo commented 1 month ago

Hello again. I think I found the problem. I was testing with the valve on my table, not on the heating. So, if the valve is not mounting on the heater , on the display appear the F5 error. That means that the vale dont't ecognize the heater and does not send any information. I piut the valve on the heater and now I can see the information. [image: image.png]

I will continue testing, but it seems that is working. Thankyou very much for your help. Regards

El vie, 4 oct 2024 a las 18:07, Misha Ship @.***>) escribió:

I think that the device is a Tuya type, so a different converter should be created, according to the documentation here: https://www.zigbee2mqtt.io/advanced/support-new-devices/02_support_new_tuya_devices.html

I had the same issue with a Moes TRV https://www.aliexpress.com/item/1005006068601747.html?spm=a2g0o.order_list.order_list_main.5.187518022PIULc (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart https://github.com/LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here https://github.com/Koenkk/zigbee-herdsman-converters/blob/13ca5a04abfb6e38a03aee0d3eb4737a18e33071/src/devices/tuya.ts#L4933C1-L4934C1

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee'); const tz = require('zigbee-herdsman-converters/converters/toZigbee'); const exposes = require('zigbee-herdsman-converters/lib/exposes'); const reporting = require('zigbee-herdsman-converters/lib/reporting'); const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend'); const e = exposes.presets; const ea = exposes.access; const tuya = require('zigbee-herdsman-converters/lib/tuya');

const definition = { // Since a lot of Tuya devices use the same modelID, but use different datapoints // it's necessary to provide a fingerprint instead of a zigbeeModel fingerprint: [ { // The model ID from: Device with modelID 'TS0601' is not supported // You may need to add \u0000 at the end of the name in some cases modelID: 'TS0601', // The manufacturer name from: Device with modelID 'TS0601' is not supported. manufacturerName: '_TZE204_9mjy74mp', }, ], model: 'TRV801Z', vendor: 'Moes', description: 'Moes old TRV/zigbee model', fromZigbee: [tuya.fz.datapoints], toZigbee: [tuya.tz.datapoints], onEvent: tuya.onEventSetLocalTime, configure: tuya.configureMagicPacket, exposes: [ e.battery(), e.child_lock(), e.max_temperature(), e.min_temperature(), e.position(), e.window_detection(), e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '), e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'), e .climate() .withLocalTemperature(ea.STATE) .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET) .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET) .withPreset( ['auto', 'manual', 'off', 'on'], 'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' + 'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' + 'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' + 'ON - In this mode, the thermostat stays open ' + 'OFF - In this mode, the thermostat stays closed', ) .withSystemMode(['auto', 'heat', 'off'], ea.STATE) .withRunningState(['idle', 'heat'], ea.STATE), ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'), e .enum('mode', ea.STATE_SET, ['comfort', 'eco']) .withDescription( 'Hysteresis - comfort > switches off/on exactly at reached ' + 'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%', ), ], meta: { tuyaDatapoints: [ [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)], [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')], [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')], [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10], [3, 'local_temperature', tuya.valueConverter.divideBy10], [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})], [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})], [8, 'window_detection', tuya.valueConverter.onOff], [12, 'child_lock', tuya.valueConverter.lockUnlock], [13, 'battery', tuya.valueConverter.raw], [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})], [15, 'min_temperature', tuya.valueConverter.divideBy10], [16, 'max_temperature', tuya.valueConverter.divideBy10], [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)], [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)], [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)], [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)], [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)], [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)], [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)], [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1], [108, 'position', tuya.valueConverter.divideBy10], [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})], ], },
extend: [
    // A preferred new way of extending functionality.
],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

Confirm as working solution

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2394030874, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6HIOC5V6JAS3N52KTLZZ24NPAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOJUGAZTAOBXGQ . You are receiving this because you were mentioned.Message ID: @.***>

-- Eugenio Garijo Azpeitia Tfn. 623214313

Joshi120 commented 1 month ago

Heya, i am very confused. i added the code from above next to the Config file and referenced it but it still says not supported. I own the exact TRV mentioned above the code

egarijo commented 1 month ago

Hi,

I don't know what is happened, but I can explain you what I did.

I copied the code on this file: TS0601_u000.js

[image: image.png]

After that, on the configuration.yml I added the following the external converters. [image: image.png]

And that's all. As you can see in my homeassistant, appears as suported.

[image: image.png]

Regards

El lun, 21 oct 2024 a las 10:05, Joshi120 @.***>) escribió:

Heya, i am very confused. i added the code from above next to the Config file and referenced it but it still says not supported. I own the exact TRV mentioned above the code

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2425918755, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6HS2XPBUPWCHO7OAJDZ4SYS3AVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMRVHEYTQNZVGU . You are receiving this because you were mentioned.Message ID: @.***>

-- Eugenio Garijo Azpeitia Tfn. 623214313

PedroKTFC commented 1 month ago

I had the same issue with a Moes TRV (_TZE204_9mjy74mp TRV801Z) and was able to make it work by following the guide above.

Inspired by @LintHart extracted DPIDs, and looking at tuya.js, I was able to find a similar config here

The converter js should be placed next to configuration.yaml and can be referenced either via the config or the Home Assistant Zigbee2MQTT side addon.

I ended up with something like this:

const fz = require('zigbee-herdsman-converters/converters/fromZigbee');                                                                                          
const tz = require('zigbee-herdsman-converters/converters/toZigbee');                                                                                            
const exposes = require('zigbee-herdsman-converters/lib/exposes');                                                                                               
const reporting = require('zigbee-herdsman-converters/lib/reporting');                                                                                           
const modernExtend = require('zigbee-herdsman-converters/lib/modernExtend');                                                                                     
const e = exposes.presets;                                                                                                                                       
const ea = exposes.access;                                                                                                                                       
const tuya = require('zigbee-herdsman-converters/lib/tuya');                                                                                                     

const definition = {                                                                                                                                             
    // Since a lot of Tuya devices use the same modelID, but use different datapoints                                                                            
    // it's necessary to provide a fingerprint instead of a zigbeeModel                                                                                          
    fingerprint: [                                                                                                                                               
        {                                                                                                                                                        
            // The model ID from: Device with modelID 'TS0601' is not supported                                                                                  
            // You may need to add \u0000 at the end of the name in some cases                                                                                   
            modelID: 'TS0601',                                                                                                                                   
            // The manufacturer name from: Device with modelID 'TS0601' is not supported.                                                                        
            manufacturerName: '_TZE204_9mjy74mp',                                                                                                                
        },                                                                                                                                                       
    ],                                                                                                                                                           
    model: 'TRV801Z',                                                                                                                                            
    vendor: 'Moes',                                                                                                                                              
    description: 'Moes old TRV/zigbee model',                                                                                                                    
    fromZigbee: [tuya.fz.datapoints],                                                                                                                            
    toZigbee: [tuya.tz.datapoints],                                                                                                                              
    onEvent: tuya.onEventSetLocalTime,                                                                                                                           
    configure: tuya.configureMagicPacket,                                                                                                                        
exposes: [                                                                                                                                                       
            e.battery(),                                                                                                                                         
            e.child_lock(),                                                                                                                                      
            e.max_temperature(),                                                                                                                                 
            e.min_temperature(),                                                                                                                                 
            e.position(),                                                                                                                                        
            e.window_detection(),                                                                                                                                
            e.binary('window', ea.STATE, 'OPEN', 'CLOSE').withDescription('Window status closed or open '),                                                      
            e.binary('alarm_switch', ea.STATE, 'ON', 'OFF').withDescription('Thermostat in error state'),                                                        
            e                                                                                                                                                    
                .climate()                                                                                                                                       
                .withLocalTemperature(ea.STATE)                                                                                                                  
                .withSetpoint('current_heating_setpoint', 5, 35, 0.5, ea.STATE_SET)                                                                              
                .withLocalTemperatureCalibration(-30, 30, 0.1, ea.STATE_SET)                                                                                     
                .withPreset(                                                                                                                                     
                    ['auto', 'manual', 'off', 'on'],                                                                                                             
                    'MANUAL MODE ☝ - In this mode, the device executes manual temperature setting. ' +                                                           
                        'When the set temperature is lower than the "minimum temperature", the valve is closed (forced closed). ' +                              
                        'AUTO MODE ⏱ - In this mode, the device executes a preset week programming temperature time and temperature. ' +
                        'ON - In this mode, the thermostat stays open ' +
                        'OFF - In this mode, the thermostat stays closed',
                )
                .withSystemMode(['auto', 'heat', 'off'], ea.STATE)
                .withRunningState(['idle', 'heat'], ea.STATE),
            ...tuya.exposes.scheduleAllDays(ea.STATE_SET, 'HH:MM/C HH:MM/C HH:MM/C HH:MM/C'),
            e
                .enum('mode', ea.STATE_SET, ['comfort', 'eco'])
                .withDescription(
                    'Hysteresis - comfort > switches off/on exactly at reached ' +
                        'temperature with valve smooth from 0 to 100%, eco > 0.5 degrees above or below, valve either 0 or 100%',
                ),
        ],
        meta: {
            tuyaDatapoints: [
                [1, null, tuya.valueConverter.thermostatSystemModeAndPreset(null)],
                [1, 'system_mode', tuya.valueConverter.thermostatSystemModeAndPreset('system_mode')],
                [1, 'preset', tuya.valueConverter.thermostatSystemModeAndPreset('preset')],
                [2, 'current_heating_setpoint', tuya.valueConverter.divideBy10],
                [3, 'local_temperature', tuya.valueConverter.divideBy10],
                [6, 'running_state', tuya.valueConverterBasic.lookup({heat: 1, idle: 0})],
                [7, 'window', tuya.valueConverterBasic.lookup({OPEN: 1, CLOSE: 0})],
                [8, 'window_detection', tuya.valueConverter.onOff],
                [12, 'child_lock', tuya.valueConverter.lockUnlock],
                [13, 'battery', tuya.valueConverter.raw],
                [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})],
                [15, 'min_temperature', tuya.valueConverter.divideBy10],
                [16, 'max_temperature', tuya.valueConverter.divideBy10],
                [17, 'schedule_monday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(1)],
                [18, 'schedule_tuesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(2)],
                [19, 'schedule_wednesday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(3)],
                [20, 'schedule_thursday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(4)],
                [21, 'schedule_friday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(5)],
                [22, 'schedule_saturday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(6)],
                [23, 'schedule_sunday', tuya.valueConverter.thermostatScheduleDayMultiDPWithDayNumber(7)],
                [101, 'local_temperature_calibration', tuya.valueConverter.localTempCalibration1],
                [108, 'position', tuya.valueConverter.divideBy10],
                [114, 'mode', tuya.valueConverterBasic.lookup({comfort: tuya.enum(0), eco: tuya.enum(1)})],
            ],
        },

    extend: [
        // A preferred new way of extending functionality.
    ],
};

module.exports = definition;

Now this converter is probably not 100% reliable, but it seems to be working fine in Home Assistant. This might be useful for you.

Looking at the code you ended up with above and the code you reference just before it as your inspiration, I can't see any significant difference. Excuse my ignorance but can you explain what you've changed. I'm using ZHA in HA and I'm getting some basic functionality but I'd like to get more if I can work out how. Thanks.

hideintheclouds commented 1 month ago

@PedroKTFC The difference is: value conversion/lookup for the alarm_switch, as the existing config was throwing an exception when deserializing the received payload from the TRV: [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})], vs. the initial one: [14, 'alarm_switch', tuya.valueConverter.onOff],

LAD47 commented 1 month ago

@hideintheclouds The same thermostat from Moes comes with manufacturerName: "_TZE200_rtrmfadk". It is automatically installed by zigbee2mqtt. However, it has the same problem you explained with the 'alarm_switch'. I have tested '_TZE200_rtrmfadk' and it works without error messages when I use your 'alarm_switch' definitions. It would probably be wise to fix '_TZE200_rtrmfadk' as well.

egarijo commented 1 month ago

Hi.

I have testedthe same. The thermostat from Moes comes with differences inside. As you can see on the picture, one is discovered as TRV602 (Tuya) and the same thermostat is discovered as TRV801Z. This difference also appears on the reference of the valve. The first one is discovered automatically by z2mqtt and for the other I have to include an external connector. Also there are a small differences on the values expossed. [image: image.png]

The alarm switch, is not working on the first one. [image: image.png]

I never tested with ZHA.

Regards

El sáb, 26 oct 2024 a las 16:12, lad4762 @.***>) escribió:

@hideintheclouds https://github.com/hideintheclouds The same thermostat from Moes comes with manufacturerName: "_TZE200_rtrmfadk". It is automatically installed by zigbee2mqtt. However, it has the same problem you explained with the 'alarm_switch'. I have tested '_TZE200_rtrmfadk' and it works without error messages when I use your 'alarm_switch' definitions. It would probably be wise to fix '_TZE200_rtrmfadk' as well.

— Reply to this email directly, view it on GitHub https://github.com/Koenkk/zigbee2mqtt/issues/23206#issuecomment-2439596754, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMGDU6EZOEVVP5XMPINDL6LZ5OPLVAVCNFSM6AAAAABKDNPMTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMZZGU4TMNZVGQ . You are receiving this because you were mentioned.Message ID: @.***>

-- Eugenio Garijo Azpeitia Tfn. 623214313

canope-25 commented 1 month ago

@PedroKTFC The difference is: value conversion/lookup for the alarm_switch, as the existing config was throwing an exception when deserializing the received payload from the TRV:
            [14, 'alarm_switch', tuya.valueConverterBasic.lookup({ON: 1, OFF: 0})],
vs. the initial one:
            [14, 'alarm_switch', tuya.valueConverter.onOff],

Hi ,

First of all thank you for the job done 😄.

I would like to share the fact that some useful settings are still not yet supported by the converters proposed there.

It will be useful to also have the ability to define :

the eco mode hysteresis
the display orientation
the light level of of the display

The associated three options are available through Tuya / gw / app:

All the other settings are working with the converter beside the three previous ones:

There is also something that I would like ( but don't know is feasible ) , it's to also have the power supply voltage.

I selected this device because it's rely on three AA cells ( and not two as usually done by other devices ). It's seems to support NiMH cells because of that ( voltage is between 3.6 and 4.5 and seems to work better , i tested another one where everything went south when the voltage was 2.4 volt ( 2 AA NiMH cells nominal voltage ).

Thank you in advance for your help!

PedroKTFC commented 1 month ago

This is what I get in ZHA. Clearly, the mapping for the equivalent ZHA quirk needs further work. Unfortunately the quirk is complicated so working out what needs to change will take some time.

Screenshot 2024-10-30 at 22-34-28 Developer Tools – Home Assistant - gamlingay pedropaula uk

canope-25 commented 1 month ago

Hi , I'm trying to extend the Tuya Developer IOT Core license hoping to be able to access to more details regarding the device but seems difficult . What can I do in order to help improve the features coverage with zb2mqtt ?

canope-25 commented 1 month ago

Hi ,

I will try ( this evening ) the js template available at this link :

https://github.com/DonaldChung-HK/z2m-moes-trv-801-converter/blob/main/moes_valve_ZTRV801.js

Seems to be more complete