refactor SolarEdge EVU for readability and a step toward 2.0 compatibility

[yankee42]

Motiviert dadurch, dass jemand im Forum mit dem Modul Probleme hat.

Mit dem Fehler den der Nutzer da bekommt:

AttributeError: 'ExceptionResponse' object has no attribute 'registers'
    value1 = resp.registers[0] 
  File "/var/www/html/openWB/modules/bezug_solaredge/solaredge.py", line 19, in <module>

Kann man leider Debugtechnisch so gut wie garnichts anfangen.

Früher oder später sollte der garnicht schöne Bestandscode ohnehin überarbeitet werden, insofern ist hier der Umschrieb auf etwas moderner.

Getestet ist das ganze noch nicht. Ich selbst habe leider keinen SE-Modbuszähler.

[okaegi]

Habe e3dc Speicher und Solaredge Wr. Evu verwende ich momentan e3dc, ich kann zum testen auf Solaredge Evu switchen. Komme wahrscheinlich erst nach dem 26 zum testen.
Kann ich die files mit direkt auf openwb/.../bezug_solaredge runterladen ? Bin nicht auf der allerneusten nightly
openWB Version: 1.9.250 (2021-12-14 18:42:34 +0100 [5284837] )
Gruss Oliver

[uwec]

Habe es getestet
https://openwb.de/forum/viewtopic.php?p=53260#p53260
openWB Version: 1.9.250 (2021-12-24 00:50:39 +0100 [b7a12c2] )

EVU Watt: -569.0
EVU Bezug: 6452266
EVU Einspeisung: 1398578

Werte passen vom Betrag, wobei ich aktuell nicht einspeise, sondern beziehe. d.h. das Vorzeichen bei "EVU Watt" passt nicht nd muss noch korrigiert werden.

Auch klasse: (!)
jetzt habe ich auch in der EVU Anzeige Kommazahlen, was bisher nur Ganzzahlen waren. Und die Powerfaktoren werden angezeigt, die bisher nur "0" waren.

[okaegi]

Sieht gut aus:
Testresultat hier
https://openwb.de/forum/viewtopic.php?f=8&t=4433&p=53263#p53263
Einzig das Vorzeichen stimmt nicht bei:
(wattbezug, bezugw1, bezugw2, bezugw3)

[yankee42]

Kann ich die files mit direkt auf openwb/.../bezug_solaredge runterladen ? Bin nicht auf der allerneusten nightly openWB Version: 1.9.250 (2021-12-14 18:42:34 +0100 [5284837] ) Gruss Oliver

Ich bin mir nicht sicher ob ich den Satz korrekt verstehe, aber falls du fragst ob du die Daten in dem von dir genannten Ordner einfach über die Dateien von deiner aktuellen nightly drüber kopieren kannst: Ja, da müsste gehen. Deine aktuelle nightly darf nicht zu alt sein, aber ich denke die vom 14.12. müsste aktuell genug sein, damit das geht.

@uwec, @okaegi: Vielen Dank für eure Tests. In der Tat, das Vorzeichen habe ich übersehen, wenn das der einzige Fehler ist klingt das schonmal gut. Ich habe den PR überarbeitet, so dass das Vorzeichen jetzt auch stimmen müsste.

[uwec]

Update getestet und passt nun alles.
Vielen Dank Yannik!