Sungrow Hybrid: fix charge sign (again) #18473

premultiply · 2025-01-28T17:29:41Z

Combines all "charging" bits of the available registers.

naltatis · 2025-01-30T08:37:20Z

@premultiply ist das schon ein testbarer Stand?

premultiply · 2025-01-30T09:04:46Z

Bislang eher Theorie und Papiertest.
Und mit dem go-Scripting fehlt es mir noch an Grundlage.
Wie gibt man da den Wert zurück?
return ist da glaube ich falsch, oder?

templates/definition/meter/sungrow-hybrid.yaml

naltatis · 2025-02-07T16:10:41Z

This works with my inverter (hold fw).

Can you please try this as well?
@zachelnet @wapman1979 @nekronomekron

naltatis · 2025-02-09T11:57:34Z

@premultiply wir können das sonst auch mergen und Feedback über den Nightly einsammeln. Für die alte Firmware konnte ich ja verifizieren, dass es korrekt funktioniert.

andig · 2025-02-09T12:01:25Z

Nach Release?

nekronomekron · 2025-02-09T13:48:16Z

Wenn mir jemand ein Windows Build mit dem Fix erstellen kann (hab keine fertige Entwicklungsumgebung aufgesetzt in Windows [WSL]), kann ich den Fix gerne mit meinem WR mit der neuen FW testen und Rückmeldung geben.

zachelnet · 2025-02-09T15:19:58Z

This works with my inverter (hold fw).

Can you please try this as well? @zachelnet @wapman1979 @nekronomekron

Looks good with my SH8.0RT

zachelnet · 2025-02-09T15:48:29Z

Wenn mir jemand ein Windows Build mit dem Fix erstellen kann (hab keine fertige Entwicklungsumgebung aufgesetzt in Windows [WSL]), kann ich den Fix gerne mit meinem WR mit der neuen FW testen und Rückmeldung geben.

@nekronomekron klappt bei dir kein make? Gerne kann ich dir ein docker container erstellen.

Alternative kannst du direkt das ganze in die Config einbauen. Schau mal dort einfach copy&past bei battery ersetzten und die IP ändern: #18197 (reply in thread)

nekronomekron · 2025-02-09T18:59:41Z

@zachelnet Danke für den Tipp, den kannte ich noch nicht. Hab die Config direkt in meine evcc.yaml Datei eingebaut und ich konnte EVCC damit starten. Auf meiner Windows Kiste ist derzeit kein WSL eingerichtet, daher habe ich kein make im Moment.

@naltatis Fazit: bei mir hat es leider nicht funktioniert, die Batterieleistung wurde mit falschem Vorzeichen ausgelesen.

Wechselrichter: SH8.0RT-V112
ARM-Firmware: SAPPHIRE-H_01011.95.03
MDSP-Firmware: SAPPHIRE-H_03011.95.03

@zachelnet Kannst du evtl. doch einen Docker Container bauen? Nicht dass ich bei der Config etwas falsch gemacht habe, aber eigentlich hab ich nur Copy&Paste gemacht.

zachelnet · 2025-02-09T20:07:23Z

@zachelnet Danke für den Tipp, den kannte ich noch nicht. Hab die Config direkt in meine evcc.yaml Datei eingebaut und ich konnte EVCC damit starten. Auf meiner Windows Kiste ist derzeit kein WSL eingerichtet, daher habe ich kein make im Moment.

@naltatis Fazit: bei mir hat es leider nicht funktioniert, die Batterieleistung wurde mit falschem Vorzeichen ausgelesen.

Wechselrichter: SH8.0RT-V112 ARM-Firmware: SAPPHIRE-H_01011.95.03 MDSP-Firmware: SAPPHIRE-H_03011.95.03

@zachelnet Kannst du evtl. doch einen Docker Container bauen? Nicht dass ich bei der Config etwas falsch gemacht habe, aber eigentlich hab ich nur Copy&Paste gemacht.

@nekronomekron hab ein Container für dich erstellt (eigentlich sollte es passen da wir die selbe Wechselrichter und Firmware stände haben):
docker pull zachelnet/evcc:master

premultiply · 2025-02-09T20:26:37Z

Und bitte Batterieladung und -entladung testen und ggf. die Ausgabe von evcc meter --log trace,db:error in beiden Fällen beifügen.

nekronomekron · 2025-02-10T08:45:15Z

Ich hab das Docker Image getestet, mit dem gleichen Ergebnis: es wird immer eine Batterieladung angezeigt, keine Entladung.
Anbei ist ein Beispiel Trace Log von mir. Zu Beginn wird die battery 1 mit ein paar Watt geladen. Das passt auch, bei mir ist es relativ bewölkt. Zum Test hab ich das Auto auf "Schnellladen" gestellt, die Batterie wird auch genutzt das kann ich in meinem anderen Dashboard sehen. Bei EVCC wird aber weiterhin eine Ladung angezeigt, das sieht man auch sehr gut in der UI.
Das Log endet auch mit der noch aktiven Schnellladung des Fahrzeugs, die angezeigten 4kW Ladeleistung der Batterie sollten in dem Moment eigentlich eine Entladung sein.

trace_log.txt

premultiply · 2025-02-10T08:54:39Z

Bitte einfach nur die Ausgabe von evcc meter --log trace,db:error in beiden Fällen. Danke!

premultiply · 2025-02-10T09:20:45Z

Laden:

2025-02-10 08:29:43.482314+00:00[battery3] TRACE 2025/02/10 09:29:43 modbus: send 00 03 00 00 00 06 01 04 32 c8 00 01
2025-02-10 08:29:43.487022+00:00[battery3] TRACE 2025/02/10 09:29:43 modbus: recv 00 03 00 00 00 05 01 04 02 00 00
2025-02-10 08:29:43.526996+00:00[battery3] TRACE 2025/02/10 09:29:43 modbus: send 00 06 00 00 00 06 01 04 32 dc 00 01
2025-02-10 08:29:43.531631+00:00[battery3] TRACE 2025/02/10 09:29:43 modbus: recv 00 06 00 00 00 05 01 04 02 ff e9
2025-02-10 08:29:43.571652+00:00[battery3] TRACE 2025/02/10 09:29:43 modbus: send 00 09 00 00 00 06 01 04 32 dd 00 01
2025-02-10 08:29:43.576325+00:00[battery3] TRACE 2025/02/10 09:29:43 modbus: recv 00 09 00 00 00 05 01 04 02 01 71
2025-02-10 08:29:43.577176+00:00[site  ] DEBUG 2025/02/10 09:29:43 battery 1 power: -369W

Entladen:

2025-02-10 08:36:45.138074+00:00[battery3] TRACE 2025/02/10 09:36:45 modbus: send 02 88 00 00 00 06 01 04 32 c8 00 01
2025-02-10 08:36:45.142908+00:00[battery3] TRACE 2025/02/10 09:36:45 modbus: recv 02 88 00 00 00 05 01 04 02 00 00
2025-02-10 08:36:45.182974+00:00[battery3] TRACE 2025/02/10 09:36:45 modbus: send 02 8b 00 00 00 06 01 04 32 dc 00 01
2025-02-10 08:36:45.187834+00:00[battery3] TRACE 2025/02/10 09:36:45 modbus: recv 02 8b 00 00 00 05 01 04 02 00 df
2025-02-10 08:36:45.240357+00:00[battery3] TRACE 2025/02/10 09:36:45 modbus: send 02 8e 00 00 00 06 01 04 32 dd 00 01
2025-02-10 08:36:45.245238+00:00[battery3] TRACE 2025/02/10 09:36:45 modbus: recv 02 8e 00 00 00 05 01 04 02 10 5c
2025-02-10 08:36:45.246252+00:00[site  ] DEBUG 2025/02/10 09:36:45 battery 1 power: -4188W

Hier wurde das Vorzeichen des Batteriestroms falsch interpretiert.

naltatis · 2025-02-10T09:31:43Z

Mit der alten SG-Firmware ist das Verhalten nach deiner Änderung weiterhin korrekt:

Laden:

[sungrow_battery] TRACE 2025/02/10 10:29:44 modbus: send 00 01 00 00 00 06 01 04 32 c8 00 01
[sungrow_battery] TRACE 2025/02/10 10:29:44 modbus: recv 00 01 00 00 00 05 01 04 02 00 2b
[sungrow_battery] TRACE 2025/02/10 10:29:44 modbus: send 00 02 00 00 00 06 01 04 32 dc 00 01
[sungrow_battery] TRACE 2025/02/10 10:29:44 modbus: recv 00 02 00 00 00 05 01 04 02 01 26
[sungrow_battery] TRACE 2025/02/10 10:29:44 modbus: send 00 03 00 00 00 06 01 04 32 dd 00 01
[sungrow_battery] TRACE 2025/02/10 10:29:44 modbus: recv 00 03 00 00 00 05 01 04 02 1f 08
Power:        -7944W

Entladen:

[sungrow_battery] TRACE 2025/02/10 10:30:33 modbus: send 00 01 00 00 00 06 01 04 32 c8 00 01
[sungrow_battery] TRACE 2025/02/10 10:30:33 modbus: recv 00 01 00 00 00 05 01 04 02 00 2d
[sungrow_battery] TRACE 2025/02/10 10:30:33 modbus: send 00 02 00 00 00 06 01 04 32 dc 00 01
[sungrow_battery] TRACE 2025/02/10 10:30:33 modbus: recv 00 02 00 00 00 05 01 04 02 00 65
[sungrow_battery] TRACE 2025/02/10 10:30:33 modbus: send 00 03 00 00 00 06 01 04 32 dd 00 01
[sungrow_battery] TRACE 2025/02/10 10:30:33 modbus: recv 00 03 00 00 00 05 01 04 02 09 51
Power:        2385W

premultiply · 2025-02-10T09:46:46Z

Bitte nochmal testen.
Hab die Logik jetzt (hoffentlich) für beide Fälle passend gedreht.

zachelnet · 2025-02-10T10:04:15Z

Bitte nochmal testen. Hab die Logik jetzt (hoffentlich) für beide Fälle passend gedreht.

habs mal bei mir angebaut und es sieht gut aus
1b4f473a
evcc-20250210-110245-debug.log

naltatis · 2025-02-10T10:05:00Z

Passt immer noch mit der alten FW.

Laden

[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: send 00 01 00 00 00 06 01 04 32 c8 00 01
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: recv 00 01 00 00 00 05 01 04 02 00 2b
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: send 00 02 00 00 00 06 01 04 32 dc 00 01
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: recv 00 02 00 00 00 05 01 04 02 01 26
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: send 00 03 00 00 00 06 01 04 32 dd 00 01
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: recv 00 03 00 00 00 05 01 04 02 1f 29
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: send 00 04 00 00 00 06 01 04 32 e2 00 02
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: recv 00 04 00 00 00 07 01 04 04 6f cc 00 00
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: send 00 05 00 00 00 06 01 04 32 de 00 01
[sungrow_battery] TRACE 2025/02/10 11:04:54 modbus: recv 00 05 00 00 00 05 01 04 02 00 2c
Power:        -7977W

Entladen

[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: send 00 01 00 00 00 06 01 04 32 c8 00 01
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: recv 00 01 00 00 00 05 01 04 02 00 2d
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: send 00 02 00 00 00 06 01 04 32 dc 00 01
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: recv 00 02 00 00 00 05 01 04 02 00 1b
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: send 00 03 00 00 00 06 01 04 32 dd 00 01
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: recv 00 03 00 00 00 05 01 04 02 02 da
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: send 00 04 00 00 00 06 01 04 32 e2 00 02
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: recv 00 04 00 00 00 07 01 04 04 6f cc 00 00
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: send 00 05 00 00 00 06 01 04 32 de 00 01
[sungrow_battery] TRACE 2025/02/10 11:03:56 modbus: recv 00 05 00 00 00 05 01 04 02 00 21
Power:        730W

wip

c11ac42

premultiply requested a review from andig January 28, 2025 17:29

andig added the devices Specific device support label Jan 29, 2025

premultiply mentioned this pull request Jan 30, 2025

Sungrow SH15 Batterie Ladung wird nicht erkannt. #18270

Closed

1 task

zachelnet reviewed Feb 4, 2025

View reviewed changes

templates/definition/meter/sungrow-hybrid.yaml Outdated Show resolved Hide resolved

premultiply and others added 3 commits February 4, 2025 19:18

ws

5f7aacf

fix go script

e53ca4e

better comment location

916b1b6

naltatis marked this pull request as ready for review February 9, 2025 11:56

fix type

98a7f11

reverse to charge state

0eef4be

premultiply requested review from zachelnet and naltatis and removed request for zachelnet February 10, 2025 09:46

premultiply requested a review from zachelnet February 10, 2025 09:47

Merge branch 'master' into fix/sungrow-charge

df0aeec

premultiply merged commit 84b5037 into master Feb 10, 2025
7 checks passed

premultiply deleted the fix/sungrow-charge branch February 10, 2025 10:15

VolkerK62 mentioned this pull request Feb 15, 2025

SUNGROW Batterie Anzeige falsch ("In" Anzeige anstelle "Out") #18838

Open

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sungrow Hybrid: fix charge sign (again) #18473

Sungrow Hybrid: fix charge sign (again) #18473

premultiply commented Jan 28, 2025 •

edited

Loading

naltatis commented Jan 30, 2025

premultiply commented Jan 30, 2025

naltatis commented Feb 7, 2025

naltatis commented Feb 9, 2025

andig commented Feb 9, 2025

nekronomekron commented Feb 9, 2025

zachelnet commented Feb 9, 2025

zachelnet commented Feb 9, 2025 •

edited

Loading

nekronomekron commented Feb 9, 2025

zachelnet commented Feb 9, 2025 •

edited

Loading

premultiply commented Feb 9, 2025 •

edited

Loading

nekronomekron commented Feb 10, 2025

premultiply commented Feb 10, 2025

premultiply commented Feb 10, 2025 •

edited

Loading

naltatis commented Feb 10, 2025 •

edited by premultiply

Loading

premultiply commented Feb 10, 2025

zachelnet commented Feb 10, 2025

naltatis commented Feb 10, 2025 •

edited

Loading

Sungrow Hybrid: fix charge sign (again) #18473

Sungrow Hybrid: fix charge sign (again) #18473

Conversation

premultiply commented Jan 28, 2025 • edited Loading

naltatis commented Jan 30, 2025

premultiply commented Jan 30, 2025

naltatis commented Feb 7, 2025

naltatis commented Feb 9, 2025

andig commented Feb 9, 2025

nekronomekron commented Feb 9, 2025

zachelnet commented Feb 9, 2025

zachelnet commented Feb 9, 2025 • edited Loading

nekronomekron commented Feb 9, 2025

zachelnet commented Feb 9, 2025 • edited Loading

premultiply commented Feb 9, 2025 • edited Loading

nekronomekron commented Feb 10, 2025

premultiply commented Feb 10, 2025

premultiply commented Feb 10, 2025 • edited Loading

naltatis commented Feb 10, 2025 • edited by premultiply Loading

premultiply commented Feb 10, 2025

zachelnet commented Feb 10, 2025

naltatis commented Feb 10, 2025 • edited Loading

premultiply commented Jan 28, 2025 •

edited

Loading

zachelnet commented Feb 9, 2025 •

edited

Loading

zachelnet commented Feb 9, 2025 •

edited

Loading

premultiply commented Feb 9, 2025 •

edited

Loading

premultiply commented Feb 10, 2025 •

edited

Loading

naltatis commented Feb 10, 2025 •

edited by premultiply

Loading

naltatis commented Feb 10, 2025 •

edited

Loading