-
Notifications
You must be signed in to change notification settings - Fork 505
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 2 Gang ZigBee Dimmer Module #5320
Comments
Can you please add all screens? |
Hi Mimiix, 👋 vishivishy here. Hope you've been well. |
It s perhaps a 1 line edit modification. Do you have a linux machine with full OS to test a code modification ? Edit: Edit2: |
Hi @Smanar thanks for looking into this. So is it that 2 gang can not be supported by deconz or that I wont be able to test the new modified code? |
Can work, as you are not using common Mac adress, from the issue it seem the only one other device that use this mac adress is a covering. You have a linux machine with full OS ? |
I can setup a new linux vm if that works.. (if you can give me details of the setup) |
Yes it work on VM, so you need to set the VM, update the OS, install deconz using package method https://www.phoscon.de/en/conbee2/install For the rest you have the procedure here https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/Compiling-the-REST-plugin-for-device-specific-testing
But from the code, you need to have at least the on/off working for one gang ? |
I already am able to use on/off for 1 gang on the current homeassistant-deconz (6.10.0) firmware(26660700). |
@Smanar so I ran the updated plugin on the linux vm and I'm able to control brightness of the 1 gang that I was able to turn on and off earlier. here are some screenshots from the test BasicGroupsScenesOther |
Yes the code create 3 "light", it s because this part is a hack used for 3 gangs devices.
|
Hi @Smanar! I've tested the new changes and here are the findings:
Node InfoBasic |
Yeap, this is normal for the moment, will be corrected in the future with ddf file, I need to remove this hack And you was right for the rest, it s because the return state was bad. Have updated the code again. But for the code is not working using the second Webapp, it s strange, are you able to find some logs somewhere ? using the dev console, deconz logs, or others ? |
Hooray! Both the gang lights are working as expected in the Phoscon app! I could only see these errors in the logs but have attached the log file in case you need to check something else out.
test_debug.log |
Hu .., In fact I have some problem ATM, I have no internet connexion, and no deconz running to check ^^ (for at least 2 days)
Something is bad on the request Something you can try is enabling the developper console on your web browser to check http request or easier check for "http" logs on deconz. Take care care thoses logs are realy talkative, so enable only "http", on thoses logs you will see incoming and outgoing requests from deconz, so you will see the request made by phoscon and the one made by the old webapp. |
Hi @Smanar, so I had a look at the requests being sent by Phoscon and deconz webapp, and seems like the two apps are sending different payload in the put request.
and deconz webapp is sending this and getting response code 400
it seemed like the api is not expecting the "id" parameter in the payload so I tested on postman and can confirm that when we send id param in payload the server sends a 400 response, but if I remove the id param the server responds with 200 and turns on the light. An interesting thing I noted is that sending "id" parameter in payload for a device with single gang does not return a bad request 400 response and it turns on the light as shown in the screenshots below. (*But the phoscon app does not send the id param in the payload regardless of what kind of light it is, probably because the "id" is a path parameter) 2 gang device1 gang device |
Ok have added a code part to make the "id" skipable for the tuya part. (Same command to update your code) |
Thanks @Smanar everything seems to be working now!! What's next? |
Have just made the PR. Next is waiting for validation, but it s possible I change the code to use DDF file ( to remove the useless "light" entry) |
Cool, sounds good. I'll look forward to troubleshooting and fixing another tuya device with you someday.. :) |
As there has not been any response in 21 days, this issue has been automatically marked as stale. At OP: Please either close this issue or keep it active It will be closed in 7 days if no further activity occurs. |
Hi @Smanar! just following up to see if this is getting merged soon.. |
Hi @Mimiix! How have you been? just following up to see if this is getting merged soon.. |
@vishivishy Hi! :) All good here. How about you? Thanks for asking :) I'll ask manup to merge it, but can't make promises. |
@Mimiix I'm good too, thanks :) |
@vishivishy It gets updated once a new deCONZ stable is out. |
Hi, what to do to have working dimming on those dimmers? Do we just have to wait to be implemented? If I understood correctly 1 gang and 3 gang moes are already supported? |
It's totaly random, there is new tuya every days, and all look same (and can have same product name), need to compare according the manufacture name and MAC address. |
As there has not been any response in 21 days, this issue has been automatically marked as stale. At OP: Please either close this issue or keep it active It will be closed in 7 days if no further activity occurs. |
not stale |
As there has not been any response in 21 days, this issue has been automatically marked as stale. At OP: Please either close this issue or keep it active It will be closed in 7 days if no further activity occurs. |
As there has not been any response in 28 days, this issue will be closed. @ OP: If this issue is solved post what fixed it for you. If it is not solved, request to get this opened again. |
not merged yet |
looking forward to buy one, so hope it will be merged soon! |
As there has not been any response in 21 days, this issue has been automatically marked as stale. At OP: Please either close this issue or keep it active It will be closed in 7 days if no further activity occurs. |
not stale |
I have the phoscon II and de deconz software on my pi. |
For the moment PR are locked to use only DDF #5733 This device can't be added using DDF, so need to improve the DDF core to manage tuya cluster. |
As there has not been any response in 21 days, this issue has been automatically marked as stale. At OP: Please either close this issue or keep it active It will be closed in 7 days if no further activity occurs. |
not stale |
For information there is a PR and a DDF for this device #5869 |
Device
Screenshots
Basic
Groups
Scenes
Other clusters that are not mentioned above
The text was updated successfully, but these errors were encountered: