Supergiovane / node-red-contrib-knx-ultimate

Control your KNX intallation via Node-Red! A bunch of KNX nodes, with integrated Philips HUE control and ETS group address importer.
https://youtu.be/egRbR_KwP9I
MIT License
141 stars 34 forks source link

HUE Tunable White Status is not working #355

Closed KoljaW closed 2 months ago

KoljaW commented 2 months ago

Hello :) First i would like to say THANK YOU for this awesome integration..

Describe the bug I am able to control kelvin in the tunable white tab, but in my ETS debug monitor i always get 13222 K back as a feedback in the status adress.

Screenshots ETS_HUE NodeRed_HUE

Knx-Ultimate Version

Are you running node-red behind homematic, docker or anything similar?

Supergiovane commented 2 months ago

Hi thanks for using my node. I'll check it immediately.

Supergiovane commented 2 months ago

Hi again. node-red-contrib-knx-ultimate@2.4.24 is out with the fix. Please re-open this issue is you encounter problems. ;-)

KoljaW commented 2 months ago

@Supergiovane Unfortunately now i get always 6535 Kelvin back on my feedback address. No matter what i type in.

Second issue:

There is no auto response for the status feedback adress. I get only a feedback when i click on read in my ets monitor.

ETS_HUE_neu

Supergiovane commented 2 months ago

Hi can i see your fow? Can you paste it here?

KoljaW commented 2 months ago

Its just a single Node(Flow) controlled by KNX.

1 2 3 4 5 6

Supergiovane commented 2 months ago

Good. Thank you. I'll try duplicating all your setting and i'll see. If you paste the json code, use the "<>" button, so i can then copy the flow.

Supergiovane commented 2 months ago

Hi, cannot reproduce your issue. I've tried sending the telegram using node-red and also using ETS. The feedback follows the right value.

https://github.com/Supergiovane/node-red-contrib-knx-ultimate/assets/37666636/acf85dfe-bfe2-444c-a66d-95ccd7645a0d

Do you maybe have other nodes 5/5/61 or 5/5/60 with a different datapoint than 7.600 in all your flows?

Supergiovane commented 2 months ago

No, stop. I've understood what you're doing. You're making a READ request to 5/5/61 I also receive 6565 in this case. Please hold on, i'm ditching into it....

KoljaW commented 2 months ago

Maybe a KNX Problem?

https://github.com/Supergiovane/node-red-contrib-knx-ultimate/assets/76074195/44a9cc5c-c78e-4a12-88d0-ca6eb8bd1253

Supergiovane commented 2 months ago

I'm testing.... will back to you asap.

Supergiovane commented 2 months ago

I'ts related to the grouped lights. In HUE, it doesn't exist a device representing a group, but instead, the group is a collection of light objects. Usually, when the device is a grouped_lights, i get the first device in the collection and read the requested property. In the "kelvin read" case, however, i don't do that by mistake (i forgot it). I'm fixing the code.

Supergiovane commented 2 months ago

I'm about to finish. It took longer than expected!

Supergiovane commented 2 months ago

Should be fixed in 2.4.26 out now.

KoljaW commented 2 months ago

@Supergiovane Hello,

Oh no. something went wrong. I have no connection to my hue bridge and its not possible to setup a connection anymore.

Bildschirmfoto 2024-05-10 um 19 53 39
Supergiovane commented 2 months ago

Shit….. downgrade to the previous version

Supergiovane commented 2 months ago

Mine upgraded without issues. Can you tell me, what the log says?

Supergiovane commented 2 months ago

It was a Matter protocol refuse. 2.4.27 should be ok.

Supergiovane commented 2 months ago

You can also comment this row


//const { AsyncObservable } = require("@project-chip/matter-node.js/util");

in the .node-red/node_modules/node-red-contrib-knx-ultimate/nodes/hue-config.js Then restart node-red

KoljaW commented 2 months ago

Upgrade to 2.4.27 and everything is fine.

Also the original issue (tunable white feedback with grouped hue lights) is now working fine !!!

THANK YOU!!!

Supergiovane commented 2 months ago

If you encounter strange behaviour, you can revert to 2.4.23 by installing the new 2.4.28 version. It 's a temporary revert, to better solve the issues you encountered.