Fix sungrow inverter power values (change from gross DC to delivered AC power) #1310
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…o Load_Power (13007) Register 5016 delivers total DC power as gross value before inversion. This includes reactive power and does not substract DC power routed to the storage. Register 13007 shows actual AC power delivered, which is the power that can be consumed.
Load_Power can have negative values (if storage is charged from AC side for ecample) and is therefor specified as S32 by Sungrow (see https://github.com/bohdan-s/Sungrow-Inverter/blob/main/Modbus%20Information/communication-protocol-of-residential-hybrid-inverterv1.0.20-1.pdf page 9 row 52)
|
Testing this PR led to errors in mixed setups, it is necessary to distinguish between SH-LAN, SH-Dongle, SG. So more work is necessary. Will cancel this PR until work is done |
|
Closed, will be replaced by new PR once fixes are ready |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changed register from Total_DC_Power (5016) to Load_Power (13007)
Register 5016 delivers total DC power as gross value before inversion. This includes reactive power and does not substract DC power routed to the storage. Register 13007 shows actual AC power delivered, which is the power that can be consumed.
Using 5016 led to the behavior that
=> this change needs existing Sungrow hybrid inverter setups to be changed in structure so that the storage is nested below the inverter