fix: correct scale errors in deye_hybrid.yaml (Battery Current + 9 power sensors)#997
Closed
polamoros wants to merge 2 commits intodavidrapan:mainfrom
Closed
fix: correct scale errors in deye_hybrid.yaml (Battery Current + 9 power sensors)#997polamoros wants to merge 2 commits intodavidrapan:mainfrom
polamoros wants to merge 2 commits intodavidrapan:mainfrom
Conversation
Register 0x00BF (Battery Current) uses 0.1 A/bit resolution per the Deye Modbus protocol, not 0.01 A/bit. The incorrect scale caused Battery Current to display 10× too low. Physics proof from observed sensor values on a DEYE 10 kW single-phase hybrid inverter: Battery Power = -723 W (register 0x00BE, no scale, correct) Battery Voltage = 53 V (register 0x00B7, scale 0.01, correct) Expected I = P / V = -723 / 53 = -13.6 A Displayed I (scale 0.01) = -1.35 A ← 10× too low Changing scale from 0.01 to 0.1 gives -13.5 A, consistent with the independently measured Battery Power and Voltage readings. PV current sensors (PV1–PV3, scale 0.1) were verified correct: PV1: 326 V × 1.40 A = 456 W ≈ 445 W (measured) ✓ PV2: 277 V × 1.40 A = 388 W ≈ 391 W (measured) ✓
Nine power sensors used `scale: [1, 10]` which resolves to scale=1 for single-phase inverters (CONF_MOD=0), causing all readings to be 10× too low. Confirmed against DEYE cloud (SolarmanPV) live values on a DEYE 10 kW single-phase hybrid inverter (SG0*LP1): Grid Power (0x00A9): HA 910 W == DEYE cloud 910 W ✓ Load Power (0x00B2): HA 85 W → DEYE cloud 850 W (10× off) Sensors fixed (scale: [1, 10] → scale: 10): - Grid L1 Power (0x00A7) - Grid L2 Power (0x00A8) - Grid Power (0x00A9) - External CT1 Power (0x00AA) - External CT2 Power (0x00AB) - External Power (0x00AC) - Load L1 Power (0x00B0) - Load L2 Power (0x00B1) - Load Power (0x00B2)
polamoros
force-pushed
the
fix/deye-hybrid-battery-current-scale
branch
from
0e02631 to
9072eaf
Compare
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Two categories of scale errors in
deye_hybrid.yamlfor the DEYE single-phase hybrid inverter (SG0*LP1), verified on a DEYE 10 kW unit.Fix 1 — Battery Current (0x00BF):
scale: 0.01→scale: 0.1Register 0x00BF uses 0.1 A/bit resolution, not 0.01 A/bit. The wrong scale caused Battery Current to display 10× too low.
Physics proof:
0x00BE0x00B70x00BFExpected:
P / V = −1016 / 53 = −19.2 A✓ (matches within 1.5%)Before fix:
−1.89 A(10× too low)Fix 2 — Power sensors (0x00A7–0x00B2):
scale: [1, 10]→scale: 10Nine sensors used
scale: [1, 10], which resolves toscale=1for single-phase inverters (CONF_MOD=0), causing all readings to be 10× too low.Confirmed against DEYE cloud (SolarmanPV) live values:
0x00A90x00B20x00AASensors fixed:
0x00A7)0x00A8)0x00A9)0x00AA)0x00AB)0x00AC)0x00B0)0x00B1)0x00B2)🤖 Generated with Claude Code