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

Sunspec: add battery control #10933

Merged
merged 2 commits into from
Jan 28, 2024
Merged

Sunspec: add battery control #10933

merged 2 commits into from
Jan 28, 2024

Conversation

andig
Copy link
Member

@andig andig commented Nov 27, 2023

Fix #10912. Depends on #11663.

TODO

  • Timeout (wohin?)

Out of scope

  • scale factors

@andig andig marked this pull request as draft November 27, 2023 08:08
@andig andig requested a review from premultiply November 27, 2023 08:08
@andig andig added the enhancement New feature or request label Nov 27, 2023
@andig andig force-pushed the feature/sunspec-battery-control branch from dfc1c5e to 1ecdd0a Compare November 27, 2023 08:09
@premultiply
Copy link
Member

sunspec-inverter auch, oder?
Da fallen auch reine Batterie-WR wie SI oder SBS drunter.

@andig
Copy link
Member Author

andig commented Nov 27, 2023

Kann gut sein. Bzw: nein. Hybrid=WR mit Batterie für uns. Für den SBS sollte es eigentlich das Hybridtemplate sein?

@premultiply
Copy link
Member

Sowohl Sunny Island wie auch Sunny Boy Storage können nur AC - also wie ein PV-WR. Das sind definitv keine Hybrid-WR.
Bisher laufen die komplett via Speedwire als sma-inverter. Hier würden wir dann dafür erstmalig auf Modbus umstellen.

@andig andig force-pushed the feature/sunspec-battery-control branch 2 times, most recently from 90b8243 to 4b740a7 Compare November 30, 2023 05:41
@andig andig closed this Dec 21, 2023
@andig
Copy link
Member Author

andig commented Dec 25, 2023

Did a quick test with Fronius Model 124- at least no error messages. Reopening.

@andig andig reopened this Dec 25, 2023
@github-actions github-actions bot added the stale Outdated and ready to close label Jan 15, 2024
@andig andig removed the stale Outdated and ready to close label Jan 20, 2024
@andig andig force-pushed the feature/sunspec-battery-control branch 2 times, most recently from d548707 to e004823 Compare January 27, 2024 10:59
@andig
Copy link
Member Author

andig commented Jan 27, 2024

@premultiply hast Du eine Idee, wir wir das für Fronius zugänglich machen? Die 124 mit ins Fronius Template übernehmen oder (gefällt mir besser) im sunspec Template auch den Fronius mit exakt dem gleichen Text ergänzen?

/cc @naltatis

@andig andig marked this pull request as ready for review January 28, 2024 11:02
template: sunspec-inverter-control
products:
- description:
de: Sunspec Batterie (Model 124)
Copy link
Member

Choose a reason for hiding this comment

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

Puh, das ist arg technisch.
Die Menge der Anwender die das Model 124 kennt dürfte sehr gering sein. :)
Schon bei "SunSpec" dürften wohl die Meisten ausgestiegen sein.

Copy link
Member Author

Choose a reason for hiding this comment

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

Isso- aber lass uns das doch zum testen mal probieren. Bisher kennen wir ja überhaupt nur Fronius als relevanten WR?

@premultiply
Copy link
Member

@premultiply hast Du eine Idee, wir wir das für Fronius zugänglich machen? Die 124 mit ins Fronius Template übernehmen oder (gefällt mir besser) im sunspec Template auch den Fronius mit exakt dem gleichen Text ergänzen?

Nicht lachen aber was würdest du für sowas und ähnliche Fälle von einer Art Alias-Templates halten, die im wesentlichen nur auf ein Basistemplate verweisen (wie hier z.B. SunSpec-Inverter) aber eben zusätzlich dieses um weitere gerätespezifische Anmerkungen und nutzerverständlichere Typenbezeichnungen ergänzt?
Somit würde man für gängige SunSpec-Inverter zusätzliche Marken-Aliastemplates mit eigenem Modelleintrag und spezifischen Konfigurationshinweisen dazu anlegen können.

@andig andig force-pushed the feature/sunspec-battery-control branch 2 times, most recently from d61c20e to 2d20157 Compare January 28, 2024 15:35
@andig andig force-pushed the feature/sunspec-battery-control branch from 2d20157 to f18bb6b Compare January 28, 2024 15:36
@andig
Copy link
Member Author

andig commented Jan 28, 2024

Geil wärs, aber da müssen wir in Ruhe überlegen wie das minimalinvasiv möglich wäre.

@andig andig merged commit a8424b0 into master Jan 28, 2024
6 checks passed
@andig andig deleted the feature/sunspec-battery-control branch January 28, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Home Battery Control: SunSpec (Model 124/802)
2 participants