Sungrow-Hybrid: fix charge

[premultiply]

Depends on #14960

[naltatis]

Scheint ein Fehler im Template zu sein:

[main  ] FATAL 2024/02/12 15:49:59 cannot create meter 'sungrow_battery': cannot create meter type 'template': cannot create meter type 'custom': battery mode: 1 error(s) decoding:

* '' has invalid keys: set

[premultiply]

@andig Da braucht das Modbus-Plugin erst noch eine passende Set-Erweiterung, oder?

[andig]

Was solls denn tun?

[premultiply]

Wert aus Register A lesen (, skalieren) und dann in Register B schreiben.

So wie mit const-Plugin als Festwert auch.
Nur halt dynamisch via Modbus gelesen.

[andig]

Mhhm. Keine Ahnung ob das mit dem Go Plugin gehen sollte (@GrimmiMeloni ?). Sonst bräuchte es wohl noch etwas Neues (copy). Langsam wirds unübersichtlich :O

[premultiply]

Wie funktioniert das denn bei const?

[andig]

Genauso. Aber in https://github.com/evcc-io/evcc/pull/12215/files#diff-0b0f0388fa0ea5962b68de7565e1131be1912c96e6d198ad57f6b53d77775b17R175 ist es ein modbus Plugin- das hat kein set.

Anders gesagt: const ist eigentlich eine Mischung aus copy und const. Die müssten wir jetzt- wenns orthogonal bleiben soll- aufsplitten und auch gleich alle Templates mit anpassen :O. Dann gäbe es ein eigenständiges copy plugin das hier zu verwenden wäre. Spannend wird dann nochmal, den passenden Typ zu finden bzw. zu matchen, ob input und output zueinander passen. Aber das haben wir ja schonmal irgendwie beim go Plugin gelöst. Müsste dann hier die gleiche Lösung werden.

Long story short: es braucht einen weiteren PR, der const umbaut und aufteilt.

[GrimmiMeloni]

Sorry, in der Ecke habe ich mich noch gar nicht umgeschaut.
Ganz neutral gefragt: was spricht denn gegen den Vorschlag von Premultiply? Der Beschreibung nach doch recht generisch einsetzbar?

[andig]

Welches Problem wird denn hier überhaupt gelöst? Es ist kein Ticket verlinkt.

[andig]

Und was hat das mit #12203 zu tun?

[premultiply]

Die Zwangsladung funktioniert derzeit nicht.
Dies soll dieser PR beheben.

[andig]

@premultiply mit #14855 wird das wieder relevant.

[andig]

@premultiply können wir das bitte testen?

[naltatis]

@andig hier passt noch was nicht

go run main.go --config /Volumes/data/evcc/evcc.yaml meter sungrow_battery -b charge
[main  ] INFO 2024/07/21 11:49:40 evcc 0.0.0
[main  ] INFO 2024/07/21 11:49:40 using config file: /Volumes/data/evcc/evcc.yaml
[db    ] INFO 2024/07/21 11:49:41 using sqlite database: /Users/michael/.evcc/evcc.db
[mqtt  ] INFO 2024/07/21 11:49:41 connecting evcc-1227070342 at tcp://192.168.31.34:1883
[mqtt  ] DEBUG 2024/07/21 11:49:41 tcp://192.168.31.34:1883 connected
[main  ] FATAL 2024/07/21 11:49:41 set battery mode: panic: reflect: call of reflect.Value.Interface on zero Value
exit status 1

[naltatis]

Hier mit mehr Logging.

go run main.go --config /Volumes/data/evcc/evcc.yaml meter sungrow_battery -b charge
[main  ] INFO 2024/07/21 15:05:23 evcc 0.0.0
[main  ] INFO 2024/07/21 15:05:23 using config file: /Volumes/data/evcc/evcc.yaml
[db    ] INFO 2024/07/21 15:05:23 using sqlite database: /Users/michael/.evcc/evcc.db
[modbus] TRACE 2024/07/21 15:05:23 modbus: send 00 01 00 00 00 06 01 06 32 f9 00 02
[modbus] TRACE 2024/07/21 15:05:23 modbus: recv 00 01 00 00 00 06 01 06 32 f9 00 02
[modbus] TRACE 2024/07/21 15:05:23 modbus: send 00 02 00 00 00 06 01 06 32 fa 00 aa
[modbus] TRACE 2024/07/21 15:05:23 modbus: recv 00 02 00 00 00 06 01 06 32 fa 00 aa
handleSetter param: batteryMode = 3
[modbus] TRACE 2024/07/21 15:05:23 modbus: send 00 03 00 00 00 06 01 04 15 fb 00 01
[modbus] TRACE 2024/07/21 15:05:23 modbus: recv 00 03 00 00 00 05 01 04 02 00 64
evaluate <invalid reflect.Value>
[main  ] FATAL 2024/07/21 15:05:23 set battery mode: panic: reflect: call of reflect.Value.Interface on zero Value
exit status 1

Ich hab der (p *Go) evaluate() methode mal nen Logging vor dem Return hinzugefügt. Da kommt <invalid reflect.Value> für v zurück.

[naltatis]

Meine lokalen Tests waren jetzt alle erfolgreich. Charge Mode lädt jetzt wie gewünscht mit voller Leistung aus dem Netz.