-
Notifications
You must be signed in to change notification settings - Fork 107
[ADD] Integration Battery Index for SolarEdge Batteries #2236
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
Changes from all commits
7df8ef0
d56c12c
49fe236
b84c9c2
29ef271
8ca3f51
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||
|---|---|---|---|---|
|
|
@@ -24,8 +24,9 @@ def __init__(self, | |||
|
|
||||
|
|
||||
| class SolaredgeBatConfiguration: | ||||
| def __init__(self, modbus_id: int = 1): | ||||
| def __init__(self, modbus_id: int = 1, battery_index: int = 1): | ||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Bei bestehenden SolarEdge-Speichern fehlt die Einstellung battery_index, diese muss in der Konfiguration noch ergänzt werden. Analog zu hier: core/packages/helpermodules/update_config.py Line 1872 in c12eae8
Außerdem in Z. 54 die DATASTORE_VERSION um eins hochzählen und deine Funktion ans Ende der Datei setzen und die Versionszahlen analog zu den anderen Funktionen hochzählen. Beim Starten wird die DATASTORE_VERSION geprüft und wenn diese nicht aktuell ist, die Funktionen ausgeführt und so die Konfiguration geupdatet.
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Es fehlt noch eine neue Methode in der |
||||
| self.modbus_id = modbus_id | ||||
| self.battery_index = battery_index | ||||
|
|
||||
|
|
||||
| class SolaredgeBatSetup(ComponentSetup[SolaredgeBatConfiguration]): | ||||
|
|
||||
Uh oh!
There was an error while loading. Please reload this page.