-
-
Notifications
You must be signed in to change notification settings - Fork 66
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
support for hybrid (sh) series of sungrow inverters #1
Comments
At quick glance most of these overlap. I should be able to add these in next week. |
@chainsawbike Hi, I have added initial support for hybrid inverters. c0dafee#diff-fa3524b0f7e519cce2f14a4c5a6b750ca2fd8e66d915d8e30d9384f678afbf39 |
sorry, limited time right now. |
@chainsawbike when you get time, upload to the latest build, |
Hi folks, In addition starting with level 2, I get the Error Message
According to the comment in the config.yaml
I guess here is something wrong in register.yaml matching the wrong registers to SH10RT and SH8.0RT. Because if not, I would say, after resolving these two mentioned one from my side, SH10 Inverters could be announced as "supported and tested" |
I would like to point out, that I added further registers in my nightly. I do not want to pull anyone away from this great project and I am happy to merge my changes back here asap. |
Looks good, if you could post what model you have and PDF you are reading registers off I can upload to my other repo with the other documents here: https://github.com/bohdan-s/Sungrow-Inverter |
thanks for your work :) |
Done. Will be in the next release :) |
@chainsawbike : Do you have a battery? Can you please give me some feedback on the values presented there? I don't get any values from the battery. But it might be a problem in the setup of the inverter. I will contact my eletrician for that next week to make sure it is not. |
I filed a pull request which adds a bunch of SH-Inverter variables. But there is still some todo. One issue is the start-stop stuff. It is not yet correctly interpreted and it is more complex than for the other inverters. Additionally the status names changed. So e.g. "running" does not reflect the status correctly for SH-inverters. |
@benni336 no battery yet it is producing the correct data for the values i would consider important for non battery use so i would say the hybrid series of inverters have working initial support, i am now using this to put data into home-assistant :) |
see: bohdan-s/SungrowModbusWebClient#2 |
v0.3.6 now should have full support :) registers taken from the current documentation |
Here is the official docs for the hybrid range:
communication-protocol-of-residential-hybrid-inverterv1.0.20-1.pdf
I doubt I'll have time to add them myself until late Jan, so I'll leave them here incase someone else wants to add them before then
The text was updated successfully, but these errors were encountered: