Storion-S5 System fix and Improvements

[Poshy163]

TODO:

• Add additional data points provided by the API.
• Check if any data points from the API are blank or a whitespace, if so. default to 0 (should fix #111 and #112)
• If multiple inverters exists, throttle the subsequent inverters calls to reduce the chance of a 6053 max requests error ....

[Poshy163]

Throttling was done in response to this. which only allows a certain amount of calls every 10s.

needs this pull in the open-API to allow the throttling to work

*currently, it spits out an error once until the amount of inverters can be read, then subsequent calls are fine. and assuming the user has a single inverter, no throttle is applied

[CharlesGillanders]

I merged your pull request in the Open API library - if you bump the library version to 0.0.10 now in the home assistant integration that should have the optional delay function working.

[Poshy163]

Cheers, ill bump the version in this PR

[Poshy163]

With the fix for the Storion-S5 systems, essentially what the best way of approaching it is having 2 lists of sensors, a "limited" and "full" list of sensors, where the Storion-S5 uses only the values it can, then within the coordinator, if we just check each value to check if its null, and if so, convert it to 0 (to stop it from chocking up). converting to 0 shouldn't make a difference as we are not displaying any of those values, or the values it is used to calculate with

(doing it this way allows us to patch up pretty quickly if alpha pulls this again in the future with a different system model)

what was removed from Storion-S5 sensor list:

• Instantaneous Generation
• Instantaneous Grid I/O Total
• Instantaneous Grid I/O L1
• Instantaneous Grid I/O L2
• Instantaneous Grid I/O L3
• Instantaneous PPV1
• Instantaneous PPV2
• Instantaneous PPV3
• Instantaneous PPV4
• Instantaneous Battery I/O
• Instantaneous Load
• State of Charge
• Instantaneous Battery SOC
• Solar to Load
• Solar to Grid

[Poshy163]

Maybe run this as a beta @CharlesGillanders to make sure i weed out any of the possible outliers 🥲

[imascarenhas]

Nice work on the changes, just installed the beta and noticed a minor issue. The new Total Income sensor seems to be hardcoded to use "$" for the currancy rather than take it from the value set in Home Assistant.

[homeconnexde]

Now I am not able to install de integration again. I got: {"message":"Invalid handler specified"}

[Poshy163]

@homeconnexde can you make a separate github issue for it

[homeconnexde]

Ok I managed to do the installation without the invalid handler failure. I entered Api and App Key. Ist says Configuration success after entering AppId and App Secret. And then it sticks with initializing... and the initializing failure, trying again...

[homeconnexde]

While in HACS it says 0.50beta in upper left corner, the Home Assitant Settings integration still says 0.49. But I deleted the alphaess folder in custom components before reinstalling via HACS

[Poshy163]

Did you reboot after installing the beta?

Try deleting the AlphaESS folder in custom components, and then reinstall the beta, restart your home assistant instance. Then try to add your appID/Secret

[homeconnexde]

Yes, that is the way how I could fix the handle exception issue.

I deleted the alpha folder in custom components.

Full reboot of HA

Then re-download via HACS.

Full reboot of HA

Add alphaess integration via HACS add integration link

[AuTHeNTiiiX]

Integration still not intialition with Version 5.2

I getting this error

Unexpected error fetching alphaess data Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 354, in _async_refresh self.data = await self._async_update_data() ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/alphaess/coordinator.py", line 90, in _async_update_data inverterdata["Instantaneous PPV1"] = await process_value(_pvpowerdetails.get("ppv1")) ^^^^^^^^^^^^^^^^^^^ AttributeError: 'NoneType' object has no attribute 'get'

[Poshy163]

@AuTHeNTiiiX cheers for that, can you make an issue. and include what the logs say, there should be a value that has includes your serial number, and model _LOGGER.info(f"Serial: {serial}, Model: {model}") (where {serial} is the serial number of the inverter, you can blank this out, and {model} is the model, which i need)

you should beable to find it quickly by opening the logs, Settings > System > Logs > Load all logs. then doing control + F and searching for model . should be a bit like this