Add battery SOC and kWh usage stats

hn / ginlong-solis

Solis inverter ESP8266 data logger, S3 WiFi stick reverse engineering and ESPhome firmware

90 stars 16 forks source link

Add battery SOC and kWh usage stats #15

Closed tomjones1977 closed 1 year ago

tomjones1977 commented 1 year ago

Adds Battery SOC (33139), Discharge kWh (33165) and Charge kWh (33161) value reads.

Linked to https://github.com/hn/ginlong-solis/issues/14

Not tested on live inverter (inverter is being installed in August).

hn commented 1 year ago

Thanks for contributing!

Please change:

"Battery SOC" -> "Battery state of health (SoC)"
Battery sensor does not have "skip_updates" set, why?
Unnecessary extra blank line at the end

I do not want to merge configs which do not have been tested on a live system. So please report back in August or maybe some other user with an hybrid inverter (@hcouplet ?) is interesting in adding/testing/commenting these things as well.

tomjones1977 commented 1 year ago

Confirmed working on Solis inverter and feeding into HA. Minor updates to code based on the recommendations above.

hn commented 1 year ago

@tomjones1977 , thanks for contributing!