EnergyHub DC Link Voltage state is zero about ha-ferroamp HOT 8 CLOSED

It should be pos / count - neg / count.

Also, it shouldn't be int(). Would be better to use something like round(float(pos / count - neg / count), 2).

from ha-ferroamp.

It should be pos / count - neg / count.
Also, it shouldn't be int(). Would be better to use something like round(float(pos / count - neg / count), 2).

That would result in ~760 in the case @bj00rn had, right? Can someone provide a PR with those changes?

I can give it a shot

from ha-ferroamp.

Works on my machine (TM) :-)

from ha-ferroamp.

No domain knowledge at my end either but I see how you're thinking. @danielolsson100 or @mickecamino perhaps knows more?

from ha-ferroamp.

@argoyle I have no clue in this matter since I am not using this value at all, I don't even store the history data in the DB for this sensor ;)

from ha-ferroamp.

The value from Ferroamp is a float, so it should be a float in the code:

from ha-ferroamp.

It should be pos / count - neg / count.

Also, it shouldn't be int(). Would be better to use something like round(float(pos / count - neg / count), 2).

That would result in ~760 in the case @bj00rn had, right? Can someone provide a PR with those changes?

from ha-ferroamp.

Hmmm, it seems that I looked at the wrong sensor, Here is the correct one:

from ha-ferroamp.