Skip to content

Add Marstek Battery#2683

Closed
seaspotter wants to merge 16 commits intoopenWB:masterfrom
seaspotter:marstek_bat
Closed

Add Marstek Battery#2683
seaspotter wants to merge 16 commits intoopenWB:masterfrom
seaspotter:marstek_bat

Conversation

@seaspotter
Copy link
Contributor

@seaspotter
Copy link
Contributor Author

Ich bekomme den flake8 Fehler für die beiden init.py nicht weg, weil er immer wieder eine Zeile einfügt?

@LKuemmel
Copy link
Contributor

Die Zeilen könnten durch Auto-Format kommen. Hast Du versucht, Auto-Format abzuschalten oder die Dateien direkt in Github zu formatieren?

@seaspotter
Copy link
Contributor Author

Die Zeilen könnten durch Auto-Format kommen. Hast Du versucht, Auto-Format abzuschalten oder die Dateien direkt in Github zu formatieren?

Ja habs direkt in Github editiert und der packt immer wieder ne leere Zeile rein wenn ich commite. Ich bekomm den Fehler nicht weg, kannst du das evtl machen?

@LKuemmel LKuemmel added this to the 2.1.9 milestone Aug 21, 2025
@LKuemmel LKuemmel modified the milestones: 2.1.9, 2.1.8-Patch.2 Oct 23, 2025
@LKuemmel
Copy link
Contributor

Es ist wirklich blöd gelaufen, dass Ihr zwei PRs zum gleichen Thema gestellt habt. Ich habe mich nun dafür entschieden, den PR von andlem zu mergen, weil dort noch die Schnittstelle zur Speichersteuerung enthalten ist.
Allerdings ist mir aufgefallen, dass Du richtiger Weise noch einen Faktor beim SoC gesetzt hast, der auch in der von dir verlinkten Doku beschreiben wird. Den muss andlem noch einbauen.
Vielen Dank für Deine Arbeit!
Zum Thema Speichersteuerung hatten wir ja kürzlich ein Online Community Meeting veranstaltet und damit nochmal das Konzept verbessert, an dessen Umsetzung ndrsnhs gerade arbeitet.

@LKuemmel LKuemmel closed this Oct 24, 2025
@seaspotter seaspotter deleted the marstek_bat branch October 31, 2025 18:49
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