shivan / homeassistant-liquidcheck

Home Assistant integration for Liquid-Check
GNU General Public License v3.0
2 stars 0 forks source link

Integration returns 0 when device is offline #7

Open FlorianWilk opened 2 months ago

FlorianWilk commented 2 months ago

Hi,

ich bin mal so frech und schreibe auf Deutsch :) Erstmal danke für die coole Integration - genau danach habe ich gesucht.

Eine Sache ist mir allerdings aufgefallen: Ich musste gestern den LiquidCheck mal vom Strom nehmen (noch zu wenig Steckdosen in der Garage) und in der Zeit wurde mir im HA dann ein Pegelstand von 0 angezeigt. Sobald das Device wieder online ist, kommt der richtige Stand zurück. Das führt dazu, dass in meinem Chart, der mir die Füllstand-Differenzen anzeigt auf einmal zwei dicke Balken nach unten und oben auftauchen.

Ich bin mir nicht sicher, was hier die schlauste Lösung wäre, ob man vielleicht als Wert sowas wie "anavailable" liefern kann, oder ob man evtl den letzten bekannten Wert liefert. Ich glaube das Erste wäre "richtiger"?

image

shivan commented 2 months ago

Da geb ich dir recht. Unavailable wäre bestimmt passender. Muss ich mal nachlesen, ob man sowas liefern kann - oder NULL (nicht 0). Denn es soll ja heißen unbekannter Wert. Ich weiß nur noch nicht, wann ich dazu komme.

FlorianWilk commented 2 months ago

Ich werde auch mal nachlesen und dir evtl einen PR fertig machen. Aber, same here, muss auch schauen, ob/wann ich dazu komme :)