fix: System mode in Namron thermostat

[bskjon]

I'm currently seeing some incorrect display of the values:

How do i solve this?

This is just one of the places

[Koenkk]

I'm currently seeing some incorrect display of the values:

What is incorrect here?

[bskjon]

I'm currently seeing some incorrect display of the values:

What is incorrect here?

The underscores doesn't go away like the rest
Will it be fine if i just remove them?

[Koenkk]

Will it be fine if i just remove them?

No, if it should be fixed (I'm not sure) it should be fixed in the frontend.

[bskjon]

Will it be fine if i just remove them?

No, if it should be fixed (I'm not sure) it should be fixed in the frontend.

It will also be wrong in homeassistant 🤔

[bskjon]

So i should not edit the file to be more like

        deviceFault: (args?: Partial<modernExtend.EnumLookupArgs>) =>
            modernExtend.enumLookup({
                name: 'fault',
                cluster: 'hvacThermostat',
                attribute: {ID: 0x8006, type: Zcl.DataType.ENUM8},
                description: 'Device fault',
                lookup: {
                    no_fault: 0,
                    over_current_error: 1,
                    over_heat_error: 2,
                    'built-in_sensor_error': 3,
                    air_sensor_error: 4,
                    floor_sensor_error: 5,
                },
                access: 'STATE_GET',
                ...args,
            }),

to:

       deviceFault: (args?: Partial<modernExtend.EnumLookupArgs>) =>
            modernExtend.enumLookup({
                name: 'fault',
                cluster: 'hvacThermostat',
                attribute: {ID: 0x8006, type: Zcl.DataType.ENUM8},
                description: 'Device fault',
                lookup: {
                    "No Fault": 0,
                    "Over current error": 1,
                    "Over heat error": 2,
                    "Built-in sensor error": 3,
                    "Air sensor error": 4,
                    "Floor sensor error": 5,
                },
                access: 'STATE_GET',
                ...args,
            }),

[Koenkk]

The convention is to have _ instead of , so then I would propose to leave it as is.

[Koenkk]

Thanks!