Vorweg: Vielen Dank daß sich das jemand anschaut!
Ich gestehe, daß ich das Ganze bei mir nur mit (m)einem SMA SHM getestet habe.
Die relevante Datei ist also modules/bezug_smashm/sma-em-measurement.py
Ich habe vorab einige Minuten mitgeloggt und dabei nur Pakete mit Länge 608 und 58 gefunden.
Und ich hab das seit gestern parallel am Laufen auf 2 verschiedenen Test-Systemen, einmal mit Bugfix und einmal ohne Bugfix.
Die Graphen sind absolut identisch, außer eben den fehlenden Aussetzern bei einem der beiden.
Die andere Datei modules/smaemd_pv/sma-em-measurement.py hab ich dann gleich mit angepasst, in der offensichtlich irrigen Annahme daß der Ablauf dort identisch ist. Mea culpa.
Wenn der SAM-EM natürlich kürzere Pakete liefert werden dann alle ignoriert.
Und ja, der Fix verwirft einfach alle Pakete kleiner als 608.
Drum erwähnte ich im PR, daß es um einen "Dirty Hack" geht.
Vielleicht sollten wir die Logik umdrehen und explizit die 58er verwerfen?
@ALDI-Tuete
Kannst/Magst Du mal den Traffic des SMA-EM mitschneiden (wireshark) und anschauen oder mir zukommen lassen?
Order magst Du den Code einfach mal in die Richtung Len <> 58 anpassen und dann beim SMA-EM testen?
Ich besitze keinen SMA-EM.
PS: Es gibt ja auch noch eine 3 Stelle, wo Pakete mit 608er Länge erwartet werden, da hab ich aber die Finger von gelassen.
Es gibt jedoch einen PR
#1846 dazu (der ist aber nicht von mir).