rany2 / open-meteo-solar-forecast

Python module that uses Open-Meteo to get solar production forecast
MIT License
11 stars 2 forks source link

Implement support for multiple PV arrays #8

Closed rany2 closed 3 weeks ago

rany2 commented 1 month ago

Fixes #5

rany2 commented 1 month ago

@MelleD Could you confirm that it works fine for you? An example of how it should be used is as follows:

    async with OpenMeteoSolarForecast(
        latitude=[52.52, 52.53],
        longitude=[13.38, 13.39],
        declination=[23.44, 23.44],
        azimuth=[10, 20],
        dc_kwp=[2.3, 4.6],
        efficiency_factor=[0.75, 0.75],

You could alter the example/estimate.py; if it looks good to you I'll merge it and implement the remaining bits in HA.

MelleD commented 1 month ago

@MelleD Could you confirm that it works fine for you? An example of how it should be used is as follows:

    async with OpenMeteoSolarForecast(
        latitude=[52.52, 52.53],
        longitude=[13.38, 13.39],
        declination=[23.44, 23.44],
        azimuth=[10, 20],
        dc_kwp=[2.3, 4.6],
        efficiency_factor=[0.75, 0.75],

You could alter the example/estimate.py; if it looks good to you I'll merge it and implement the remaining bits in HA.

How I said in forum, with the new template sensor looks good so far to have all information now :). So If you would like we can close the request

gieljnssns commented 3 weeks ago

I think this FR is still valid. If you have multiple arrays on one inverter and your inverter has a maximum ac power, this should still be calculated on this inverter.

rany2 commented 3 weeks ago

In that case, I'll rework it a bit and merge.