New Module for Solax Gen5 #1972
Conversation
Added new Module vor Solax GEN5 Inverters with up to 3 PV Strings and up to 2 Batteries
Error: packages/modules/devices/solax/solax_gen5/inverter.py:3:1: F401 'pymodbus.constants.Endian' imported but unused
Error: packages/modules/devices/solax/solax_gen5/bat.py:36:15: E111 indentation is not a multiple of 4 Error: packages/modules/devices/solax/solax_gen5/bat.py:38:15: E111 indentation is not a multiple of 4 Error: packages/modules/devices/solax/solax_gen5/bat.py:41:15: E111 indentation is not a multiple of 4 Error: packages/modules/devices/solax/solax_gen5/bat.py:42:15: E111 indentation is not a multiple of 4 Error: packages/modules/devices/solax/solax_gen5/bat.py:44:15: E111 indentation is not a multiple of 4 Error: packages/modules/devices/solax/solax_gen5/bat.py:54:1: E305 expected 2 blank lines after class or function definition, found 1
Error: packages/modules/devices/solax/solax_gen5/counter.py:31:121: E501 line too long (138 > 120 characters) Error: packages/modules/devices/solax/solax_gen5/counter.py:46:29: E122 continuation line missing indentation or outdented Error: packages/modules/devices/solax/solax_gen5/counter.py:50:22: E124 closing bracket does not match visual indentation Warning: packages/modules/devices/solax/solax_gen5/counter.py:51:1: W293 blank line contains whitespace
Error: packages/modules/devices/solax/solax_gen5/device.py:10:121: E501 line too long (127 > 120 characters)
| power_pv3 = self.__tcp_client.read_input_registers(292, ModbusDataType.UINT_16, unit=self.__modbus_id) * -1 | ||
| power_temp = (power_pv1, power_pv2, power_pv3) | ||
| power = sum(power_temp) | ||
| exported = self.__tcp_client.read_input_registers(80, ModbusDataType.UINT_16, unit=self.__modbus_id) * 100 |
There was a problem hiding this comment.
Ist beim Solax Gen5 Register 80 der Gesamt-Ertrag oder der Tagesertrag?
There was a problem hiding this comment.
Das ist Tagesertrag
Oder muss hier der Gesamt-Ertrag hinein?
There was a problem hiding this comment.
Das ist der Gesamtertrag.
There was a problem hiding this comment.
Ok, habe ich angepasst.
Dann hätte ich gleich noch eine Frage. Im counter lese ich auch die Ströme mit aus. Die Ströme passen aber nicht 100%ig zu den berechneten (P/U) Das dürfte so wie ich es am Zähler direkt gesehen habe mit der Blindleistung zusammenhängen). Ich habe die register für die Leistung aus dem original Solax-Modul übernommen, und da wird eben rein mit der Einspeiseleistung gearbeitet Wie ist es denn mehr oder weniger gewünscht bzw. Wie wird es bei den meisten Modulen im counter gemacht? Soll der Strom einfach berechnet werden, oder sollen wenn möglich die tatsächlichen Werte verwendet werden?
There was a problem hiding this comment.
Von der Regelung wird aktuell bei den WR und Speichern nur die Leistung verwendet. Du kannst aber z. B. die Ströme gerne mit auslesen. Die müssen auch nicht 1:1 zur Wirkleistung passen.
There was a problem hiding this comment.
Dann lass ich die Ströme mit drin.
There was a problem hiding this comment.
Viele Register stimmen mit dem Ursprungsmodul überein. Es liegen hier nur vereinzelte Registerabweichungen vor (bspw zusätzliches Erfassen von PV3 Power; SoC Register 302 statt 28; Auslesen der Gesamtzählerstände).
Es wäre besser das als zusätzliche Option zur Auswahl im bestehenden Modul anzubieten statt ein eigenes Modul zu erstellen (auch um redundanten Code zu vermeiden).
There was a problem hiding this comment.
@ndrsnhs Ist auch ein Ansatz. Wo kann man denn die UI für die Einstellungsseite in einem laufenden System zum testen anpassen?
Changed exported from daily register to total register
Error: packages/modules/devices/solax/solax_gen5/bat.py:41:121: E501 line too long (122 > 120 characters) Error: packages/modules/devices/solax/solax_gen5/bat.py:42:121: E501 line too long (122 > 120 characters) Error: packages/modules/devices/solax/solax_gen5/bat.py:54:1: E305 expected 2 blank lines after class or function definition, found 1 Warning: packages/modules/devices/solax/solax_gen5/bat.py:55:1: W391 blank line at end of file
Added new Module vor Solax GEN5 Inverters with up to 3 PV Strings and up to 2 Batteries
Adresses Issue 1919
UI: openwb-ui-settings/pull/570