-
-
Notifications
You must be signed in to change notification settings - Fork 641
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
fix sunspec-hybrid #16291
fix sunspec-hybrid #16291
Conversation
The template assumed an inverter with exactly one actual MPPT tracker, and where the second MPPT tracker connected to the battery. It did not use the actual sunspec battery model. After the change the battery model is used for reading Soc and powers. Active battery control is also added based on the 802 model.
Bei einem Hybrid WR kannst Du die PV Leistung nicht auf der AC Seite ermitteln. Welches konkrete Problem soll denn hier gelöst werden? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich habe einen hybrid WR mit 4 Strings. Die vorherige Heuristik funktioniert da schlicht nicht.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Die PV Leistung beinhaltet hier in der Tat ggf die Batterie Ladung/Entladung, wird bei mir aber korrekt bei der Berechnung der Produktion nicht doppelt gezählt. Daher der scale: -1
bei der Batterie.
Das wird so nicht universell funktionieren, denn Um welchen WR geht es denn überhaupt ganz konkret? Im Zweifelsfall einfach dafür ein neues Template anlegen. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Konkret geht es um einen Kontron SolBrid 10-3-4 mit Unterstützung für folgende SunSpec Modelle:
- 1 "Common"
- 103 "Inverter (Three Phase) "
- 120 "Nameplate"
- 123 "Immediate Controls"
- 160 "Multiple MPPT Inverter Extension Model"
- 203 "wye-connect three phase (abcn) meter"
- 802 "Battery Base Model"
Ich würde argumentieren, dass der Sunspec Support möglichst generisch sein sollte, und nicht auf WR mit genau 2 MPPTs beschränkt.
Alternativvorschlag für die battery: Das sunspec-battery-control
template nutzt schon das Model 802. Könnte man dort power
auf Model 802 umstellen? Dann würde sunspec-hybrid
:grid
, sunspec-inverter
:pv
, sunspec-battery-control
:battery
für meinen Fall funktionieren. Weniger ergonomisch, aber auch weniger Änderungen.
Ist auch im PR nach wie vor DC, nur nicht als Summe der Strings, sondern gesamt. |
DCW finde ich ne gute Idee, aber WH ist AC, oder? |
WH ist AC, da gibt es leider kein DC Äquivalent. |
160:x:DCWH |
@premultiply Ich meine aggregiert, ohne zu wissen, wie viele MPPTs es gibt. |
Das gibt es bei SunSpec so mWn nicht. Du solltest einfach ein Template für dein WR-Modell bauen.
Das klingt zusätzlich nach einer guten Verbesserung. Ich mache hier mal zu. Gerne zwei neue PRs für die beiden Themen. |
Ich finde die Idee spezfischer Templates gut. Hier ist eines für einen Hybrid WR mit 2 Strings. Das könnten wir auch so umbenennen. Die Idee einzelne Parameter finde ich auch super- wenn wir passende Lösungen finden :) Jetzt bräuchte es ein weiteres Template für einen 4-String WR oder eine Möglichkeit, dieses Template konfigurieren zu können. |
@andig model 160 liefert die Anzahl Strings, man könnte das also prinzipiell generisch gestalten. Allerdings läuft das templating zur compile Zeit. Hast Du eine Idee, wie man halbwegs einfach die MPPTs dynamisch generieren kann? |
Leider nein. Man könnte aber ein 2 und ein 4 MPPT Template anbieten. |
The template assumed an inverter with exactly one actual MPPT tracker, and where the second MPPT tracker connected to the battery. It did not use the actual sunspec battery model.
After the change the battery model is used for reading Soc and powers. Active battery control is also added based on the 802 model.