Use color picker white point to switch to white mode.

[ultratoto14]

Fix also unwanted DP request that can generate unneeded log entry in DEBUG mode.

[FeikoJoosten]

@ultratoto14 seems to function as intended. However, did find another bug. If you change the brightness of the lamp, the previous color value is taken into account instead of the last assigned mode (white/color).

[ultratoto14]

@FeikoJoosten can you describe a little more the steps for your brightness issue ?

[FeikoJoosten]

@ultratoto14 I have the lamp configured as followed.

With the setup from the screenshot. The lamp is set to white with a low brightness.
Adjusting the brightness changes the lamp to a blue color and allows me to change the brightness.
Adjusting the color temperature back to the desired color, reverts the brightness to the low brightness again (no clue what the brightness value is).

[ultratoto14]

On the screenshot, the white point is not centered, it means HA thinks your device is in color mode.
If you use this PR, close the window and reopen it, the color picker should be really centered, if it's not the case, the white status is not reflected in HA. Do you use the tuya app at the same time ?

[FeikoJoosten]

I have it perfectly centered now and I still have the same issue I mentioned before.

I do not use the Tuya app for this device at the same time. This is only running through LocalTuya.

[ultratoto14]

Can you enable the debugging, do the steps and post here the log file ?
It could also help to point the time for each step you are doing.

[FeikoJoosten]

Per your request here are the logs when changing the light settings.

2020-11-23 15:30:00 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Send payload: b'{"devId":"80157022e09806021245","uid":"80157022e09806021245","t":"1606145400","dps":{"1":true,"5":"727272002d0075","2":"colour"}}'
2020-11-23 15:30:00 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Waiting for sequence number 19
2020-11-23 15:30:00 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Dispatching message TuyaMessage(seqno=19, cmd=7, retcode=0, payload=b'', crc=1902033541)
2020-11-23 15:30:00 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Dispatching sequence number 19
2020-11-23 15:30:00 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Decrypted payload: {}
2020-11-23 15:30:01 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Sending command set (device type: type_0a)
2020-11-23 15:30:01 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Send payload: b'{"devId":"80157022e09806021245","uid":"80157022e09806021245","t":"1606145401","dps":{"1":true,"2":"white","3":3,"4":141}}'
2020-11-23 15:30:01 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Waiting for sequence number 20
2020-11-23 15:30:02 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Dispatching message TuyaMessage(seqno=20, cmd=7, retcode=0, payload=b'', crc=204324061)
2020-11-23 15:30:02 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Dispatching sequence number 20
2020-11-23 15:30:02 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Decrypted payload: {}
2020-11-23 15:30:03 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Sending command heartbeat (device type: type_0a)
2020-11-23 15:30:03 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Send payload: b'{}'
2020-11-23 15:30:03 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Waiting for sequence number -100
2020-11-23 15:30:03 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Dispatching message TuyaMessage(seqno=0, cmd=9, retcode=0, payload=b'', crc=2958142211)
2020-11-23 15:30:03 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Got heartbeat response
2020-11-23 15:30:03 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Decrypted payload: {}

Full log: home-assistant.log

[ultratoto14]

Hi @FeikoJoosten thanks for the logs. What we can see is that the HA light does not receive any data from the device, all the decrypted payload are empty.
I also see in the sent commands that your brightness is set to 3 ("3":3):

2020-11-23 15:30:01 DEBUG (MainThread) [custom_components.localtuya.pytuya] [801...245] Send payload: b'{"devId":"80157022e09806021245","uid":"80157022e09806021245","t":"1606145401","dps":{"1":true,"2":"white","3":3,"4":141}}'

Can you set a min value of 25 as you showed in your post and check again ?

[FeikoJoosten]

The issue still occurs if I set the brightness min to 25. However, I think there might be something else going on as well, possibly causing this issue. The lamp seems to be unaware of it's actual state. For example, turning off the lamp in HA causes HA to turn it back on in Lovelace. (The lamp does stay off though).

Included updated log from the moment I've set the min brightness to 25. home-assistant.log

Edit: Gif of issue where the lamp turns on again in Lovelace.

[ultratoto14]

I had this issue before, when you set a value out of the range, the device may fail. Can you try to physically switch off and on the device ?

[FeikoJoosten]

Hmm and now I have a feeling I might need to reinstall LocalTuya.

Platform localtuya does not generate unique IDs. ID local_80157022e09806021245_1 already exists - ignoring light.kitchen_light
Home Assistant has started!``` 

[ultratoto14]

You can first try to remove the entity from the integration page and restart HA first.

@postlund, this error mentioned by FeikoJoosten, also occurred sometimes on my setup, maybe it's linked to YAML device configuration migration. Are you aware of that one ? Is it problematic or an expected behavior ?

[FeikoJoosten]

Okay, readding the light and with the change from this PR resolves all of my issues. The light seems to function properly!

[postlund]

@ultratoto14 I think I've seen it in some other issue before, I don't know what has happened but I sure do want know. If you can manage to replicate, I would like to know how and also see the config entries created by the integration.

[postlund]

Sorry 'bout the accidental close, I'm on mobile and the UI jumped all over the place and I was too fast...

[ultratoto14]

No problem.