cdpuk / givenergy-local

Home Assistant integration for local access to GivEnergy inverter and battery systems
MIT License
50 stars 14 forks source link

Are you planning on adding write value abilities? #3

Closed SciFi-Bob closed 2 years ago

SciFi-Bob commented 2 years ago

I would like to set some of the inverter settings via HA automations, specifically I am interested in:-

Eco mode enable Timed Charge enable Battery charge and discharge power setting

I just wanted to know if you are planning on adding the feature before I start trying to work it out myself :-)

Thanks for this great add-on!

cdpuk commented 2 years ago

Thanks for your interest 😄 I'm currently experimenting with the best way to write values. The main issue I currently have is those fairly regular errors that the underlying library reports when reading responses from the inverter. At least I assume it isn't just me that gets these.

I can get single values changes working OK - even if the response can't be read, the change still applies (e.g. set the max charging rate). However for mode changes that write to multiple registers, a failed response from the first write request causes the rest of the operation to be stopped, landing you in an inconsistent state.

Anyway, yes, work in progress but I'm keen to release something realiable rather than flaky code!