CharlesGillanders / homeassistant-alphaESS

Monitor your energy generation, storage, and usage data using the official API from Alpha ESS.
MIT License
101 stars 22 forks source link

Support State of Charge for Storion-S5 #126

Closed Poshy163 closed 3 months ago

Poshy163 commented 3 months ago

Breaking

Removes state of charge from non Storion-S5 inverters, instead opting to use only Instantaneous Battery SOC

Changed

Closes https://github.com/CharlesGillanders/homeassistant-alphaESS/issues/124

homeconnexde commented 3 months ago

Thank you. Then I'm waiting for the API change.

CharlesGillanders commented 3 months ago

OpenAPI 0.0.11 is published to PyPi.

Poshy163 commented 3 months ago

Cool, @homeconnexde give it a crack and see how it goes

homeconnexde commented 3 months ago

Hmm, i now got 17 entities but no SOC

image
homeconnexde commented 3 months ago

The upper are called sensors, the lower diagnosis

homeconnexde commented 3 months ago

Here is a screenshot with the full entity names

image
Poshy163 commented 3 months ago

Interesting, no State of Charge sensor, did you copy the sensorlist.py file? it should be going into the LIMITED_SENSOR_DESCRIPTIONS list area as you can see in the pull details

homeconnexde commented 3 months ago

I did a reinstallation and now there is a SOC but it is 0% while it is 100% actually.

image
Poshy163 commented 3 months ago

Hey @homeconnexde i think i made a mistake from when i was testing locally, can you change the stuff on line 92 from if _onedatepower and _soc is not 0: to if _onedatepower and _soc is 0:

should look like this

image

homeconnexde commented 3 months ago

Worked:

image
homeconnexde commented 3 months ago

Great job :-) Thank you very much.

Poshy163 commented 3 months ago

Heh dope, ill do some clean and merge it. thanks so much for your help!

Poshy163 commented 3 months ago

Heya @homeconnexde you mind testing just these last few changes? have cleaned some things up in regards to non-storion systems and wanna make sure it still works for you

homeconnexde commented 3 months ago

It is still working. now it says Version 0.5.4

Poshy163 commented 3 months ago

Nice, cheers for that!

Poshy163 commented 3 months ago

@CharlesGillanders this pull bumps the API version to 0.0.11 and also bumps the version in the manifest file to 0.5.4