search

faanskit / ha-esolar

Custom Integration of SAJ eSOLAR
https://github.com/faanskit/ha-checkwatt
GNU General Public License v3.0
6 stars 4 forks source link

Backlog items #1

Closed faanskit closed 1 year ago

faanskit commented 1 year ago

(1) Inverter level sensor for battery percentage - DONE Currently battery % is for the complete plant and not possible to track per battery in a multi inverter/battery system. Assumption is that this will become a new sensor, but an attribute to the battery power sensor should be considered as well

(2) kitList/mastermcufw missing as attribute - DONE Should be an attribute on the inverter Energy sensor. Correction; slavemcufw is not used. Instead, use mastermcufw and displayfw.

(3) Inverter Type unknown - DONE For kitList.type, anything but 0 (on-grid) is reported as unknown. Battery systems reports type = 2

Plant list

                  <span class="title_name">Plant Type</span><!-- 电站类型 -->
                  <div class="select_set">
                    <select name="type">
                        <option value="" selected="selected">All</option>
                        <option value="0">Grid</option>
                        <option value="1">Storage</option>
                                       <option value="3">AC Coupling</option>

Inverter list

                          <select name="deviceType" id="deviceType">
                              <option value="">All</option>
                              <option value="0">On-grid</option>
                              <option value="1">Storage</option>
                              <option value="2">AC Coupling</option>

(4) Device status can be: - DONE Review what values that device status reports against:

                                <select name="deviceStatus" id="deviceStatus" onchange="findDevicePageList();">
                                    <option value="" selected="selected">Status</option>
                                    <option value="1">Normal</option>
                                    <option value="2">Alarm</option>
                                    <option value="3">Off-line</option>
                                    <option value="4">Stock</option>
                                    <option value="5">History</option>

(5) Add protective code if protocol from SAJ changes - DONE Possible future refactoring to not be transparent with the SAJ protocol to the sensors

(6) Add logging output - DONE Minimalistic logging currently. Add both exceptions, errors and debug logging.

(7) Update README.md - DONE Document how the system works

(8) Make available in HACS - DONE Learn how to deploy for HACS

RobbieDemaegdt commented 1 year ago

For #1 I think it would be nice to see the separate battery percentages in a sensor to see easily the history and how they behave. The name for the entity could be linked to the inverter SN value as the battery of a H1 system is actually in the same enclosure as the inverter so it would make sense to make the difference with the SN value.