rany2 / ha-open-meteo-solar-forecast

Home Assistant Open-Meteo Solar Forecast Integration
Apache License 2.0
41 stars 3 forks source link

[Feature request] Support multiple arrays #17

Open FredericMa opened 5 months ago

FredericMa commented 5 months ago

Hi,

First of all; thanks a lot for your awesome work!

I was wondering if it would be possible to support multiple arrays/inverters? In my case we have one inverter with two strings. One string of 12 panels with a 45° inclination and a second string of 6 panels with a 15° inclination. Both do have the the same azimuth but I can imagine that there are people that have even more strings or multiple inverters with different orientations and inclinations.

Thanks!

Greetings, Frederic

FredericMa commented 5 months ago

Now I think about it, am I supposed to create two config entries in my case? I was comparing with the Solcast integration which exposes one set of sensors in one config entry, even if you have multiple rooftops configured in the Solcast portal. Looking at Forecast solar, it also requires you to create multiple config entries for multiple arrays.

On the other hand it can also give incorrect results since it will also assume you have multiple inverters, which is not the case in my setup. I have a 5 kW inverter and a total of 7.02 kWp so clipping occurs in my setup.

rany2 commented 5 months ago

For now, you will need to create two config entries and a template sensor (working examples are in the forum thread) to combine the data but in the future I will add support for it via configuration.yaml.

ChirpyTurnip commented 5 months ago

I second this - I have two inverters with two strings each, pointing in different directions. North + East to one inverter, North + West to a second inverter. Like FredericMa my inverters are 5kW and will clip the solar panel output....

I really need to be able to combine the two strings, then set a maximum to clip the forecast. The forecast though is in the attribute data, rather than the state so it's not easy. What would be ideal is to be able to define the integration to match the real world - that is, specify the inverter limit and efficiency, and then add string definitions (as per the current setup). There's a lot of twin string inverters, but very very few that have more than four strings....and from an integration perspective it probably doesn't make big difference whether a user specifies one or four strings - just forecast for each supplied string and then spit out the sum of each period (or the inverter max, whichever is less).

iainJersey commented 5 months ago

Mind you - I have a 6kw SolarEdge inverter with one string (in the usual sense of the word) That single string however has 4 distinct ‘segments’ - different roofs different azimuths, different angles, so I currently have them set up as 4 ‘strings’, and use a group sum helper to add them up. Sub optimal, but useable Can’t wait for one inverter with separate strings (and I have another inverter with two more ‘real strings’