Ignoriere Null-Werte für numerische Datenpunkte

[petterpet]

Describe the bug
Ich möchte von numerischen Datenpunkten den Durchschnitt sowie Min- & Max-Werte haben. Vor jeder Aktualisierung der Datenpunkte werden diese jedoch auf Null gesetzt. Somit ist der Min-Wert immer Null und auch der Durchschnitt verfälscht.

To Reproduce
Dies ist bei den Spritpreisen des Tankerkönig-Adapters der Fall.

Expected behavior
Ich wünsche mir einen Haken (Checkbox), mit der man aktivieren kann, dass Null-Werte ignoriert werden.

Im konkreten Fall mit dem Tankerkönig-Adapter kann der Spritpreis nun mal nie Null werden.

Screenshots & Logfiles

Versions:

• Adapter version: 2.1.1

[klein0r]

Auf null (deleteState) oder auf 0? Numerisch 0 kann ich ja schlecht ignorieren

[petterpet]

Numerische 0

Wieso kann man das nicht ignorieren?

[klein0r]

Weil es tausende Beispiele gibt, wo 0 sinnvoll wäre. Außentemperatur z.B.

[klein0r]

Ich würde im Tankerkönig-Adapter diese Einstellung entfernen um das Verhalten zu umgehen.

[petterpet]

Weil es tausende Beispiele gibt, wo 0 sinnvoll wäre. Außentemperatur z.B.

Korrekt. Deshalb möchte ich ja auch nicht, dass die 0 immer ignoriert wird.
Sondern eine Checkbox, mit der dies bei Bedarf aktiviert werden kann.

Ich würde im Tankerkönig-Adapter diese Einstellung entfernen um das Verhalten zu umgehen.

Beeinflusst das den Tankerkönig nicht negativ? Denn soweit ich mich erinnern kann ist der Haken im default gesetzt.

[klein0r]

Beeinflusst das den Tankerkönig nicht negativ?

Soweit ich weiß geht es nur darum, dass man so vermeiden möchte, dass "alte Werte" stehen bleiben, wenn der Dienst nicht verfügbar ist. Besser wäre es, wenn der Tankerkönig Adapter die Werte löschen würde (deleteState). ODER es müsste mit dem q-Flag gearbeitet werden. Da könnte man wirklich drüber sprechen, dass nur state.q == 0 vom statistics Adapter berücksichtigt wird.

Meiner Meinung nach versucht der Issue hier ein Problem zu beheben, welches ein anderer Adapter verursacht. Wenn der 0 als gültigen Wert schreibt, dann sollte man damit auch arbeiten dürfen.

[klein0r]

Beeinflusst das den Tankerkönig nicht negativ?

Hier die Logik. Würde nicht sagen, dass das den Adapter "negativ beeinflusst". https://github.com/Pix---/ioBroker.tankerkoenig/blob/720b9fef07ee0d4ae11e517a161c1330158dfeac/main.js#L155-L182

Ich würde den Haken entfernen und das Problem ist gelöst.

[petterpet]

Meiner Meinung nach versucht der Issue hier ein Problem zu beheben, welches ein anderer Adapter verursacht. Wenn der 0 als gültigen Wert schreibt, dann sollte man damit auch arbeiten dürfen.

Naja, in den Datenbank-Adaptern gibt es die angefragte Checkbox "Ignoriere Null-Werte" auch.
Die Implementierung des q-Flags in beiden Adaptern wäre meiner Meinung nach die sauberste Lösung.

Ich würde den Haken entfernen und das Problem ist gelöst.

Werde ich machen. Da nur die billigsten Werte resettet werden glaube ich auch nicht mehr das so viel schiefgehen kann. Danke für das Codesnipsel.

[klein0r]

Da nur die billigsten Werte resettet werden glaube ich auch nicht mehr das so viel schiefgehen kann.

Nein, ein paar Zeilen tiefer wird auch der rest mit dem flag auf 0 gesetzt:

https://github.com/Pix---/ioBroker.tankerkoenig/blob/720b9fef07ee0d4ae11e517a161c1330158dfeac/main.js#L190-L207

[xXBJXx]

Ich werde das auf 0 setzen, mit dem q-flag in dem umgebauten Tankerkönig Adapter einbauen, muss mal schauen, wo ich mich ein wenig über den q-flag einlesen kann.

Und der PR muss natürlich noch erst gemerge werden, wo auch schon eine ganze weile, keine fortschritte ersichtlich sind.

[klein0r]

Ich werde das auf 0 setzen, mit dem q-flag in dem umgebauten Tankerkönig Adapter einbauen

Bevor Du da Zeit investierst, schau mal in die Pull Requests - aktuell ist jemand dabei den Adapter komplett zu überarbeiten.

[xXBJXx]

ja das bin ich

[klein0r]

Ach, lustig. Sorry ich habe nicht auf die Namen geachtet 😄