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

Huawei smartlogger #841

Merged
merged 18 commits into from
Jul 24, 2023
Merged

Huawei smartlogger #841

merged 18 commits into from
Jul 24, 2023

Conversation

sreinhold95
Copy link
Contributor

No description provided.

@sreinhold95 sreinhold95 deleted the huawei_smartlogger branch March 11, 2023 18:20
@sreinhold95 sreinhold95 restored the huawei_smartlogger branch March 11, 2023 18:20
@sreinhold95
Copy link
Contributor Author

Modul für Huawei Smartlogger, nicht für den Dongle, da andere Register für den EVU Zähler
läuft seit 1,5 Wochen bei mir Stabil im Test.
Device Einstellung ist IP Adresse
und pro Komponente die ModBus_ID.
Batterie konnte ich leider nicht testen. habe keine.

@sreinhold95 sreinhold95 reopened this Mar 11, 2023
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 Register des Speichers sind die gleichen wie beim Huawei-Modul. Ist es nicht sinnvoller, Smartlogger-Zähler und -WR als zusätzliche Komponente ins Huawei-Modul aufzunehmen? Ähnlich wie beim Solaredge-Modul die Komponente "Externer Wechselrichter".

@sreinhold95
Copy link
Contributor Author

Hallo,
theoretisch möglich.
Hatte es der übersichtlichkleit halber als extra Gerät programmiert, das es keine Verwechslung gibt.
Die Modbus Register der Wechselrichter sind auch die gleichen.
Es unterscheidet sich der SmartMeter und nach Test evtl. die Batterie.
Ob die Batterie funktioniert kann ich nicht sagen, da ich wie gesagt keine Habe.
Am SmartDongle würden die Abfragen auch funktionieren, dort müssten aber Pausen eingebaut werden, da dieser sonst abstürzt.
Der Smartlogger braucht auch keine ca. 7 Sec sleep um die Verbindung zu öffnen am Anfang, wie der SmartDongle.
Er trennt diese per Timeout, wenn zu lange keine Anfrage kommt.
Das war auch der Grund warum ich den Smartlogger gekauft habe.

@sreinhold95
Copy link
Contributor Author

Habe heute nochmals im Code geschaut,
durch den 7 Sec sleep im Huawei Device geht es leider nicht, das man alles in eins packt.
kann man den Pull Request so wie er ist mergen oder spricht eurer seits noch was dagegen, was unbedingt geändert werden muss?
das Device läuft seit dem Pull Reqeust Datum Fehlerfrei.

@LKuemmel
Copy link
Contributor

Bitte die Datei .github/workflows/manual-workflow.yml aus dem PR entfernen, dann der PR so rein.

@sreinhold95
Copy link
Contributor Author

Die datei ist draußen

@LKuemmel LKuemmel merged commit efb6585 into openWB:master Jul 24, 2023
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.

2 participants