Qcells active battery control

[Xoffroad]

adding active battery control for qcells and new field for overload security

[Xoffroad]

Neues Feld entfernt und auf die Nutzung von max_charge_power und max_discharge_power umgebaut.

[seaspotter]

Mir sei nochmal der Kommentar erlaubt, dass du damit ja im Prinzip jetzt keine zusätzliche Sicherheitsebene einbaust sondern die bereits vorhandene und funktionierende duplizierst und damit ist sie eigentlich unnötig.
Denn schon in der Regelung in ttps://github.com/openWB/core/blob/f3983a443e8f331c1399aeb428f189872185f2ef/packages/control/bat_all.py ist ja sichergestellt das zum einen wenn keine Max Lade-Entladeleistung definiert ist, eine Warnung kommt und zudem das der übergebene power_limit Wert ans Batteriemodul niemals über der Lade- oder der Entladeleistung liegt. Meines Erachtens ist das damit unötig.
In anderen Modulen hat sich zudem power_value etabliert, clamped als Wert ist nicht wirklich sprechend.

[Xoffroad]

Mir sei nochmal der Kommentar erlaubt, dass du damit ja im Prinzip jetzt keine zusätzliche Sicherheitsebene einbaust sondern die bereits vorhandene und funktionierende duplizierst und damit ist sie eigentlich unnötig. Denn schon in der Regelung in ttps://github.com/openWB/core/blob/f3983a443e8f331c1399aeb428f189872185f2ef/packages/control/bat_all.py ist ja sichergestellt das zum einen wenn keine Max Lade-Entladeleistung definiert ist, eine Warnung kommt und zudem das der übergebene power_limit Wert ans Batteriemodul niemals über der Lade- oder der Entladeleistung liegt. Meines Erachtens ist das damit unötig. In anderen Modulen hat sich zudem power_value etabliert, clamped als Wert ist nicht wirklich sprechend.

Jein, mir ist bewusst, das dies an anderer Stelle schon existiert. Ich hatte den Code tatsächlich 1:1 so wie deine Anmerkung jetzt. Aber dann viel mir ein, man kann ja über MQTT die Daten manipulieren und die angesprochene Ermittlung könnte auch mal fehlerhaft sein. Es sollte eine zusätzliche Absicherung sein. Ich habe gestern nochmal die Remotesteuerung von QCells/Solax geprüft. Es scheint so oder so nicht möglich zu sein per Remotecontrol den Wechselrichter dazu zu bringen, Werte zu akzeptieren die außerhalb der festen Einstellungen im Wechselrichter selbst liegen. Solange der Anwender also den Wechselrichter passend zu den Spezifikationen der Batterie konfiguriert hat, ist eine Manipulation außerhalb dieser definierten Grenzen über die Befehle die ich nutze nicht möglich. Somit braucht es die Sicherheitsebene überhaupt nicht.

Edit: die Hinweise in der Oberfläche am Batteriemodul würde ich aber gern lassen, siehe UI Repo.