Repository navigation
qwen4exp: reduce prefill QSA score workspace - #17
Conversation
Assisted-by: Codex
Work report - QSA memory candidate handed off for reviewOwner and scope: Codex subagent Why: This candidate reduces peak live workspace for four-head QSA prefill and keeps the original graph for unqualified head counts, short queries and small reservations. The reviewed result is a capacity improvement with neutral measured throughput, not a speedup claim. Done and evidence: The commit changes only Checks: At handoff, Remaining: Root review of this exact head and the repository's remaining PR checks. Issue #9 still tracks the broader context, near-full-pool and combined-runtime acceptance criteria. No merge was performed in this delivery step. Next: Root reviews PR #17 at |
Work report - QSA candidate merged; runtime acceptance remains with rootOwner and scope: The root-authorized delivery step merged PR #17 at reviewed head Done and evidence: Before merge, issue #9 was OPEN, PR #17 was OPEN/MERGEABLE on the exact reviewed head, the PR contained only the two approved paths, and the merged contents' source hashes matched the reviewed values. The merge used the repository's squash strategy with Checks: Root and the independent reviewer found no source blockers. The reported CPU/native/reference/state and proxy-campaign evidence is in the approved PR body and preceding issue reports; this delivery step did not rerun those hardware tests. GitHub's optional Windows test jobs completed with failures after successful builds: Remaining: A build from exact merged source Next: Root rebuilds and deploys the merged revision through the approved lifecycle, runs the outstanding maximum-context/near-full and combined acceptance checks, and updates issue #9 with their results. No runtime action was taken in this delivery step. |
Work report - CI failure provenance follow-upOwner and scope: This follow-up records the two optional Windows CI failures observed after PR #17 merged; it does not change the merged candidate or claim either failure is pre-existing. Windows server test: Job log identifies Windows x64 OpenBLAS test: Job log identifies CTest Scope and limits: The merged change's selector is in Qwen4Exp and is enabled only for four-head QSA prefill. The two failing runs use TinyStories/Llama fixtures and do not exercise that selector. No baseline run was performed, so these failures are not labeled pre-existing. No broad rerun was done. The repository branch was unprotected and these platform checks were not required merge gates; root retains any follow-up triage. Remaining: Root continues merged-revision rebuild/deployment and the issue's maximum-context, near-full-pool and combined-runtime acceptance. Issue #9 remains open. |
Addresses #9
Four-head QSA prefill currently keeps the combined head-score product alive across a reshape and ReLU. Build and accumulate one rectified head at a time to reduce the pool-by-token workspace. Keep the original graph for other head counts, queries of eight tokens or fewer, and small reservations. The PP guard leaves enough QSA workspace for the unchanged small-query path.
Add an H4 architecture fixture while retaining H64, a two-sequence eight-token verify/MTP case, and an H4 generated model for the maintained state tests. Public APIs, GGUF data and KV formats are unchanged.
Validation:
Codex implemented and reviewed this change under the owner's approved Flash Next plan.