search

robbinjanssen / home-assistant-omnik-inverter

Read the current, daily and total Wh from your Omnik Inverter via local network (no cloud!)
MIT License
55 stars 21 forks source link

buffer size too small (85 instead of at least 125 bytes) #273

Closed Pakl8 closed 8 months ago

Pakl8 commented 8 months ago

buffer size too small (85 instead of at least 125 bytes) using TCP

Detailed description

During the setup of the Omnik inverter, I used TCP as data source, filled in the host IP address and serial number, an error appears, see snaphot, any suggestions? TCP_error

MarijnS95 commented 8 months ago

I'm very sure we fixed this quite some time ago with https://github.com/klaasnicolaas/python-omnikinverter/pull/376

klaasnicolaas commented 8 months ago

But we need to bump the package, to fix it in the integration.

Pakl8 commented 8 months ago

Thanks for your reaction, https://github.com/klaasnicolaas/python-omnikinverter is another package, I tried this one but had some issues during installation, but I will give a try again. This one was easier but run into a 3000TL inverter issue, as explained.

klaasnicolaas commented 8 months ago

You are not supposed to install the package, because it is not an integration. The integration uses this pypi package to retrieve the data, if we have performed a version bump then this issue will be resolved.

robbinjanssen commented 8 months ago

Created a PR to bump the version: https://github.com/robbinjanssen/home-assistant-omnik-inverter/pull/276

robbinjanssen commented 8 months ago

@Pakl8 please try the latest version :)

Pakl8 commented 8 months ago

For sure I must do something wrong, as I will get the same error, but I also do not see any changes in https://github.com/robbinjanssen/home-assistant-omnik-inverter/tree/main/custom_components/omnik_inverter exept for manifest.json, I used HACS by searching for Omnik Inverter Solar Sensor (No Cloud) to get things installed.

robbinjanssen commented 8 months ago

@MarijnS95 @klaasnicolaas ~are we sure this has been released?~ We need to tag the python-omnikinverter package, and then bump again ;-)

https://github.com/klaasnicolaas/python-omnikinverter/releases

MarijnS95 commented 8 months ago

@robbinjanssen from the linked PR/commit there appears to be no tag pointing to it, so it wasn't released yet. Now 1.0.0 was just published, and 1.0.0 shows up in the merge commit:

https://github.com/klaasnicolaas/python-omnikinverter/commit/b0498f228971c310d4a0cd5d626aa5a593280ea1

robbinjanssen commented 8 months ago

Released v2.6.4

Pakl8 commented 8 months ago

Thanks for the update, works as it should be!