Repository navigation
feat: add experimental local Deye ToU adapter and isolated preview - #5497
arturonaredo wants to merge 2 commits into
Conversation
Combined local shadow trial — initial evidence, not the final-duration reportThe three independent draft PRs are #5486 (import diagnostics), #5497 (local DEYE)
What the live trial has found so far:
As of 00:55 UTC, this is roughly 22 minutes of combined-trial evidence, |
Keep manual SOC floors and power ceilings, and replay proposed commands only against isolated states. Physical timetable repair and hold commissioning remain explicit hypotheses.
Sources: PR springfall2008#5498 at 091716c and PR springfall2008#5497 at 68682c7. Conditional stale-SOC scenarios retain their freshness blocker. All controls remain shadow-only in the local rollout.
Shadow trial update — corrected planner and command replay installedThe integration branch now runs signed revision Corrections supported by a real-data replay:
Validation: the complete local test runner, with the native kernel required, Live findings and preserved boundaries:
SK and the existing EV executor still control the installation. There is no joint |
|
Adoption / migration tracking: #5501. This separate feature request records the proposed local Deye/Solarman scope, compatibility and commissioning gates, and the relationship to the existing SK ToU report #3895. It does not claim upstream approval or physical commissioning and is not an automatic closure of the earlier issue. |
|
Observed PR source: Local DEYE remains independent from SK/Deye Cloud; HA read-back is only observed, never hardware confirmation. Adoption is tracked separately in #5501. Final bounded shadow-observation checkpoint — not an activation approvalThe scheduled eight-hour checkpoint closed at 2026-10-12 07:37:44 UTC.
Combined-installation elapsed time: 7h 4m 48s. Observed outcome and limits
Reproducible software checksThe installed combined source previously passed the complete local native-kernel All three published PR heads also passed a read-only GitHub Actions matrix on Maintainer request: please approve the pending upstream DecisionNot ready to activate. The observed installation is still off/shadow-only. |
|
Superseded by #5505, rebuilt independently from upstream v9.3.8 following the shared-template direction in #5501. The replacement uses a maintained Deye/Solarman custom profile and opt-in generic observed service sequences, not a renamed brand-specific adapter. This branch and its evidence are retained; no deployment is part of the replacement. The separate joint-apply dependency remains explicitly out of scope for #5505. |
Summary
Adds an opt-in, experimental
DEYElocal Solarman profile, independently ofthe existing
SKandDeyeCloudprofiles. Predbat remains the only planner andinverter executor; no additional HA automation, cloud client or Modbus poller.
row boundaries and leaving an inconsistent table.
enabling grid charge. Bounded retries, drift reconciliation and visible failure.
peak shaving, export policy or EV relays.
explicitly commissioned per installation; no global SK freeze override.
deye_shadowexercises the same translator against a privatestate copy alongside an unchanged existing controller. Its services cannot
reach HA. Invalid real tables stay blocked; any sorted-table hypothesis is
explicitly labelled and remains non-executing.
This PR does not include the import diagnostic (#5486) or joint-planning changes
(requirements: #5490). Those are separate workstreams and will be tested together
on a separate integration branch.
Validation completed locally
--quicksuite withPREDBAT_KERNEL_REQUIRED=1: passed; four existing slowsuites excluded by the standard quick selector.
fresh fixtures, block network connections and stub only settling waits.
Execute.execute_planthrough a realInverter: demand, partial target,commissioned hold, target change, midnight-spanning window and return to demand.
Exceptions are not swallowed; actual proposed service targets/values are checked.
service-template calls remain identical; private observations/config are unchanged.
missing-link/image warnings.
inverter registry gives a broad/critical graph radius; disabled-path regression
tests and explicit unchanged-profile checks are therefore important.
Why draft / what is not proven
Solarman v26.01.31 can publish an optimistic HA state after a register write ACK.
The adapter calls this observed, never hardware-confirmed. Software tests do
not prove hold behaviour, firmware compatibility, electrical limits or topology.
Immediate control also retains its last physical table if Predbat/HA stops; it
does not provide an independent watchdog/rollback controller.
No physical commissioning result is claimed. An installation-level shadow trial
will be reported in a follow-up comment with its actual observed duration,
installed commit, failures and remaining activation gates. No private configs,
vehicle identifiers, credentials or raw household captures are attached.