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 Module (54) on 6.6.0 gives 6 power switches #6039

Closed
john5788 opened this issue Jul 7, 2019 · 11 comments

Comments

@john5788
Copy link

commented Jul 7, 2019

Tuya dimmer (KS-7011) works fine on 6.5.0, but upgrading to 6.6.0 converts the device into something that has 6 power channels and the dimmer slider no longer works. See attached images.

unknown

unknown (1)

The device reports back to HomeAssistant as multiple switches instead of a light with the update. Toggling the leftmost bit in the web UI toggles the light switch on and off.

Using this template: https://blakadder.github.io/templates/zemismart-ks-7011.html

@Jason2866

This comment has been minimized.

Copy link
Contributor

commented Jul 7, 2019

@s-hadinger can you take a look into? Just selecting module 54 (Tuya) results in this.
Thx!

@s-hadinger

This comment has been minimized.

Copy link
Contributor

commented Jul 7, 2019

This part of Tasmota is still obsure to me. I will try to see when the regression happened.

@ascillato2 ascillato2 added the bug label Jul 7, 2019

@arendst

This comment has been minimized.

Copy link
Owner

commented Jul 7, 2019

I'll investigate tomorrow.

@s-hadinger

This comment has been minimized.

Copy link
Contributor

commented Jul 7, 2019

I'm exploring commits. Up to ab9baf2 (6.5.0.13) it still works.

@arendst

This comment has been minimized.

Copy link
Owner

commented Jul 7, 2019

I did some changes in tuya in one of the latest commits ...

@s-hadinger

This comment has been minimized.

Copy link
Contributor

commented Jul 7, 2019

And broken in 12da2fd

@arendst arendst self-assigned this Jul 8, 2019

@arendst

This comment has been minimized.

Copy link
Owner

commented Jul 8, 2019

Caused by commit aaacb20 (thirug010) which uses uninitialised, undocumented and messy param setting (SetOption38) to simulate amount of switches.

As the use of param SetOption38 was not documented the same parameter is used and initialised with 6 by IRRemote support resulting in 6 toggles pop up.

Performing command SetOption38 1 could be used as a workaround if IRReceive was not enabled in the binary. This is not always the case making this issue even more problematic.

I will need to dig into this further and would request PR suppliers to more thoroughly test their submitted PR's.

To be continued.

arendst added a commit that referenced this issue Jul 8, 2019

Fix ambiguous Tuya set relay option by adding command SetOption41 all…
…owing to control number of virtual relays

Fix ambiguous Tuya set relay option by adding command SetOption41 allowing to control number of virtual relays (#6039)
@arendst

This comment has been minimized.

Copy link
Owner

commented Jul 8, 2019

Use SetOption41 to increment number of relays/switches. Default set to zero to supply legacy support.

@thirug010

This comment has been minimized.

Copy link
Contributor

commented Jul 8, 2019

@arendst, My apologies for not using with the proper document Setting parameter Index

@arendst

This comment has been minimized.

Copy link
Owner

commented Jul 8, 2019

@thirug010 accepted ;-)

@ascillato2

This comment has been minimized.

Copy link
Collaborator

commented Jul 8, 2019

Closing as this issue has been fixed. Thanks 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.