incub77 / solis2mqtt

Modbus RTU / RS485 to MQTT bridge for Solis Mini solar inverter.
GNU General Public License v3.0
53 stars 12 forks source link

pano-mqtt client issue #14

Open bsamarketing opened 3 months ago

bsamarketing commented 3 months ago

not sure of this project is still being developed/monitored, but if it is, there seems to be an issue with the mqtt client used.

The project uses the pago-mqtt library, and when this moved to V2 it seems tho have broken solis2mqtt, causing it to throw an mqtt client socket error when run (sorry, don’t have the exact error message)

I have fixed the issue by forcing v1.1 in the requirements.txt file using the following:

replace the line paho-mqtt with paho-mqtt==1.1

i them removed a line in the mqtt.py file: self.enable_logger() rerun the setup and the the integration is now delivering data to home assistant.

however it would b great if it could be fixed to run with paho-mqtt v2 but is a bit above my pay grade I am afraid.