search

WillCodeForCats / solaredge-modbus-multi

Modbus/TCP local polling to one or more SolarEdge inverters. For single inverters, multiple inverters, meters, and batteries.
Apache License 2.0
181 stars 15 forks source link

Add Grid Power Available binary sensor #620

Closed WillCodeForCats closed 2 months ago

WillCodeForCats commented 3 months ago

Add Grid Power Available binary sensor

WillCodeForCats commented 3 months ago

Info for this entity contributed by @dgilmour77

WillCodeForCats commented 2 months ago

Two confirmed OK tests.

deqw commented 2 months ago

@WillCodeForCats Is this enhancement now in the 2.4.16 release? Is 2.4.17-pre.3 now the same as 2.4.16?

WillCodeForCats commented 2 months ago

I haven't made a 2.4.17 release yet becasue I was waiting to see if there was more test feedback.

deqw commented 2 months ago

No problem, still running 2.4.17-pre.3 here. I looked at the Code page and saw the GridStatusOnOff class there. Thought that was the latest release 2.4.16 from a few days ago (but it isn't).

WillCodeForCats commented 2 months ago

I can probably just make the release, I was just hoping for more than 2 confirmations.

deqw commented 2 months ago

Just wait a little bit longer then. I had some additional questions from SE last week in the case of my questions about the M_Events and the 40113 register. Waiting now again for their response.

Since the 40113 returns 0 (zero) when grid is On and 1 (one) when grid is Off, I'm thinking this undocumented register is related more to the Backup Interface status, like Backup On (1) and Off (0) and not meant for Grid On or Off because for that we have the M-Events already. Just a thought. Hope SE can shed some light into this.

Edit: see 2.4.17 now.

WillCodeForCats commented 2 months ago

I just made the release, but if there's more to add I'll just do another one. I'm pretty active about getting things added.