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

tuya ZG-204ZM setting motion detection mode fails (pull request #7553) #7590

Closed
rolfvreijdenberger opened this issue Jun 1, 2024 · 6 comments

Comments

@rolfvreijdenberger
Copy link

rolfvreijdenberger commented Jun 1, 2024

in the new zigbee2mqtt release https://github.com/Koenkk/zigbee2mqtt/releases/tag/1.38.0 a feature was introduced via pull request #7553 where detection modes can be set.

After updating zigbee2mqtt, restarting the service, making sure I have the correct firmware as specified for the functionality, I get an error when trying to use the new detection mode settings.

It seems to me a mapping from the frontend button input to the actual settings is going wrong.

error message + debug from log after pressing button in zb2mqtt frontend.
image

Debug 2024-06-01 14:35:45Received MQTT message on 'zigbee2mqtt/presence sensor 7/set' with data '{"motion_detection_mode":"PIR+Dadar"}'
Debug 2024-06-01 14:35:45Publishing 'set' 'motion_detection_mode' to 'presence sensor 7'

Error 2024-06-01 14:35:45Publish 'set' 'motion_detection_mode' to 'presence sensor 7' failed: 'Error: Value: 'PIR+Dadar' not found in: [only_pir, pir_and_dadar, only_dadar]'
Debug 2024-06-01 14:35:45Error: Value: 'PIR+Dadar' not found in: [only_pir, pir_and_dadar, only_dadar] at Object.getFromLookup (/app/node_modules/zigbee-herdsman-converters/src/lib/utils.ts:581:15) at Object.to (/app/node_modules/zigbee-herdsman-converters/src/lib/tuya.ts:436:53) at Object.convertSet (/app/node_modules/zigbee-herdsman-converters/src/lib/tuya.ts:1157:57) at Publish.onMQTTMessage (/app/lib/extension/publish.ts:259:52) at EventEmitter.wrappedCallback (/app/lib/eventBus.ts:174:23) at EventEmitter.emit (node:events:529:35) at EventBus.emitMQTTMessage (/app/lib/eventBus.ts:115:22) at MQTT.onMessage (/app/lib/mqtt.ts:147:27) at WebSocket.<anonymous> (/app/lib/extension/frontend.ts:130:27) at WebSocket.emit (node:events:517:28)

correct firmware
image

correct zb2mqtt release
image

error from gui
image

@Koenkk
Copy link
Owner

Koenkk commented Jun 1, 2024

Fixed!

Changes will be available in the dev branch in a few hours from now.

@aliorala
Copy link

aliorala commented Jun 4, 2024

I couldn't figure it out, how should it be done? Can you explain in detail?

@aliorala
Copy link

aliorala commented Jun 4, 2024

Should I always keep static detection at 10?

@aliorala
Copy link

aliorala commented Jun 4, 2024

Also, how can I reduce the refresh rate of the lux value?

@ArkadiuszNiemiec
Copy link
Contributor

@Koenkk do you intended to keep "Dadar" instead of "Radar"? ;)
The sensor also is not very reliable, the presence is not beeing updated to away untill I change a setting of the sensor. I guess we have no control over that, right?

@Koenkk
Copy link
Owner

Koenkk commented Jun 5, 2024

Fixed the typo

The sensor also is not very reliable, the presence is not beeing updated to away untill I change a setting of the sensor. I guess we have no control over that, right?

It looks like the sensor doesn't send updates, so I don't think so.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants