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

Add support for Sungrow SH5k-20 #4

Open
wants to merge 20 commits into
base: master
Choose a base branch
from
Open

Conversation

tjhowse
Copy link

@tjhowse tjhowse commented Sep 14, 2019

I've made a fair few modifications to how the interface works. I've added the registers that the SH5k-20 holds, as well as added a type key to the register map per register, to allow for translation to signed values before publishing to MQTT.

I slightly reworked how modbus reconnects are handled.

I'm still not entirely happy with this interface. Sometimes the inverter stops responding to modbus requests for a while. I'm not sure if this is a timing issue, or whether the inverter doesn't properly implement modbus, or what. It's a little annoying.

In any case, this is a better starting point for someone wanting to get data from an SH5k-20 using pvstats than not having it at all.

I also stored the registers using integer keys, not strings, since they sort correctly. When adding registers in the 13000 range they sorted to before the registers in the 5000 range, causing difficulties.

@tjhowse
Copy link
Author

tjhowse commented Sep 19, 2019

I have drastically improved the performance of this interface by hard-wiring an ethernet connection to the inverter rather than using the wifi dongle. Modbus communication failures have completely gone away. It's quite likely the modbus re-connection change I made in this branch are unnecessary.

@tjhowse
Copy link
Author

tjhowse commented Oct 24, 2019

@ptarcher Ping! Are you alive?

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.

1 participant