Configure on_level via config

[Wireheadbe]

Hi All,

Question - I've got a bunch of Ikea lights, but when they lose power, they lose the following configured parameter - and revert to "previous":

"level_config": {
        "on_level": 1
    },

Is there a way this parameter can be configured for devices, upon start-up of zigbee2mqtt? It's a very handy setting to be able to control from the start, as it improves the way adaptive lighting works.

[sjorge]

I'm guessing it needs another branch here to restore on_level too /~https://github.com/Koenkk/zigbee-herdsman-converters/blob/e4317cc8704211e8a56b3cc98da4f25580fafc33/devices/ikea.js#L19

Probably replacing the function with this one should work.

const bulbOnEvent = async (type, data, device, options, state) => {
    /**
     * IKEA bulbs lose their configured reportings when losing power.
     * A deviceAnnounce indicates they are powered on again.
     * Reconfigure the configured reoprting here.
     *
     * Additionally some other information is lost like
     *   color_options.execute_if_off. We also restore these.
     *
     * NOTE: binds are not lost so rebinding is not needed!
     */
    if (type === 'deviceAnnounce') {
        for (const endpoint of device.endpoints) {
            for (const c of endpoint.configuredReportings) {
                await endpoint.configureReporting(c.cluster.name, [{
                    attribute: c.attribute.name, minimumReportInterval: c.minimumReportInterval,
                    maximumReportInterval: c.maximumReportInterval, reportableChange: c.reportableChange,
                }]);
            }
        }

        // NOTE: execute_if_off default is false
        //       we only restore if true, to save unneeded network writes
        if (state !== undefined && state.color_options !== undefined && state.color_options.execute_if_off === true) {
            device.endpoints[0].write('lightingColorCtrl', {'options': 1});
        }
        if (state !== undefined && state.level_config !== undefined && state.level_config.execute_if_off === true) {
            device.endpoints[0].write('genLevelCtrl', {'options': 1});
        }
        if (state !== undefined && state.level_config !== undefined && state.level_config.on_level !== undefined) {
            let onLevel = state.level_config.on_level;
            if (typeof onLevel === 'string' && onLevel.toLowerCase() == 'previous') {
                onLevel = 255;
            } else {
                onLevel = Number(onLevel);
            }
            if (onLevel > 255) onLevel = 254;
            if (onLevel < 1) onLevel = 1;

            device.endpoints[0].write('genLevelCtrl', {onLevel});
        }
    }
};

I'm away from my dev setup at the moment so I didn't test this code.

[Wireheadbe]

Wow - that seems to be working, a quick change of that file in my docker container, and cutting power from my bulbs seems to have done the trick. @Koenkk - maybe worth taking up into the converters? It's a lifesaver for people running adaptive lighting - setting this to 1, and having it remain after the bulbs lose power, saves you from being blasted with light at night 😄

[sjorge]

I am currently traveling but i can do a PR id i have some time, or when i get back. Good to know it works.