Skip to content

Commit 3ae8291

Browse files
0xAHAclaude
andcommitted
fix: DTC 5201 is confirmed, not assumed (MIN 10000TL-X, holding 43)
Marked ASSUMED on the grounds that the maintainer's MIN 10000TL-X reported no DTC. That was wrong twice over. I read input register 43 (Iac2, phase 2 current) instead of holding 43, where the V1.39 DTC actually lives -- the same input/holding mislabelling in the scanner's Suggested Match column that I had flagged to a reporter an hour earlier. And the scan I checked was from v0.7.7, which does not scan the legacy holding range at all, so I treated a range that was never read as evidence the register was empty. A v1.1.6 scan on the same unit reads 5201 from holding 43. Evidence text records that this unit has no VPP range, so the V2.01 key is downgraded to min_7000_10000_tl_x by convert_to_legacy_profile() -- confirmed as the behaviour of both the live path and the scanner. Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 1bf42ad commit 3ae8291

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

custom_components/growatt_modbus/auto_detection.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -562,7 +562,11 @@ class DtcEntry(NamedTuple):
562562
),
563563
5201: DtcEntry(
564564
'MIN 7-10KTL-X/X2/X2(E)', 'min_7000_10000_tl_x_v201',
565-
ASSUMED, 'no device report on this DTC',
565+
CONFIRMED, "maintainer's MIN 10000TL-X, scan on v1.1.6 — DTC 5201 read from legacy "
566+
"holding 43. Note this unit has no VPP range (30000/30099 return Illegal "
567+
"Function), so the V2.01 key below is downgraded to min_7000_10000_tl_x "
568+
"by convert_to_legacy_profile(). It is the reference device most fixes "
569+
"are verified against",
566570
),
567571

568572
# -- MOD/MID-XH series --

docs/troubleshooting/dtc-debugging.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,7 @@ These statuses live in `DTC_REGISTRY` in `auto_detection.py`, which is the singl
103103
|----------|-------------|---------|--------|
104104
| 5100 | MIN 2500-6000TL-XH/XH2/XHE/XA | tl_xh_3000_10000_v201 | ✅ Confirmed (#71) |
105105
| 5200 | MIC 600-3300TL-X/X2/X2(Pro); MIN 2500-6000TL-X/X2/X2(Pro)/X2(Pro.E) | min_3000_6000_tl_x_v201 | ⚠️ Unconfirmed — refined at runtime by per-MPPT energy check |
106-
| 5201 | MIN 7-10KTL-X/X2/X2(E) | min_7000_10000_tl_x_v201 | ⚠️ Unconfirmed |
106+
| 5201 | MIN 7-10KTL-X/X2/X2(E) | min_7000_10000_tl_x_v201 | ✅ Confirmed (MIN 10000TL-X, DTC from legacy holding 43) |
107107

108108
### MOD / MID / MAC Series — Three-Phase Hybrid / Grid-Tied
109109

0 commit comments

Comments
 (0)