Conversation
|
@cr0i Magst Du Dir die Änderung ansehen bzw vielleicht sogar den Feature-Branch testen? |
d53b29c to
9b2e7c6
Compare
|
Enthält es auch einen Fix für Batterie Modbus Peaks? :-) Vielen lieben Dank |
Ich habe den Feature Branch mal ausprobiert, power und soc werden artig abgerufen. Die Speichersteuerung funktioniert damit noch nicht. Ist es vielleicht das hier? |
dcb6ddf to
15951ab
Compare
|
Danke für Deine Rückmeldung.
Nein, das ist nicht gewollt. Ich habe nicht gesehen, das Copilot das beim Review eingebaut hat. Ist entfernt.
Die Fehlerbehandlung war nicht getrennt von den IO-Modulen und das hat einen Fehler geworfen und dann wurde auch das Speicher-Limit nicht mehr gesetzt. Ich habe es korrigiert.
Habe ich mit einem Rebase auf den Master behoben. Würdest Du bitte nochmal probieren? |
ist in Arbeit, hat aber nichts mit dem Thema dieses PRs zu tun. |
|
Hallo @LKuemmel , jetzt kommt zwar keine Fehlermeldung mehr, aber trotzdem sehe ich keine Logeinträge von set_power_limit. |
|
Für set_power_limit fehlt noch eine Fehlerbehandlung, da wird offenbar der Fehler verschluckt. Wir bauen das erst ein, dann schaue ich hier nochmal, dann ist die Fehlersuche leichter. |
Bei #3166 ist mir aufgefallen, dass der SolarEdge-Speicher keine Register-Blöcke ausliest. SolarEdge empfiehlt das aber.