-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Undocumented Holding Registers destroys SH10RT Config #283
Comments
Hi, sorry to hear that! Could you describe in more detail what happened? Strange thing is that I recently installed one SH10.RT rev112 myself for a friend. All the set* functions worked fine via GUI sliders. I will make some updates soon and ensure to write a comment on each undocumented sensor. This leaves following registers as possible culprits: Could you test max charging and discharging registers? Was anything regarding the other registers in the HA log files? |
Can I somehow back up all settings using modbus? I only have a user iSolarCloud account so I cannot back them up there and don't want to have my electrician to set it up again. ;-) Don't know which parameter caused this. And don't know how to figure out. If I change a parameter through modbus does it send the whole configuration again or the parameter to change only? I don't have much knowledge about the modbus protocol. Do you have an idea why the holding register values differs to the other one? |
unfortunately, its not easily possible via modbus (possible, but involves some work). it is easier via isolarcloud: you can backup the current settings by logging in to isolarcloud via installers account. if you don't have one, create one with another email address. Sungrow does not check, if it is an anctual company behind installers accounts or not. I assume the set parameters from the installers are
|
Okay, I've got the time to set up a anstaller account and to create a template of my parameters. But in HomeAssistant it's like it's not reading the parameters as it should. Logs look like that: ` During handling of the above exception, another exception occurred: Traceback (most recent call last): The above exception was the direct cause of the following exception: Traceback (most recent call last): And now to show you what the difference is: The Min SoC looks strange too with two different values: Am I missing something? |
Cannot find (states('sensor.total_imported_energy_cost')) in this repositories YAML file. I guess the error comes from somewhere else. Can you double-check? |
The max battery charge / discharge entities come with a default maximum of 5000 W since @mkaiser can't know your actual battery size / parameters. You can find them here: https://github.com/mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant/blob/main/modbus_sungrow.yaml#L2325 |
regarding installers accounts to enable advanced settings and backup @mkaiser -- I can create an installers account and login on it -- but how do I attach my existing plant to it, in order to do anything? When I "create" a plant and then enter a serial number of the comms dongle it says (rightly) it is already taken. thanks. |
You share the plant from your regular account to your installer account. |
Before you create an issue, make sure to update to the current version of modbus_sungrow.yaml
Describe the bug:
The undocumented holding registers of the Inverters destroys the configuration of my inverter.
If I use one of them, they are overwritting a couple of settings and make the inverter not discharge the battery anymore until I load configuration defaults of the inverter.
The holding register discharge is not the same as the input register one.
It seems like it's destroyed after changing the min soc too.
Model: SH-10.RT v112
LAN (internal port)
[] WiNet-S (LAN) (connected but not used for Modbus)
Are you using a Modbus Proxy (mark one)
Home Assistant version:
modbus_sungrow.yaml:
** Inverter Firmware Status:**
To Reproduce
Steps to reproduce the behavior:
Expected behavior
They should be marked as EXPERIMENTAL... That took me so much of my time to figure out what the problem is and what it did change.
Screenshots
The text was updated successfully, but these errors were encountered: