StephanJoubert / home_assistant_solarman

Home Assistant component for Solarman collectors used with a variety of inverters.
Apache License 2.0
508 stars 191 forks source link

[New] Integration overhaul #616

Open davidrapan opened 2 days ago

davidrapan commented 2 days ago

Hello, I had a lot of issues with your current implementation (mainly with stability of the connection, etc.). So I was tweaking, tweaking and end up with hefty overhaul which works flawlessly (with my Deye SG04LP3, 272**)...

So I decided to share it, ofc... ha-solarman

There is really A LOT of changes so... I will be happy to answer any questions.

Have fun.

PV-Joe commented 2 days ago

Hi David,

just moved from this integration to yours and can‘t believe how much better it works.

Especially the time between updates is much faster with approx. 5 seconds.

There are many new sensors available as well! I recommend to switch to your intigration.

Many thanks and keep up the good work!

davidrapan commented 2 days ago

That's wonderful to hear!

If you don't mind I have few questions for you:

Anyway update interval is 5 - 25 seconds while it's 5 s most of the time and 25 s when Inverter sending data to the Solarman servers.

PS: I just found small bug which I probably introduced with some of the latest changes and it affects the connection, meaning it disconnects once an hour for around 10 seconds. I'll look into it as soon as I can. ;-) And I also have many more planned so stay tunned. ;-)

Have fun!

PV-Joe commented 2 days ago

Go here for my configuration and more discussion:

https://github.com/davidrapan/ha-solarman/issues/1#issue-2385451872