Smanar / Domoticz-deCONZ

deCONZ plugin for Domoticz (Zigbee application)
GNU General Public License v3.0
36 stars 27 forks source link

Current values should be in milli Ampere #132

Closed jgaalen closed 2 years ago

jgaalen commented 2 years ago

The new current values are shown as A, but the measurements for the Blitzwolf SHP-15 at least are in milli Amperes. Not sure if this is for every device, but perhaps the values should be divided by 1000 before passed to domoticz

Smanar commented 2 years ago

Hello. Values in the API are in mA (or at least need to be) because it s generaly small value, and are send as it in domoticz. The domoticz widget is in A and no way to use mA.

So I think this issue is for all devices.

You are using the new flag ENABLEMORESENSOR ? So it s something new as issue ?

jgaalen commented 2 years ago

Yes I have that flag on, and have these current/A values just recently. But currently it is showing '2380 A' for a plug which is using 555 watt, so clearly it's in mA, not A. As you say, the values are in mA, so if you divide the value with 1000, you will get 2.380 as a stored value, which would be right.

Smanar commented 2 years ago

Done, if you want to try the beta ?

git pull
git checkout beta
git pull

I hope there is not a rounding issue.

jgaalen commented 2 years ago

Looks good to me now, thanks again!