Skip to content

fix: add minimum energy deviation to PeakFilter for qcells inverter#3333

Open
Xoffroad wants to merge 1 commit into
openWB:masterfrom
Xoffroad:fix_peakfilter_min_step
Open

fix: add minimum energy deviation to PeakFilter for qcells inverter#3333
Xoffroad wants to merge 1 commit into
openWB:masterfrom
Xoffroad:fix_peakfilter_min_step

Conversation

@Xoffroad
Copy link
Copy Markdown
Contributor

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.

@LKuemmel LKuemmel requested a review from ndrsnhs May 5, 2026 09:06
@ndrsnhs
Copy link
Copy Markdown
Contributor

ndrsnhs commented May 20, 2026

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?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants