ProudElm / solaredgeoptimizers

Intergration to get optimizers information from the SolarEdge portal
49 stars 13 forks source link

Minor change - Timeout from 60 to 300 for large panel installs #47

Closed DeFlanko closed 1 year ago

DeFlanko commented 1 year ago

When I installed the integration I would never get values -- after rabbit holing in the code I found that in sensor.py (line 147) changing this

from:

 async with async_timeout.timeout(60):

to:

async with async_timeout.timeout(300):

allowed me to then pull in data:

image

image

I attribute this to the possibility of having "48" panels on the SolarEdge monitoring site. When the reality is that I only have "24", but because Solar City/Telsa never went back into the Solar Edge site to clean up the old panel's the script has to cycle though all "48" panel entities, and It looks like the 60 sec time out occurs before it can complete the full page scrape.

Might help others, figured id open a ticket.