Conversation
Implement AnkerInverter class with initialization and update methods for inverter functionality.
Implement AnkerCounter class for managing counter data.
Initialized SimCounter and updated client references in the update method.
Added device_id to KwargsDict and initialized it in AnkerCounter.
Add device_id to KwargsDict and initialize method
Updated import statements and removed an unused line.
Remove unused import of 'modbus' from inverter.py
| # Register 10002 ist die PV_power also die DC Leistung, eine AC Leistung gibt es so nicht | ||
| power = self.client.read_input_registers(10002, ModbusDataType.INT_32, | ||
| wordorder=Endian.Little, unit=unit) * -1 | ||
|
|
There was a problem hiding this comment.
Der Wechselrichter kann mit der DC-Leistung leider nicht angebunden werden, da alle anderen Leistungen AC sind und dann nicht miteinander verrechnet werden können.
There was a problem hiding this comment.
Ja ist mir bewusst Lena, es gibt noch eine "Load Power" Register 10010 die die AC Leistung sein könnte, aber mangels Hardware kann ich es halt leider nicht testen. Kann man halt dann ggf noch anpassen wenn es mal User mit dem Setup gibt. Es kann aber auch ein ganz anderer Wert sein, daher erscheint es mir dennoch am besten erstmal die PV Leistung zu nehmen?
https://github.com/anker-charging/ha-anker-solix-official/blob/5eb27a1a4616afc4786d0171dd574f0a24224852/custom_components/anker_solix_official/config/8fcbb87c685781b1d70d784a79eb923098955df2aaf199095ce7767bb70b913d.yaml#L127
There was a problem hiding this comment.
Dann nimm Register 10002 für dc_power und "Load Power" für power. Dann ist zumindest das DC-Power-Register an der richtigen Stelle.
Klar, ohne Hardware ist das schwierig für Dich. Eigentlich sollte es recht schnell auffallen, weil Hausverbrauch, Ladekosten, .. nicht passen, aber das wird nicht unbedingt so klar mit einer nicht plausiblen PV-Leistung in Verbindung gebracht.
There was a problem hiding this comment.
Mach ich, aber dann solltet ihr anderswo auch konsequent sein wenn keine DC Leistung als "power" verwendet werden soll :) Siehe: https://github.com/openWB/core/blob/master/packages/modules/devices/solakon/solakon_one/inverter.py#L32
Oder bei Victron wenn die Leistung der MPPTs hergenommen wird, ist das auch nur DC Leistung beim inverter :)
There was a problem hiding this comment.
Danke für den Hinweis. Wir sehen uns die vorhandenen Module mittelfristig nochmal an. Kann gut sein, dass in der Vergangenheit da nicht so sauber gearbeitet wurde.
There was a problem hiding this comment.
Danke für den Hinweis. Wir sehen uns die vorhandenen Module mittelfristig nochmal an. Kann gut sein, dass in der Vergangenheit da nicht so sauber gearbeitet wurde.
Gerne, vllt ist es ja ein denkbarer Ansatz wenns nur die PV Leistung gibt diese zwar im Modul als "dc_power" anzugeben, wenn keine "power" vorhanden ist, ersatzweise dann aber "dc_power" für die Anzeige herzunehmen mit ggf. Infomeldung?
There was a problem hiding this comment.
Wenn Ströme aus der Wirkleistung berechnet werden, sollte die Ungenauigkeit nicht sehr groß sein, da ein WR in der Regel nur sehr wenig Blindleistung produziert. Bei der Wandlung von DC nach AC hingegen treten größere Verluste auf. Die sollte man nicht vernachlässigen.
There was a problem hiding this comment.
Der Kommentar passt irgendwie gar nicht in die Diskussion, was haben die Ströme mit der Leistung zu tun?Es geht ja darum wie verfahren wird wenn eben (wie beim Solakon oder auch bei den Victron MPPTs) gar keine AC Leistung als Register vorhanden ist. So hat Andreas das aber jetzt auch mit dem aktuellen PR (#3294) eingebunden, weil es nur die DC Leistung gibt. Soll laut euch ja aber eigentlich nicht sein?
Wenn es aber keine AC Leistung gibt, ist es doch immer noch besser die DC Leistung zu nehmen statt gar nichts? So ist (vermute ich) auch die Denkweise bei Andreas dazu gewesen. Nur sauberer wäre es ja das dann auch als dc_power zu deklarieren und wenn in einem Invertermodul kein "power" vorhanden ist, dann eben ersatzweise "dc_power" zu nehmen für die Anzeige der Leistung, statt gar nichts. Und dann eben vllt noch als Infomeldung im Modul zu schreiben: "Achtung der Wechselrichter gibt als Leistung nur die PV Leistung aus und kann somit in der Leistungsbilanz etwas abweichen".
Um das noch zu verfollständigen: auch bei allen DC angebundenen Batterien ist "power" immer eine DC Leistung und niemals eine AC Leistung, auch das ist in der Leistungsbilanz nicht ganz sauber :) Aber vermutlich vernachlässigbar.
There was a problem hiding this comment.
Wie schon geschrieben, wir sehen uns das generell nochmal an. Aktuell passt das zeitlich mit den Vorbereitungen des Release 2.2.0 leider nicht.
3 participants
UI: openWB/openwb-ui-settings#953
Anker hat offiziell für die ersten Produkte nun einen Modbus TCP Support per Firmware nachgeliefert. Basierend auf ihrer eigenen HA Integration hier auch als erster Entwurf für openWB:
https://github.com/anker-charging/ha-anker-solix-official/tree/main
Es werden aktuell nur folgende Produkte supported:
Anker SOLIX Smart Plug (hier im PR nicht enthalten, ggf für Consumer Branch relevant)
Anker SOLIX Solarbank Max AC (enthalten im PR)
Anker SOLIX Smart Meter Gen 2 (enthalten im PR)
Mangels Hardware ist es nicht getestet, Anpassungen sind also ggf. möglicherweise notwendig wenn die ersten User die Hardware nutzen.