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

Bug: Hue Light: Color Temperature absolute values wrong #316

Closed ColdFireIce closed 5 months ago

ColdFireIce commented 6 months ago

Hi,

is it possible, that the Kelvin values for Color Temperature are a bit off? Everything works and you can change them reliably, (What goes in comes out). But I think they are off by a factor 10: image I change the color temperature from very cold (~6500K) to very warm (~2000K)

Was as shown in KNX the values are off by a faktor of 10 and I'm not sure about the low end? Shouldn't it be at about 2000? currently it comes back as 0. Looks more like an 0..100% streched to 0..65535 (0x0000..0xFFFF). Is this intended for the color temperature absolute DPT? Looking at https://www.home-assistant.io/integrations/knx/#color_temperature_mode and some other sources (including my own previous implementation) I think the values should be "real" Kelvin numbers e.g. 2200K..6500K I think this text ist also a bit off regarding the range: image https://www.home-assistant.io/integrations/knx/#value-types (see. 7.600 and 9.002) Additionally here: https://www.promotic.eu/en/pmdoc/Subsystems/Comm/PmDrivers/KNXDTypes.htm 9.002 is mentioned as 2byte double

When implementing the light with HA this shows as not beeing able to select cold colors, but the light will only show warm colors

Best regards Daniel

Supergiovane commented 6 months ago

Hi Daniel I've developed all in a hurry, so it may be prone to optimization/fixing. You're the only one pointing this me out. I've spent many time in developing the HUE integration and i need some time for my "real" work. I'm fixing the bugs with the maximum priority, but i'm forced to leave other issues behind. At least for a while... I'm not really using any HSV and Tunable White function and i'm developing this for free for other users than me. ;-) I'll watch your other issues, then i'll proceed as soon as possible. If you're comfortable with javascript, you could join the developing by forking the code and, then, making a PR. Currently, i'm alone :-/ If not, please give me some time, as i'm currently a little busy.

ColdFireIce commented 6 months ago

Sure thing and no worries. I thought I just open some issues it's more for a backlog than an must.

Thanks again for your work. I know how unrewarding the opensource projects sometimes are.

Supergiovane commented 6 months ago

Hi Daniel Please put the unzipped files into the .node-red/node_modules/node-red-contrib-knx-ultimate/nodes folder, by replacing the existing files and restart nodered. Then, please, verify if this issue is resolved. Thanks. Archivio.zip

Supergiovane commented 6 months ago

Hi Daniel please install the 2.4.0-beta.0 You must do that by issuing the command npm install node-red-contrib-knx-ultimate@2.4.0-beta.0 in the .node-red/node_modules folder. Then, let me know. Thanks.