Skip to content
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

Error on level control for Busch-Jaeger 6715 (RM01) #4618

Closed
dreamsinmotion opened this issue Oct 9, 2020 · 9 comments · Fixed by Koenkk/zigbee-herdsman-converters#1665
Closed
Labels
problem Something isn't working

Comments

@dreamsinmotion
Copy link

I updated to the version 1.15.0 and the level control for my Busch Jaeger 6715 (RM01) is not working any more. I have returned to the previous version 1.13.0 and everything works fine again. In version 1.15.0 I get following error log:

Zigbee2MQTT:debug 2020-10-09 11:44:20: Received MQTT message on 'zigbee2mqtt/livingroom_buschjaeger_switch/set' with data '{"brightness":120}'
Zigbee2MQTT:debug 2020-10-09 11:44:20: Publishing 'set' 'brightness' to 'livingroom_buschjaeger_switch'
Zigbee2MQTT:error 2020-10-09 11:44:20: Publish 'set' 'brightness' to 'livingroom_buschjaeger_switch' failed: 'TypeError: Cannot read property 'convertSet' of undefined'
Zigbee2MQTT:debug 2020-10-09 11:44:20: TypeError: Cannot read property 'convertSet' of undefined
    at Object.convertSet (/app/node_modules/zigbee-herdsman-converters/converters/toZigbee.js:2453:58)
    at EntityPublish.onMQTTMessage (/app/lib/extension/publish.js:225:52)
    at Controller.callExtensionMethod (/app/lib/controller.js:372:44)
    at Controller.onMQTTMessage (/app/lib/controller.js:269:14)
    at MQTT.emit (events.js:315:20)
    at MQTT.onMessage (/app/lib/mqtt.js:94:14)
    at WebSocket.<anonymous> (/app/lib/extension/frontend.js:86:27)
    at WebSocket.emit (events.js:315:20)
    at Receiver.receiverOnMessage (/app/node_modules/ws/lib/websocket.js:797:20)
    at Receiver.emit (events.js:315:20)
Zigbee2MQTT:info  2020-10-09 11:44:20: MQTT publish: topic 'zigbee2mqtt/bridge/log', payload '{"message":"Publish 'set' 'brightness' to 'livingroom_buschjaeger_switch' failed: 'TypeError: Cannot read property 'convertSet' of undefined'","meta":{"friendly_name":"livingroom_buschjaeger_switch"},"type":"zigbee_publish_error"}'
@dreamsinmotion dreamsinmotion added the problem Something isn't working label Oct 9, 2020
@Koenkk
Copy link
Owner

Koenkk commented Oct 9, 2020

make sure you have no changes to devices.js (for hassio zigbee_sheperd_devices: true)

@dreamsinmotion
Copy link
Author

I'm running zigbee2mqtt in docker and using the default devices.js

@Koenkk
Copy link
Owner

Koenkk commented Oct 9, 2020

Are you sure there is no Docker volume mapped overriding zigbee-herdsman-converters? It looks that your zigbee-herdsman-converters is not compatible with your Zigbee2MQTT verison.

@dreamsinmotion
Copy link
Author

Yes, I'm only mapping /app/data to a folder. My other devices are working as expected and the set state on the Busch Jaeger 6715 (RM01) is working as well. Only the level control doesn't work.

@Koenkk
Copy link
Owner

Koenkk commented Oct 9, 2020

I've tried to (virtually, since I don't have the device myself) reproduce this issue but failed to do so.

@prutseltje @WinSCaP @docbobo is this working for you in Zigbee2MQTT 1.15.0?

@prutseltje
Copy link

I have the Busch-Jaeger 6711 U, this is only a relay. So I'm unable to test the dimmer function.

I am running Zigbee2MQTT 1.15.0 and do not experience any issues.

@docbobo
Copy link
Contributor

docbobo commented Oct 10, 2020

Don't have a dimmer either... But 1.15.0 has been running without issues so far for me...

@Olli-Lolli
Copy link

I have the same issue with the level control of the Busch Jaeger 6715 dimmer.
I also looked into the code and the problem seems to be that the converter RM01_light_brightness in toZigbee.js uses the converter light_brightness, but this converter has been removed in a recent commit

    RM01_light_brightness: {
        key: ['brightness', 'brightness_percent'],
        convertSet: async (entity, key, value, meta) => {
            if (utils.hasEndpoints(meta.device, [0x12])) {
                const endpoint = meta.device.getEndpoint(0x12);
                return await converters.light_brightness.convertSet(endpoint, key, value, meta);
            } else {
                throw new Error('LevelControl not supported on this RM01 device.');
            }
        },

@Koenkk
Copy link
Owner

Koenkk commented Oct 15, 2020

Tanks to @Tom-Black this got fixed. Changes will be available in the latest dev branch tomorrow (https://www.zigbee2mqtt.io/how_tos/how-to-switch-to-dev-branch.html)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
problem Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants