fix(gfql): release-blocker canvas — stale memo, polars-gpu target, LazyFrame crash, divergence pins - #1878
Conversation
|
Boundary-testing pass (owner review concern) updates on this branch:
|
|
Verification bundle — all green at final head
Test-amplification round-001 (owner-directed) is running around the touched seams + error classes; any additions land as follow-up commits on this branch before merge. 🤖 Generated with Claude Code |
|
Final: all gates green at head
Amplification round-001 (owner-directed) is folded in: ~600 probes across 3 error-class agents → 25 confirmed instances → 7 fixed here (incl. 3 HIGH: nan-clean stale-verdict silent-wrong rows, direct-hop lazy crash class ×16 shapes, WITH…MATCH re-entry self-contradictory NIE), 5 deferred behind issues #1879–#1882 with strict-xfail pins. Provenance audit: nearly every fixable bug shipped in 0.58.0 (released mid-campaign, 2026-07-21); three are pre-campaign-ancient; zero introduced by this stack. Artifacts: plans/gfql-release-amplification/round-001/. 🤖 Generated with Claude Code |
a56457b to
0b0a615
Compare
0b0a615
into
feat/gfql-engine-resolve-modern
c1775ca to
0b0a615
Compare
|
This PR was NOT merged by anyone — GitHub auto-marked it. During branch-ref repair, this PR's head briefly pointed at the base branch's own commit ( |
Stacked on #1873 (next in the line). The owner-directed issue canvas verified 13 open issues empirically at the stack head; this PR fixes the three confirmed blockers and pins the two deferred divergences:
count(*)returned a STALE answer after in-place frame mutation — the equal-domain degree-count memo setattr'd onto the caller keyed byid()(the BLOCKER-1 pattern). Removed, not re-keyed: per-execution it never hits, and cross-call reuse is the fingerprint-validated degree facts' job. String-id mutation pin (the int-id repro is masked by the dense kernel), repeat==one-shot identity.engine='polars-gpu'(they run before the chain route, the only place the GPU target was set). Dispatch now sets the target; all collects go throughlazy.collect(GPU-or-error); a non-GPU-executable plan is an honest recorded decline. Source pin forbids bare collects. Needs the dgx GPU run before merge.Also verified fixed at head and commented on: #1712, #1753, #1748, #1723, #1713, #1744, #1726 (polars CPU), #1745 (crash class). Closed: #1795. Canvas + probe logs: plans/gfql-benchmark-numbers/.
🤖 Generated with Claude Code
https://claude.ai/code/session_01MF7uRZLKZaD6Q9FGWSmyXi