Inverter Basisklasse

[Awienert]

Hallo Andreas,

großtartiges Projekt und ein super Ansatz, danke dafür. Ich schaue mir gerade die Möglichtkeit einer Integration für Victron Inverter an und habe dabei festgestellt, dass eine gemeinsame Basisklasse für die weitere Integration von Invertern eventuell sinnvoll wäre. Wie siehst du das?

VG
Andreas

[ohAnd]

Hi Andreas,

in kurz, ganz genauso ... ;-)

Durch den Fonius Change vor kurzem hat sich da auch einiges angesammelt, was aufgeräumt gehört... (Tests fehlen hier ja noch ebenso...) Also wenn du hier eine "schöne" Abstraktion hin bekommst, wäre das natürlich die saubere Basis für weitere Inverter Interfaces.

Mangels Zeit stand das für mich bis dato nicht im Fokus ...

Gruss
Andreas

[Awienert]

Hallo Andreas,

ja da muss man einiges umstricken. Aber ich kann mal einen Grundgedanken skizzieren. Den Victron VE habe ich schon testweise eingebaut. Ich würde mich da gern nochmal abstimmen wie die beste vorgehensweise ist. Auch dein Testsetup/Debugging würde mich interessieren, wie Du da vorgehst.

Danke und VG
Andreas

[ohAnd]

Hi Andreas,

vielleicht als Idee und möglicher Weg, wie die Abstraktion der Inverter aussehen könnte ... schau mal in develop den Umbau von eos_interface.py zu optimization_interface.py und die einzelnen backend Klassen an ...
So könnte vielleicht auch die Abstraktion der Inverter aussehen. Sollte zu guter Letzt noch lesbar sein und darüber verständlich ;-) (sofern mir das hier noch gelungen ist)

Test als unit tests über pytest ... -> für den optimizer z.B. test_optimization_interface.py - so mal als Startpunkt.

Sonst über DM ...

Gruss
Andreas

[Awienert]

Hallo Andreas,

ich habe mir das angeschaut. Gibt es hier eine DM Funktion?

VG
Andreas

[ohAnd]

Oh sorry... ganz übersehen.

ich habe mir das angeschaut. Gibt es hier eine DM Funktion?

hatte ich gehofft aber ebenso nicht gefunden ;-)

Vorschlag: ich schau die nächste Zeit mal, das ich da was vorbereite ... und nehme das hier rüber als ticket und erstelle dort einen branch ...

[Awienert]

Ich bin schon relativ weit. Bin gerade dabei den Code der beiden vorhandenen Inverter in das Format der Basisklasse zu überführen. Da kann eventuell sogar wieder was weg, da ich manches nicht so richtig verstanden habe. zum Bsp. warum es 2 Fronius Klassen gibt. Aus dem Code habe ich vermutet, das eine ist ein älteres Gerät. Ich habe lokal einen Branch erstellt. Muss dann noch testen ob sich alles so verhält wie gewünscht... Ich würde dann wenn ich soweit bin den Feature Branch pushen dann kann dieser reviewed werden und diskutiert werden. Was sagst du?