NosIreland / solismod

15 stars 6 forks source link

error('required argument is not an integer'). Exiting #5

Open D43DBULL opened 3 weeks ago

D43DBULL commented 3 weeks ago

Schermafbeelding 2024-08-22 112936

I get this error after running a MQTT:Publish sctrip in home-assistent.

D43DBULL commented 3 weeks ago

I found out that the S3-WIFI-ST is not compatible. Couldn't realy find out what is compatible. I only found somthing about the "WIFI (en2-8m)", is this the same as the DLS-W? Because if not i can't find any data loggers with this in the name/description.

alienatedsec commented 3 weeks ago

@D43DBULL Currently, there is nothing reliable for HA with S3-WIFI-ST other than custom solutions that do not rely on a datalogger. See this custom solution for instance https://github.com/alienatedsec/solis-ha-modbus-cloud?tab=readme-ov-file#supported-solis-dataloggers

D43DBULL commented 3 weeks ago

I figured that out, but I'm willing to buy a different one (and sell this one) but I want to know which one I need to buy. Is the DLS-W compatible or which one do I need?

alienatedsec commented 3 weeks ago

@D43DBULL Amid hard availability and their obsolete status, the earlier versions of DLS-W used to be compatible with this repo, but some later versions of the same model were not. In fact, these were producing garbage when combined with anything additional than the inverter. More details here https://github.com/StephanJoubert/home_assistant_solarman/issues/120

NosIreland commented 3 weeks ago

Schermafbeelding 2024-08-22 112936

I get this error after running a MQTT:Publish sctrip in home-assistent.

This error is due to new API version. Change paho-mqtt to 1.6.1 or if using requirements use this:

umodbus
paho-mqtt==1.6.1
pysolarmanV5==2.4.0
D43DBULL commented 2 weeks ago

@alienatedsec For now I will only use it for a inverter. The only thing I need it to do is to give me some data and to automatically turn off when my energy prices go below zero and back on when above zero. (I know I need to add this function in the registers.py).

@NosIreland

Change paho-mqtt to 1.6.1 or if using requirements use this:

This did fix the "DeprecationWarning". The error is still there but I think this is because of the incompatible S3-WIFI-ST?

NosIreland commented 2 weeks ago

@alienatedsec For now I will only use it for a inverter. The only thing I need it to do is to give me some data and to automatically turn off when my energy prices go below zero and back on when above zero. (I know I need to add this function in the registers.py).

@NosIreland

Change paho-mqtt to 1.6.1 or if using requirements use this:

This did fix the "DeprecationWarning". The error is still there but I think this is because of the incompatible S3-WIFI-ST?

Unfortunately I have no way in testing this since I do not own S3-WIFI-ST. If I remember correctly the protocol is a bit different. I have it working with new Deye inverter but it is using different wifi stick. If S3-WIFI-ST works with new pysolarmanv5 then solismod can be modified also to work but again I have no way in doing this without having S3-WIFI-ST