search

cdpuk / givenergy-local

Home Assistant integration for local access to GivEnergy inverter and battery systems
MIT License
50 stars 14 forks source link

Suggestion: Add additional model to "Compatible with" #5

Closed digdilem closed 2 years ago

digdilem commented 2 years ago

Firstly, thanks for writing and making this available. I've been using it for three weeks on my new install and it's been 100% perfect.

I can confirm that it works with at least one additional model of inverter and battery.

Inverter: HY3.6 Battery: Giv-Bat5.2

The docs only mention one model, so my suggestion is that the above is added to the "Known to be working with" list.

Thanks again

image

image

image

cdpuk commented 2 years ago

Hi, thanks for the feedback 😄

I actually very recently removed the list of supported models as it felt like too much of a moving target to keep up to date. At the moment it seems the product range is fairly consistent so most (all?) inverters will work.

One area I'm keen for input on is the behaviour of different battery packs on newer inverters. The challenge here is they have different capabilities when it comes to things like charge/discharge power. The recently added sensors and services to set these are based on my Gen1 inverter with 8.2kWh battery, so I'm not sure how they will behave with others. Something to keep an eye on - I'm really just waiting for someone to raise an issue related to this so we can work though the logic.

image