fix: add minimum energy deviation to PeakFilter for qcells inverter

[Xoffroad]

Problem: PeakFilter berechnet die maximal erlaubte Energieabweichung anhand der Leistung des Wechselrichters. Es gibt aber Wechselrichter, die die Daten nicht sofort senden sondern in einer Mindestschrittweite. Für diese wirft der Peakfilter somit Fehler/Warnungen, sobald der Wechselrichter an die entsprechende Mindestschrittweite gezählt hat und diese Änderung nun an die openWB sendet.

• PeakFilter unterstützt jetzt einen optionalen Mindestwert für die erlaubte Energieabweichung.
• Für den QCells-Inverter wird 100Wh gesetzt, damit gültige Registersprünge nicht mehr fälschlich gefiltert werden.

[ndrsnhs]

Ich sehe das ein bisschen mit gemischten Gefühlen und ich möchte auch erklären wieso:
Als der Peakfilter geschrieben wurde wurde die QCells Besonderheit bereits bedacht. QCells (auch ein paar andere) erhöhen die Zählerstände stufenweise. Das ist dann de facto ein Peak.
In der aktuellen Logik wird dieser erkannt und im nächsten Durchlauf übernommen. Die Funktionalität ist also nicht beeinträchtigt es passiert einfach eine Regelschleife später, was bei einem Zähler der so ungenau ist dann auch nicht mehr wehtut (meine Meinung).
Dass die Warnung unter Status möglichweise irritiert da gehe ich mit. Die Lösung die Warnung nicht anzuzeigen ist so aber nicht allgemeingültig.
Es gibt auch QCells Anlagen bei denen der Zähler um 1,2kW springt (einige GoodWe machen das glaube ich auch in der Größenordnung). Die würden bei einem Wert von 100 auch weiterhin Warnungen zeigen.

Man könnte auch über die Sinnhaftigkeit der Anzeige überhaupt nachdenken. Wenn man eine "normale" Anlage mit einmaligen Peak hat wird man den unter Status schwerlich genau zu dem Zeitpunkt wo er Auftritt erwischen. Wenn wiederholte, "übliche" Peaks nicht als Warnung sichtbar sein sollen wäre es ggfs. sinnvoller einfach die Warnung aus dem UI zu entfernen und es rein in den Debug Logs zu belassen.

Was meinst du?

[Xoffroad]

Die Funktionalität ist also nicht beeinträchtigt es passiert einfach eine Regelschleife später, was bei einem Zähler der so ungenau ist dann auch nicht mehr wehtut (meine Meinung).

Es geht hier um den Zähler des PV Ertrags, der muss auch nicht wirklich genauer sein. Die Übernahme habe ich auch beobachtet und das passt ja auch.
Mittlerweile bin ich aber auch nicht sicher, ob dies unter Umständen ein Grund ist (definitiv nicht der Einzige nach meinen Analysen), warum es bei der aktiven Speichersteuerung zu Regelproblemen kommt. Denn die openWB reagiert jetzt erst im nächsten Regelintervall auf geänderte PV Leistung und dies grob alle 60s in meinem Fall.

Man könnte auch über die Sinnhaftigkeit der Anzeige überhaupt nachdenken. Wenn man eine "normale" Anlage mit einmaligen Peak hat wird man den unter Status schwerlich genau zu dem Zeitpunkt wo er Auftritt erwischen. Wenn wiederholte, "übliche" Peaks nicht als Warnung sichtbar sein sollen wäre es ggfs. sinnvoller einfach die Warnung aus dem UI zu entfernen und es rein in den Debug Logs zu belassen.

Ja, das wäre als Alternative sinnvoll. In meinem Fall ist die Wahrscheinlichkeit gar nicht so gering, da der Zustand der Warnung grob aller 60-70s auftritt. Also immer dann, wenn der Wechselrichter seinen 100W Schritt für die PV Leistung voll hat. Das bedeutet, der Peak ist nicht einmalig sondern kontinuierlich.

Ich wollte mit der Lösung erreichen, dass unplausible Peaks gemeldet werden, aber Peake die aus der Standard-Schrittweite des Wechselrichters resultieren nicht. Die dürfte grundsätzlich nicht abweichen, zumindest für ein konkretes Modell.