Ve.Direct improvement: take the Victron load output into account when calculating effectiveness

[SW-Niko]

On MPPT with used DC load output the load current is now taken into account when calculating effectiveness.

Switched from Moving Average to Weighted Average (Advantage: We do not need to store historical values)

[SW-Niko]

Hallo @schlimmchen ,
jetzt funktioniert die Berechnung auch wenn der Lastausgang aktiv ist.
Es kommen 183 W von den Solarplatten und davon werden 84W in die Batterie geliefert. Bei einem Wirkungsgrad von 96,4 % gehen also 92,4W in die Last.

Ich habe langsam Zweifel ob unsere Entscheidung von float auf Ganzzahlen umzusteigen eine gute Idee war.
Ein float benötigt den gleichen Speicherplatz wie ein uint32_t und anstatt sofort in float umzuwandeln wandeln wir nun in der Web page in Mqtt und teilweise bei weiteren Berechnungen.
Vielleicht sollten wir bei Spannungen, Strömen und Leistungen wieder auf float zurück gehen?

[schlimmchen]

Ich bin mit der Änderung am MovingAverage nicht einverstanden. Warum wird das mit diesem Thema verwurstelt? Warum sagst du, es hätte den Vorteil, dass alte Werte nicht gespeichert werden müssen, aber unterschlägst dass der geänderte Ansatz das Verhalten der Mittelwertberechnung deutlich verändert? Beim veränderten Ansatz haben alle alten Werte einen Einfluss auf die Berechnung des Mittelwertes, statt einer begrenzten Anzahl alter Werte. Bis diese Mittelwertberechnung reagiert bei einer Verschattung vergeht meiner Einschätzung nach zu viel Zeit.

[SW-Niko]

Hallo @schlimmchen ,
Ja sorry. Der Wirkungsgrad ist bei mir ziemlich gesprungen und ging teilweise kurzzeitig über 100%. (Was technisch nicht geht)

• Habe gerade erst realisiert das Wirkungsgrad in die Berechnung für den SolarPassthrough eingeht. Dann sollte er auch zeitlich zu der Regelung passen.
• Grob überschlagen ist mein Ansatz zirka 4mal langsamer

Ich muss mir das noch mal genauer anschauen ... aber einfach Zeit für die Mittelwertbildung zu erhöhen ist definitiv der falsche Ansatz.

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.