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
154 stars 34 forks source link

Send write TypeError: value.toFixed is not a function #344

Closed andreklug closed 7 months ago

andreklug commented 7 months ago

Hi @Supergiovane

I am using node-red-contrib-knx-ultimate for a long time already, so first thank you for this very useful project!

Recently I realised that some KNX values don't get updated anymore:

grafik

What all of them have in common is that they are using DPT 9.001 (Temperature °C)

As you see in the screenshot, the error message is the same for them: Send write TypeError: value.toFixed is not a function

To Reproduce

I am not sure if this can be reproduced but will be happy to provide any log files you need.

Knx-Ultimate Version

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

Node-Red runs in a Proxmox LVM Container, for years (no recent change here)

Additional context I believe the only change I did was updatingf a few weeks back, after not updating for a few weeks - months before that. The configuration of the data that is sent to node-red-contrib-knx-ultimate hasn't changed for a few years.

Thank you for looking into this!

Supergiovane commented 7 months ago

Hi i’ll take a look tomorrow morning!

Supergiovane commented 7 months ago

Hi the issue occurs because you're passing a number as string. Anyway, i'm updating the code to do an auto conversion from string to numeric value.

Supergiovane commented 7 months ago

2.4.20 ist out with the updated code.

andreklug commented 7 months ago

Wow, that was quick. Everything is green now.

Thank you very much!!!