Skip to content
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

[sungrow] added Backup Current/Power registers #1

Open
wants to merge 14 commits into
base: sungrow
Choose a base branch
from

Conversation

stl01
Copy link

@stl01 stl01 commented Oct 24, 2023

No description provided.

soenkekueper and others added 14 commits July 16, 2023 11:11
Signed-off-by: Sönke Küper <[email protected]>
…king. Added categories and tags for channels.

Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
… (instead of floats issues).

Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
Signed-off-by: Sönke Küper <[email protected]>
@soenkekueper
Copy link
Owner

Hey,
thanks for your contribution!

I've checked the sungrow-spec, which this addon was based on, and could not find any information about the backup registers, do you have a newer version?
When trying to read the registers manually i get a read error - this would cause openhab to read no data :/
So my question is, which kind of sungrow inverter are you using, that this is working on?
I'll build and try what happens, but think it would break compatibility to some other inverters.

Thanks for the correction of energy register - i think this is actually wrong within the documentation.
As i'm using the DAILY_EXPORT_ENERGY channel i've not remarked this issue - is this channel working for you too?

BR

Sönke

@stl01
Copy link
Author

stl01 commented Oct 28, 2023

Hey, sorry for this rather uncommented and blank PR. I'm not too familiar with the git workflow.

I'm using a Sungrow SH10RT-V112 and a SBR128 Battery with a permanent load on the backup line. In this setup I get the correct values in Watt over the specified input registers. I don't know if these are implemented in other inverters as well.

The backup registers and 4 other register (Meter Active Power Total and for each Phase) are not documented in the sungrow-spec as far as I know. I found this information here:
https://www.photovoltaikforum.com/thread/166134-daten-lesen-vom-sungrow-wechselrichtern-modbus/?postID=2645527#post2645527

another great source for the sungrow input and holding registers is this Home Assistant integration
https://github.com/mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant/blob/bf94a1f215c3b67de28347c8cc5d61fe327242f4/modbus_sungrow.yaml#L311

The DAILY_EXPORT_ENERGY_FROM_PV is indeed just wrong in the sungrow documentation. The last days were not sunny enough for me to export any meaningful amount of energy so that I could not check or understand the logic and the difference to the DAILY_EXPORT_ENERGY channel. But I've seen a 0,1 kWh twice on both channels so that I know that it is at least technically working.

Since this PR I was working on the four Meter Active Power registers and a decomposition/decoding of the Running State (13001) and System State (13000) Registers. That's why they were also added as comments in the code.

Do you know if it's possible to read/write holding registers with this addon?

BR
stl01

@soenkekueper soenkekueper force-pushed the sungrow branch 3 times, most recently from efb341b to cbee75a Compare December 18, 2023 20:15
soenkekueper pushed a commit that referenced this pull request Dec 30, 2024
* Fixes in indexes from Ruepert #1

Signed-off-by: Konstantin Polihronov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants