Repository navigation
docs(debug-journal): fold in the 2026-09-27 queue slice (15 candidates); correct entries overtaken by the #5126/#5251/#5220/#5231/#5216/#5247/#5256 merge batch - #5260
Merged
Conversation
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
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.
Automated journal flush (triage bot, 2026-09-27). Folded in the first 15 queued candidates in filename sort order (
5172-gateway-ack-two-designs→5211-component-constructor-in-tests-aliases-live-args); 22 candidates remain queued (5213 onward).Folded in
_subscribe_acks,_send_control30s window + one re-send +_acks_seenfallback,_process_ackrefusal-outranks-ok, clock-seededPBAT<n>ids, prune-after-send) and warns against starting triage from PR fix: reconcile Hub command ACKs before retrying inverter writes #5172's own symbol names.parse_cid_int()/cid_value_matches()and the coreget_arg()float coercion all accept"1234567890_00"as a huge number with no exception; B0063 unhandled. Re-verifiedparse_cid_inton current main.dailyUsedkept counting; docstring's mode/battery-export claim is DEYE-inferred and not settled (marked suspected); manual override does not win (set_arg_autooverwrite=True); the log-forensics tip (component logs every API payload; widensubstrfield width when mining).rate_replicate()negative-minute back-fill +history_to_future_rates()shift, exact only on fixed tariffs; master issue Wrong Rates in history #3267;_previous_day_ratesas the in-tree previous-day pattern.compute_metric()is now a signed weighted average with the expertpv_metric90_weight(default 0.15) alongsidepv_metric10_weight, andload_scaling10is clampedload_scaling90 <= load_scaling <= load_scaling10; the candidate's "PV10-only downside blend at plan.py:1903" was overtaken by the PV90/envelope rework. Finding 1 (days_previous_auto ignores user weights) verified as written.run()call site — verified on main atsolis.pyrun()).has_service_api/*_servicesymptom row: the six*_serviceitems are free-form parameterized service routes (call_service_template(),{placeholder}resolution, dedup/repeatinterplay), window programming stays entity-based, noset_charge_windowitem exists; the third integration Pho3niX90/solis_modbus does expose dispatch (register 34502 gate) while wills106/hultenvp do not — Solis "remote dispatch" is a template/config question. The corrected Unneccessay updates to inverter causing eeprom wear #2328 story is in the Solis row: PR fix(inverter): Persist inverters across planning cycles #5126 made the commit-once ledger real (pre-fix(inverter): Persist inverters across planning cycles #5126 the every-cycle export commit was live).fill_load_from_power()default-on, repeats up to ~8 days, benign triggers (quantisation plateaus, pre-helper zero region), silence knob, plus the three minute_data replay traps (oldest-first, relative series, smoothing branch inert for backwards data).calc_dawn()'s deliberate per-day latch reset, verified live on main; combine_charge_slots gating), and the v9.1.0+ numpy-dtype loader crash recorded in the replay section with the lossless guard shape.Execute.__new__stub trap:ComponentBase.__init__setsself.args = base.args, so a real constructor in a test binds live shared args;__new__+ explicit attrs is the safer shape.Existing entries corrected (merge-driven)
car_charging_plan_smartdefault (symptom row): PR fix(car): default car_charging_plan_smart to on (#5237) #5251 (GH#5237) flipped the default to true, merged 2026-09-26, unreleased at the time of writing — the old "defaults false by design" explanation is now version-gated.refresh_config()re-reads live state and re-creates the dummy entities intoself.argsevery cycle, while the commit-once ledger survives. Pre-fix(inverter): Persist inverters across planning cycles #5126 logs keep the older reading. The GivTCP row'sin_calibrationsentence re-anchored ontorefresh_config()(per-cycle re-sampling survives).last_committed/commit_pending/commit_needed()—last_export_schedule_committedno longer exists) and marked version-dependent: pre-fix(inverter): Persist inverters across planning cycles #5126 the rebuild reset the latch each cycle, so the Unneccessay updates to inverter causing eeprom wear #2328 every-cycle commit signature applies to older logs only.Self-Use(33) hold is now PR fix(solis): manage the Energy Storage Control Switch on FB00 firmware #5256 (merged 2026-09-26); earlier mains never wrote the switch on GS_fb00 at all.Execute.__new__trap bullet: theinverter_clock_skew_discharge_startread is now cited by symbol and the args-membership branch, not the driftedinverter.py:1624/:1563line numbers.Verification notes
a364a5ce): symbols grepped, mechanisms read, and the two replay-verified candidates (5204, 5205) kept their "replay-verified" status explicit; the Sunsynk trigger question stays marked suspected../run_pre_commitgate ran: cspell + markdownlint pass on the journal and dictionary (5 new dictionary words:dtype,hultenvp,kwarg,multiarray,setstate); the quick suite ran 122/122 passed before aborting attest_teslemetry_local_weekday_follows_the_base_clockat 00:34 BST — the documented host-clock window failure (00:00–01:00 local), unrelated to this docs-only change — and all 22 modules registered after teslemetry were run separately and passed.