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

fix sunspec-hybrid #16291

Closed
wants to merge 2 commits into from
Closed

Conversation

runtologist
Copy link
Contributor

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.

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.
@andig
Copy link
Member

andig commented Sep 23, 2024

Bei einem Hybrid WR kannst Du die PV Leistung nicht auf der AC Seite ermitteln. Welches konkrete Problem soll denn hier gelöst werden?

@andig andig marked this pull request as draft September 23, 2024 16:09
Copy link
Contributor Author

@runtologist runtologist left a 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.

Copy link
Contributor Author

@runtologist runtologist left a 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.

@premultiply
Copy link
Member

Das wird so nicht universell funktionieren, denn
nicht alle Hybrid-WR haben Model 802 usw.

Um welchen WR geht es denn überhaupt ganz konkret?

Im Zweifelsfall einfach dafür ein neues Template anlegen.
Für die Mehrzahl der existierenden Hybrid-WR mit zwei MPPTs passt das sunspec-hybrid Template, aber auch dabei gibt es bereits einige Ausnahmen mit eigenem Template.

Copy link
Contributor Author

@runtologist runtologist left a 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.

@runtologist
Copy link
Contributor Author

runtologist commented Sep 23, 2024

Bei einem Hybrid WR kannst Du die PV Leistung nicht auf der AC Seite ermitteln.

Ist auch im PR nach wie vor DC, nur nicht als Summe der Strings, sondern gesamt.

@andig
Copy link
Member

andig commented Sep 23, 2024

DCW finde ich ne gute Idee, aber WH ist AC, oder?

@runtologist
Copy link
Contributor Author

WH ist AC, da gibt es leider kein DC Äquivalent.

@premultiply
Copy link
Member

160:x:DCWH

@runtologist
Copy link
Contributor Author

@premultiply Ich meine aggregiert, ohne zu wissen, wie viele MPPTs es gibt.

@premultiply
Copy link
Member

Das gibt es bei SunSpec so mWn nicht.

Du solltest einfach ein Template für dein WR-Modell bauen.

Das sunspec-battery-control template nutzt schon das Model 802. Könnte man dort power auf Model 802 umstellen?

Das klingt zusätzlich nach einer guten Verbesserung.

Ich mache hier mal zu. Gerne zwei neue PRs für die beiden Themen.

@andig
Copy link
Member

andig commented Sep 24, 2024

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.

@runtologist
Copy link
Contributor Author

runtologist commented Sep 25, 2024

@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?

@andig
Copy link
Member

andig commented Sep 25, 2024

Leider nein. Man könnte aber ein 2 und ein 4 MPPT Template anbieten.

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.

3 participants