search

mkaiser / Sungrow-SHx-Inverter-Modbus-Home-Assistant

Sungrow SH Integration for Home Assistant for SH3K6, SH4K6, SH5K-20, SH5K-V13, SH3K6-30, SH4K6-30, SH5K-30, SH3.RS, SH3.6RS, SH4.0RS, SH5.0RS, SH6.0RS, SH5.0RT, SH6.0RT, SH8.0RT, SH10RT, SH5.0RT-20, SH6.0RT-20, SH8.0RT-20, SH10RT-20, SH5.0RT-V112, SH6.0RT-V112, SH8.0RT-V112, SH10RT-V112, SH5.0RT-V122, SH6.0RT-V122, SH8.0RT-V122, SH10RT-V122, SH4.6R
305 stars 79 forks source link

Battery Sensor (W) #159

Closed mjunker61 closed 9 months ago

mjunker61 commented 10 months ago

In the file "modbus_sungrow.yaml" should the power with kW configured, like below. Otherwise the Sensor cannot show the correct Value in the GUI.

I did it, and now I have the correct Value in the Dashboard.

mkaiser commented 10 months ago

What inverter are you using?

My SH10RT shows the corrrect values (in W, not kW) image

SieresGH commented 10 months ago

Hello, I wanted to get involved. This is displayed correctly for me. The problem, however, is that neither "Battery charging power" or "Battery discharging power" is displayed correctly. For me the number remains at zero, see screenshot. Battery power raw is displayed normally, as well as many others are also displayed correctly. I have a Sungrow SH10RT inverter and BYD storage. I use HomeAssistant version 2023.8.4. Screenshot 2023-09-09 080844

mjunker61 commented 10 months ago

We have a Sungrow Hybrid Inverter SH10RT. Maybe the reason was, that it was one of the first SH10RT, sold in GE. The first steps were very stressfull with a lot of communication, Resets and Firmware Updates. Now it works good.

Here my screenshots for you:

2023-09-10-ha-sh10rt-in-dashboard

2023-09-10-ha-sh10rt-part-of-modbus_sungrow yaml

I didn't check all sensors, but I think it is the same at meter_active_power and so on here.

mkaiser commented 10 months ago

So this issue can be closed? :)

mjunker61 commented 9 months ago

I think it cannot closed, because you declare the "unit of measurement" in "W" and it have to be in "kW".

Louisbertelsmann commented 9 months ago

No, it doesn't. Some people prefer it to be in W, some in kW. You can set it to anything by just changing the unit of measurement and the scale. For me, this config works and shows the correct values.

mjunker61 commented 9 months ago

@Louisbertelsmann and @mkaiser Pls, I do not want to waste your all time. I only want to inform and say, in my homeassistant I saw a problem and I have to declare it manualy = "unit of measurement" in "kW", otherwise I get the wrong values.

It has nothing to do with formatting it in any guis or so. Certainly I checked this, befor I wrote this Issue.

But when I am the only one, who have this ==> it is no problem. I change it manualy an all is ok. I only want to help !!!

Louisbertelsmann commented 9 months ago

What you could try is doing a Firmware Update. If you already created an installer Account, so it that way. If not, so it via the App and local access.

dylan09 commented 9 months ago

@mjunker61 I think there is an error in your configuration or something went wrong inside your inverter.

Sungrow protocol definition states 1W as unit for register 13022. Maybe you could check the definition for sensor.battery_power_raw in your configuration. Is it the same as in github?

mjunker61 commented 9 months ago

@Louisbertelsmann I updated the firmware until 07/2023 but not with update with SAPPHIRE-H_B001.V000.P004-20230719.zip. I will try it.

@dylan09 I only took the modbus_sungrow.yaml from ttps://github.com/mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant and customized it with IP, Port and "unit of measurement" in "kW" for Battery charging power and Battery discharging power. In this way, so it works. But I will look further.

Both, thx for information and support.