Add WallBox Support and other various bugfixes#168
Conversation
CharlesGillanders
left a comment
There was a problem hiding this comment.
Re: removing serial number prefix: when I initially went with this convention it was intended to allow those with multiple inverters to identify which sensor went with which inverter. If the prefix is removed does that result in either potentially duplicate sensors or potential confusion about how to tell those sensors apart?
|
Sounds great! Thank you. |
|
I was doing some testing with the alpha wall box (ev charger), and it seems like alpha has either dropped, or massively reduced the limit to the POST requests? was able to make multiple within a few minutes? might query the support team on it (if it goes anywhere and attempt find out what happened) and if it was an intentional change |
|
Also seems like the arbitrary limit on the amount of api calls that can be made per minute has been removed? will follow up on this before releasing, but its very strange |
|
Even if they have relaxed the limitations on the frequency of API /POST calls, I think that we still need to set a sensible limit in the integration. We don't want people to have the option of abusing this making too many calls. |
|
i have removed the throttle if you have multiple inverters, with my testing (with 2), you don't seem to run into any issues with being rate-limited. with the natural delay between API calls, it should be more than enough to conform to what the "new" limits apparently are |
|
Its basically done, just pending hearing back about vitapower support for charge and discharge, @mauricebish and i are in contact with EvaPower/AlphaESS to hopefully speed this up |
|
Haven't heard anything back, ill just merge this PR and will deal with the vitapower stuff whenever we hear back |
CharlesGillanders
left a comment
There was a problem hiding this comment.
Let's get this in now but I'm thinking that using hard coded English strings for the charger states is probably against the design guidelines for home assistant. I'm guessing we should be using the built in language capabilities even if the only language we provide is English.
Have you any experience with that?
|
Yeah, can do. Will fix tonight |
|
thank you for this integration. |
|
Heya @djangopap, currently AlphaESS does not expose this info to the openAPI, Your best bet might be asking them if they can allow it? Otherwise there isn't much I can do here unfortunately |





This is still a work in progress, all feedback is welcome
prerequisites
New Features:
more to come
Bug Fixes
Resolves #159