[Sungrow SH15T & Sungrow SBR128] Missing Data for "Battery Charging Power" and "Battery Discharging Power"

[bgilles]

Hello everyone,
Yesterday an SH15T inverter and an SBR128 battery were installed in my home. After reading a bit here, I immediately tested the integration in Home Assistant (LAN via WiNet-S). The dashboards work pretty well, all the important parameters are displayed accordingly.
Only for "Battery Charging Power" and "Battery Discharging Power" I consistently have a value of 0 W. Even if the battery is charging or discharging.
Does anyone know the problem or have any ideas on how I can troubleshoot this?
Thank you and best regards
Bastian

[Gnarfoz]

Does the WiNet-S interface or iSolarCloud show an appropriate value for this?
This integration uses register 13022 but the documentation also mentions registers 5214-5215 for "battery power - wide range". Maybe you can try that instead.
Beware, it has different properties (signed 32 bit, not unsigned 16 bit). Refer to the Modbus documentation: https://github.com/Gnarfoz/Sungrow-Inverter/blob/main/Modbus%20Information/TI_20231019_Communication.Protocol.of.Residential.Hybrid.Inverter_V1.1.2_EN.pdf

[ptar]

Did you check the states of the other sensors used in the calculations for availability and state?

[Remigius0815]

Did you check the states of the other sensors used in the calculations for availability and state?

The only one sticking out is sensor.running_state - it has been 0 ever since I don't get readings anymore for charge/discharge. Which sensor is this and what could it be related to?

Thanks
michael

[dterrison]

Unfortunately does not work for me with SH20T, getting "Not available" for charge & discharge.

Just had this issue myself. Managed to figure it out, it was a result of editing the YAML incorrectly. Make sure that when you add the new lines you keep the old section under "state", as the existing section contains the workaround for the running_state sensor. The first line in the existing state section must be replaced with the final line of the new YAML. Can't just copy paste the code block as is.
So the final section should look like:

      - name: Battery charging
        unique_id: sg_battery_charging
        # TODO: test more with these parameters for template binary sensors
        # device_class: none / power
        # state_class: measurement
        availability: >-
          {{ 
          not is_state('sensor.battery_current', 'unavailable')
          or
          not is_state('sensor.running_state', 'unavailable')
          or
            (   not is_state('sensor.ems_mode_selection_raw', 'unavailable')
            and not is_state('sensor.battery_forced_charge_discharge_cmd_raw', 'unavailable') 
            and not is_state('sensor.battery_power_raw', 'unavailable') 
            and not is_state('sensor.total_dc_power', 'unavailable')
            and not is_state('sensor.load_power', 'unavailable') 
            )
          }}
        state: >-
          {% if (states('sensor.battery_current')| float) != 0 
                and (states('sensor.battery_power_raw')| int) != 0  %}
            {% if (states('sensor.battery_current') | float) < 0 %}
              on
            {% else %}
              off
            {% endif %}              
          {% elif states('sensor.running_state')|is_number %}
            {# use available sensor running_state #}
            {% if states('sensor.running_state')|int|bitwise_and(0x2) > 0 %}
              on
            {% else %}
              off
            {% endif %}
          {% else %}
            {# workaround for SH*RS inverters without working running_state #}
            {% if (states('sensor.ems_mode_selection') ) == "Forced mode" %}
              {# EMS forced mode #}
              {% if (states('sensor.battery_forced_charge_discharge_cmd') == "Forced charge") %}
                {# in mode Forced charge #}
                {% if (states('sensor.battery_power_raw')|int > 0 ) %}
                  {# power flow from/to battery #}
                  on
                {% else %} 
                  {# no power flow from/to battery #}
                  off
                {% endif %}
              {% else %}
                {# in EMS mode, but not in mode Forced charge #}
                off
              {% endif %}
            {% else %} 
              {# not in EMS forced mode, assuming self consumption mode #}
              {% if states('sensor.total_dc_power')|int > states('sensor.load_power')|int %}
                {# more power generated than consumed. assuming battery charging #}
                on
              {% else %} 
                off
              {% endif %}
            {% endif %}
          {% endif %}

Hope this helps

[Remigius0815]

I checked it line by line, unfortunately this is not the reason.

[bgilles]

I guess we should wait for some more confirmations from other folks and then craft a PR to adjust the type based on models as you suggested 👍

I have tried to implement your suggestions and code snippets for my SH15T, but I have to admit that I am reaching the limits of my current YAML skills here. None of these work out, but I'm pretty sure that this is due to incorrect syntax (tried to copy & paste in different ways). So, unfortunately for the moment no confirmation for my SH15T. Seems that I'll have to wait for an upcoming release. Nevertheless, thank you very much for your efforts!

[wapman1979]

This works on my Sungrow SH15T with a SBR 128 battery.
Connectivity via LAN at WiNet-S2.
No Modbus-Proxy.

Installed Firmwares:

• M_WiNet-S2_V01_V01_A
• Battery: SBRBCU-S_22011.01.22
• Inverter:
  • ARM Software Version | PEARL-H_01011.01.21
  • MDSP Software Version | PEARL-H_03011.01.21
  • AFCI Software Version | AFCI_06002.01.10

No Changes at "data_type" for "Battery current" because this is already adopted in the mkaiser update from 2024-08-19.
This looks like this:

  - name: Battery current
    unique_id: sg_battery_current
    device_address: !secret sungrow_modbus_slave
    address: 13020 # reg 13021
    input_type: input
    data_type: int16
    precision: 1
    unit_of_measurement: A
    state_class: measurement
    device_class: Current
    scale: 0.1
    scan_interval: 10

This is my whole entry for "Battery charging":

  - name: Battery charging
    unique_id: sg_battery_charging
    # TODO: test more with these parameters for template binary sensors
    # device_class: none / power
    # state_class: measurement
    availability: >-
      {{ 
      not is_state('sensor.battery_current', 'unavailable')
      or
      not is_state('sensor.running_state', 'unavailable')
      or
        (   not is_state('sensor.ems_mode_selection_raw', 'unavailable')
        and not is_state('sensor.battery_forced_charge_discharge_cmd_raw', 'unavailable') 
        and not is_state('sensor.battery_power_raw', 'unavailable') 
        and not is_state('sensor.total_dc_power', 'unavailable')
        and not is_state('sensor.load_power', 'unavailable') 
        )
      }}
    state: >-
      {% if (states('sensor.battery_current')| float) != 0 
            and (states('sensor.battery_power_raw')| int) != 0  %}
        {% if (states('sensor.battery_current') | float) < 0 %}
          on
        {% else %}
          off
        {% endif %}              
      {% elif states('sensor.running_state')|is_number %}
        {# use available sensor running_state #}
        {% if states('sensor.running_state')|int|bitwise_and(0x2) > 0 %}
          on
        {% else %}
          off
        {% endif %}
      {% else %}
        {# workaround for SH*RS inverters without working running_state #}
        {% if (states('sensor.ems_mode_selection') ) == "Forced mode" %}
          {# EMS forced mode #}
          {% if (states('sensor.battery_forced_charge_discharge_cmd') == "Forced charge") %}
            {# in mode Forced charge #}
            {% if (states('sensor.battery_power_raw')|int > 0 ) %}
              {# power flow from/to battery #}
              on
            {% else %} 
              {# no power flow from/to battery #}
              off
            {% endif %}
          {% else %}
            {# in EMS mode, but not in mode Forced charge #}
            off
          {% endif %}
        {% else %} 
          {# not in EMS forced mode, assuming self consumption mode #}
          {% if states('sensor.total_dc_power')|int > states('sensor.load_power')|int %}
            {# more power generated than consumed. assuming battery charging #}
            on
          {% else %} 
            off
          {% endif %}
        {% endif %}
      {% endif %}

This is my whole entry for "Battery discharging":

  - name: Battery discharging
    unique_id: sg_battery_discharging
    # TODO: test more with these parameters for template binary sensors
    # device_class: none / power
    # state_class: measurement
    availability: >-
      {{ 
      not is_state('sensor.battery_current', 'unavailable')
      or
      not is_state('sensor.running_state', 'unavailable')
      or
        (   not is_state('sensor.ems_mode_selection_raw', 'unavailable')
        and not is_state('sensor.battery_forced_charge_discharge_cmd_raw', 'unavailable')
        and not is_state('sensor.battery_power_raw', 'unavailable')
        and not is_state('sensor.total_dc_power', 'unavailable') 
        and not is_state('sensor.load_power', 'unavailable') 
        )
      }}
    state: >-
      {% if (states('sensor.battery_current')| float) != 0 
            and (states('sensor.battery_power_raw')| int) != 0  %}
        {% if (states('sensor.battery_current') | float) > 0 %}
          on
        {% else %}
          off
        {% endif %}              
      {% else %}
        {# workaround for SH*RS inverters without working running_state #}
        {% if (states('sensor.ems_mode_selection') ) == "Forced mode" %}
          {# EMS forced mode #}
          {% if (states('sensor.battery_forced_charge_discharge_cmd') == "Forced discharge") %}
            {# in mode Forced discharge #}
            {% if (states('sensor.battery_power_raw')|int > 0 ) %}
              {# power flow from/to battery #}
              on
            {% else %} 
              {# no power flow from/to battery #}
              off
            {% endif %}
          {% else %} 
            {# in EMS mode, but not in mode Forced charge #}
            off
          {% endif %}
        {% else %} 
          {# not in EMS forced mode, assuming self consumption mode #}
          {% if ( ( states('sensor.total_dc_power')|int < states('sensor.load_power')|int ) ) and states('sensor.battery_power_raw')|int > 0 %}
            {# more power consumed than generated and some battery power --> assuming battery discharging #}
            on
          {% else %} 
            off
          {% endif %}
        {% endif %}
      {% endif %}

[Remigius0815]

I have the same firmware versions & code and it still does not work (for the 2 values only)...