Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sungrow modbus: reading more / correct adresses to complete counter and inverter values #1537

Merged
merged 25 commits into from Apr 19, 2024

Conversation

Mantelinho
Copy link
Contributor

@Mantelinho Mantelinho commented Apr 10, 2024

Die Modbus-Registeradressen für Sungrow Devices wurden vervollständigt (s. packages/modules/devices/sungrow/modbus.md) und in den Devices entsprechend berücksichtigt, falls unterstützt:

  • Sungrow Zähler zeigt jetzt alle Werte pro Phase: Strom, Wirkleistung und Leistungsfaktor hinizugefügt
  • (nur SH* via LAN) Spannungswerte pro Phase sind nun korrekt vom Meter anstatt Ausgangsspannungen vom WR
  • Sungrow Wechselrichter zeigen nun die effektive AC Wirkleistung anstatt der DC Brutto-Leistung

Abhängigkeit zu openwb-ui-settings: Für die Auswahl der korrekten WR-Version inkl. Anschlussart gibt es ein verändertes Dropdown nun auf Device-Ebene. Dazu gibt es einen entsprechenden Pull Request im openwb-ui-settings repro: openwb-ui-settings/pull/443

Für korrekte Werte muss nun anders als bisher unter Lademanagement->Struktur der Speicher innerhalb des SH WRs konfiguriert sein:

Screenshot 2024-04-10 151521

Copy link
Contributor

@LKuemmel LKuemmel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Die Änderungen sehen gut aus, allerdings funktioniert bei allen Sungrow-Benutzern nach diesem Update das Modul nicht mehr, weil die Versions-Einstellung von der Komponente ins Gerät gewandert ist und der Speicher in der Struktur unter den WR geschoben werden muss. Um Frust zu vermeiden, müssen diese Änderungen automatisiert nachgezogen werden.
Dazu in der update_config.py die DATASTORE_VERSION um 1 erhöhen und eine neue upgrade_datastore_ Methode hinzufügen und dort die beiden Änderungen nachziehen.

packages/modules/devices/sungrow/version.py Outdated Show resolved Hide resolved
# Conflicts:
#	packages/helpermodules/update_config.py
@LKuemmel
Copy link
Contributor

Kannst Du bitte noch die Flake8 Anmerkungen umsetzen? Dann merge ich den PR.

@Mantelinho
Copy link
Contributor Author

Mantelinho commented Apr 16, 2024

Kannst Du bitte noch die Flake8 Anmerkungen umsetzen? Dann merge ich den PR.

OK, sollte jetzt passen. Hätten auch ein paar Pushes weniger sein können.. Nach Deinem Approval habe ich noch L#1460 in packages/helpermodules/update_config.py hinzugefügt, um die Kompatibilität bei Umstellung SH -> SH_LAN, SH_WiNet zu erhöhen. SH_WiNet kann ja bis auf die Phasenspannungen in der Zählerkomponente ansonsten alles, was über den LAN Port geht, daher sollte SH_WiNet als Default besser sein.

Eventuell wäre nach dem Update eine Systemmessage hilfreich?
if version == 3: pub_system_message(payload, "Die Konfiguration von Sungrow Geräten wurde aktualisiert. Bitte die Version überprüfen und Speicher im Lademanagement innerhalb der Hybrid-Wechselrichter anordnen", MessageType.INFO)

(Edit) Habe einen entsprechenden Commit angefügt, kannst Du optional mit mergen

packages/helpermodules/update_config.py Outdated Show resolved Hide resolved
@LKuemmel LKuemmel merged commit 89828d4 into openWB:master Apr 19, 2024
1 check passed
@Mantelinho Mantelinho deleted the patch_sungrow_modbus branch April 19, 2024 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants