cdpuk / givenergy-local

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

Additional Register Suggestions - Inv Power & Cutoff % #67

Closed hoggyhoggy closed 8 months ago

hoggyhoggy commented 8 months ago

Is your feature request related to a problem? Please describe. Would it be possible to add some additional supported registers for: Inverter Total Power - Total of all Power Sources being used by the inverter. Battery Cutoff % (as opposed to reserve which doesn't seem to be obeyed - Cutoff % is respected still)

Describe the solution you'd like 2x New entities from Givenergy_Modbus P_INVERTER_OUT - Register 24 (I believe this is the total Inverter Power) BATTERY_DISCHARGE_MIN_POWER_RESERVE - Register 114 (I believe this is the Cutoff % as used in the remote control page)

Describe alternatives you've considered Inverter power could be derived through summing of other existing registers but native would better match the portal Cutoff % doesn't have any other supported method other than an automation that sets discharge to 0 if the SOC reaches X%

Additional context Registers as named in "register.py" of Givenergy_Modbus Library.