fix(apr-cli): §69 RC3 CONFIRMED on gx10 — prepend prompt preamble to HumanEval full_program#1635
Merged
Merged
Conversation
This was referenced May 12, 2026
Closed
320cfde to
e263115
Compare
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…RGED via 3/3 §68-trio flips (PMAT-CODE-SHIP-TWO-SECTION-70) (#1636) §69 (PR #1633) enumerated 4 candidate root causes for the apr eval HumanEval false-failure. §70 reports the empirical disambiguation on gx10 via the diagnostic surface (PR #1634), the 1-PR fix (PR #1635), and the discharge proof. 70.1 RC disambiguation on gx10 (canonical 7B Q4K APR teacher): - RC1 (state leak) : FALSIFIED — coherent 1031-byte response - RC2 (false-negative) : FALSIFIED — python3 actually exited 1 - RC3 (format!() bug) : CONFIRMED — imports stripped - RC4 (max_tokens trunc) : FALSIFIED — 524-char completion present 70.2 Why §68 was wrong: §68's R1+R2 0/3 flip rate on the known-failed trio was correct evidence; the inference ("Class B sampling/ quantization") was a leap. The TRUE class was Class C (harness-RC3), invisible to R1+R2 because R1+R2 doesn't touch the format!() at line 400. 70.3 The fix (PR #1635): new `extract_prompt_preamble(prompt, entry)` helper + ChatML-branch prepend in run_humaneval_inference. 7 unit tests cover the helper + RC3 falsifier. 70.4 Discharge proof — 3/3 §68 trio flip: | Task | §68 pre-fix | §68 R1+R2-only | §70 RC3-fix | | HumanEval/1 | FAIL | FAIL | PASS | | HumanEval/3 | FAIL | FAIL | PASS | | HumanEval/6 | FAIL | FAIL | PASS | Flip rate: 100%. 70.5 SHIP-005 path: 164-run dispatched on gx10 (commit b7e69bf); ~5h CPU wall. Discharge condition: post-fix pass@1 >= 84.80%. 70.6 Methodology lesson #17 NEW: pre-fix RED smoke can mask the bug class. A 0/N flip rate in a smoke proves only that the candidate fix doesn't move the needle, NOT that any specific failure class is responsible. The class must be identified via diagnostic instrumentation (APR_EVAL_DEBUG=1), not inferred from a flip rate. 70.7 Cumulative methodology lessons through §70 (lesson #17 added). 70.8 Ship-% movement: MODEL-1 stays 94% pending 164-run completion; path to 95% is single rerun + verdict check, no further code changes. MODEL-2 unchanged at 57%. Spec version: 3.14.0 → **3.16.0** (also reapplies §69 banner at v3.15.0 since PR #1633 has not yet landed on main — when #1633 lands, the §69 section will exist; this commit's banner stack accommodates that). Refs: - contracts/apr-eval-humaneval-harness-invariant-v1.yaml v1.1.0 - evidence/section-70-rc3-fix-2026-05-12/findings.json - /tmp/apr_eval_debug_HumanEval_{1,3,6}.json (gx10 evidence) - PR #1633 (§69 spec), PR #1634 (diagnostic surface), PR #1635 (RC3 fix) Closes task #52 (PMAT-CODE-SHIP-TWO-SECTION-70). Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
e263115 to
dd1b6e8
Compare
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…-CODE-MBPP-DIAG-001) The §69 diagnostic surface (PR #1634) and §70 RC3 fix (PR #1635) closed the harness-bug class for HumanEval. MBPP's path (run_mbpp_inference + run_mbpp_inference_cuda) was not yet instrumented. This PR extends APR_EVAL_DEBUG to MBPP so future investigation of MBPP failures has ground-truth diagnostics on the same surface. What changes: - run_mbpp_inference (CPU path) now calls execute_python_test_with_diagnostics and emits /tmp/apr_eval_debug_MBPP_<task>.json when APR_EVAL_DEBUG=1 is set. - run_mbpp_inference_cuda (CUDA path) gets the same treatment. What does NOT change: - run_mbpp_inference still uses the legacy AprTransformer::forward_with_cache + AprKVCache path. PMAT-CODE- SHIP-005-FIX (PR #1616) replaced this for HumanEval with realizar:: run_inference + OwnedQuantizedModel::from_apr. MBPP needs the same routing fix — but that's a separate multi-PR cascade scope (also includes H4 ChatML wrap + R1+R2 extraction equivalents for MBPP). Out of scope for this PR. - MBPP prompts are natural language (not Python signatures), so the §70 RC3 import-stripping bug does NOT apply to MBPP. Why ship this now: - Pure diagnostic — zero behaviour change for non-APR_EVAL_DEBUG callers - Lets us run a 1-problem MBPP smoke under APR_EVAL_DEBUG=1 to verify the legacy path's failure mode (currently undiagnosed) - Mirrors the pattern that successfully diagnosed §69 RC3 in 5 minutes on gx10 Test plan: - [x] cargo check -p apr-cli --features inference → clean - [x] cargo check -p apr-cli --features "inference,cuda,training" → clean - [x] cargo fmt --all → clean - [ ] gx10 single-MBPP-problem APR_EVAL_DEBUG=1 smoke (next slice; will document MBPP failure mode in a §72-class amendment) Refs: - crates/apr-cli/src/commands/eval/inference.rs::write_apr_eval_debug - contracts/apr-eval-humaneval-harness-invariant-v1.yaml v1.1.0 - PR #1634 (HumanEval diagnostic surface) - PR #1635 (HumanEval RC3 fix; cascade base for this branch) Closes task #53 (MBPP harness diagnostic extension; renamed from "RC3 prompt-preamble fix" since RC3 does not apply to MBPP's NL prompts — that decision recorded in commit body). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…gram (PMAT-CODE-SHIP-005-RC3-FIX) §69 (PR #1633) enumerated 4 candidate root causes for the apr eval HumanEval harness bug. The diagnostic surface (PR #1634 APR_EVAL_DEBUG=1) ran live on gx10 (Blackwell GB10) against the canonical 7B teacher Q4K APR. HumanEval/1 diagnostic JSON: task_id: HumanEval/1 response_len: 1031 completion_len: 524 exit_code: 1 ← python3 ACTUALLY exited 1 timed_out: false success: false stderr_head: Traceback (most recent call last): File "/tmp/apr_eval_*.py", line 1, in <module> def separate_paren_groups(paren_string: str) -> List[str]: ^^^^ NameError: name 'List' is not defined. Did you mean: 'list'? RC disambiguation: - RC1 (model state leak): FALSIFIED — apr eval emitted coherent 1031-byte response (matches `apr run` output). - RC2 (false-negative): FALSIFIED — python3 actually returned exit 1; harness reported correctly. - RC3 (format!() bug): CONFIRMED — full_program drops `from typing import List` from problem.prompt. - RC4 (max_tokens truncation): FALSIFIED — closing fence present, 524-char completion extracted successfully. Root cause: the ChatML/markdown branch of run_humaneval_inference uses the extracted code block AS the program (no preamble prepended). The extracted block starts with `def f(x) -> List[str]:` but the typing import lives in problem.prompt (NOT in the model's emitted code block). Result: NameError at line 1 of every program whose signature uses typing aliases (List, Tuple, Dict, Optional, Any, Set, Union — ~70% of the canonical 164 HumanEval set). The 80.49% pass@1 measured in §67 was the LOWER BOUND of the model's real performance; the harness was rejecting otherwise-correct solutions because of stripped imports. Fix (crates/apr-cli/src/commands/eval/inference.rs): - New `extract_prompt_preamble(prompt, entry_point)` helper that returns everything in `prompt` BEFORE `def {entry_point}(`. Empty when: * entry_point is empty or "unknown" * `def {entry_point}(` not found in prompt * No content before the def line - ChatML/markdown branch of run_humaneval_inference now prepends the preamble to the extracted code block: full_program = preamble + "\n" + code + "\n\n" + test + "\n\n" + check - 7 new unit tests cover the helper + the RC3 falsifier. Contract update (contracts/apr-eval-humaneval-harness-invariant-v1.yaml): - v1.0.0 → v1.1.0 - validation_result_v1_1 records the gx10 empirical confirmation: host, binary commit, artifact, problem, exit_code, stderr, RC table, root cause, fix, unit tests, expected lift. - New FALSIFY-HEH-005 falsifier wired to rc3_falsifier_composed_program_is_valid_python. - `pv validate` PASS (2 non-blocking warnings: planned Kani bounds). Expected ship impact: - HumanEval problems using typing aliases (~70% of 164) now compile. - Empirical lift estimate: +5-15pp over the §67 80.49% baseline. - If post-fix pass@1 >= 84.80%, SHIP-005 LIVE-discharges → MODEL-1 95%. - Empirical confirmation requires rerun on gx10 (separate slice). Test plan: - [x] cargo test -p apr-cli --lib --features inference \ extract_prompt_preamble_tests → 7/7 pass - [x] pv validate contracts/apr-eval-humaneval-harness-invariant-v1.yaml → valid - [x] cargo check -p apr-cli --features inference → clean - [ ] rerun APR_EVAL_DEBUG=1 apr eval on HumanEval/1 with fixed binary — expect json.success == true (next slice) - [ ] gx10 164-problem rerun — expect pass@1 ≥ 84.80% Methodology lesson #16 confirmed: manual end-to-end replication (§69 step 2 with the same extracted code) MISSED the RC3 bug because the manual program I built by hand happened to include the import line (or my hand-typed `python3 -c` didn't enforce strict typing). The diagnostic surface (APR_EVAL_DEBUG=1) captured the EXACT byte-for-byte full_program that apr eval executes, exposing the import-stripping bug in 5 minutes on gx10 — vs the §66-§68 chain spending ~10 hours on wrong-class hypotheses. Closes task #49 (PMAT-CODE-SHIP-005-RC3-FIX). Refs: - docs/specifications/aprender-train/ship-two-models-spec.md §69 - contracts/apr-eval-humaneval-harness-invariant-v1.yaml v1.1.0 - PR #1633 (§69 spec); PR #1634 (diagnostic surface) - /tmp/apr_eval_debug_HumanEval_1.json (gx10 evidence) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
57bc90d to
d27bcc1
Compare
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…-CODE-MBPP-DIAG-001) The §69 diagnostic surface (PR #1634) and §70 RC3 fix (PR #1635) closed the harness-bug class for HumanEval. MBPP's path (run_mbpp_inference + run_mbpp_inference_cuda) was not yet instrumented. This PR extends APR_EVAL_DEBUG to MBPP so future investigation of MBPP failures has ground-truth diagnostics on the same surface. What changes: - run_mbpp_inference (CPU path) now calls execute_python_test_with_diagnostics and emits /tmp/apr_eval_debug_MBPP_<task>.json when APR_EVAL_DEBUG=1 is set. - run_mbpp_inference_cuda (CUDA path) gets the same treatment. What does NOT change: - run_mbpp_inference still uses the legacy AprTransformer::forward_with_cache + AprKVCache path. PMAT-CODE- SHIP-005-FIX (PR #1616) replaced this for HumanEval with realizar:: run_inference + OwnedQuantizedModel::from_apr. MBPP needs the same routing fix — but that's a separate multi-PR cascade scope (also includes H4 ChatML wrap + R1+R2 extraction equivalents for MBPP). Out of scope for this PR. - MBPP prompts are natural language (not Python signatures), so the §70 RC3 import-stripping bug does NOT apply to MBPP. Why ship this now: - Pure diagnostic — zero behaviour change for non-APR_EVAL_DEBUG callers - Lets us run a 1-problem MBPP smoke under APR_EVAL_DEBUG=1 to verify the legacy path's failure mode (currently undiagnosed) - Mirrors the pattern that successfully diagnosed §69 RC3 in 5 minutes on gx10 Test plan: - [x] cargo check -p apr-cli --features inference → clean - [x] cargo check -p apr-cli --features "inference,cuda,training" → clean - [x] cargo fmt --all → clean - [ ] gx10 single-MBPP-problem APR_EVAL_DEBUG=1 smoke (next slice; will document MBPP failure mode in a §72-class amendment) Refs: - crates/apr-cli/src/commands/eval/inference.rs::write_apr_eval_debug - contracts/apr-eval-humaneval-harness-invariant-v1.yaml v1.1.0 - PR #1634 (HumanEval diagnostic surface) - PR #1635 (HumanEval RC3 fix; cascade base for this branch) Closes task #53 (MBPP harness diagnostic extension; renamed from "RC3 prompt-preamble fix" since RC3 does not apply to MBPP's NL prompts — that decision recorded in commit body). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
3 tasks
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…s@1 (PMAT-CODE-SHIP-TWO-SECTION-71) (#1642) §70 (PR #1636) confirmed RC3 (format!() drops imports) on gx10 and shipped the fix (PR #1635) + diagnostic surface (PR #1634). §71 reports the empirical 164-run discharge proof on gx10: Result: 142/164 problems passed → pass@1 = 86.59% Floor: 84.80% (AC-SHIP1-005 with 1.2% tolerance) Headroom above floor: +1.79pp Compared to §67 baseline (H4 ChatML only): 80.49% (132/164) RC3 fix flipped 10 additional problems → +6.10pp gain pass@10 ≈ 100%, pass@100 = 100% SHIP-005 LIVE-DISCHARGED. The §65→§71 cascade is closed for SHIP-005. Run metadata: Host: gx10-a5b5 (Blackwell GB10, aarch64) Binary: /home/noah/src/aprender/target/release/apr @ b7e69bf Artifact: qwen2.5-coder-7b-instruct-q4k.apr Wall: 5h 50min (08:10 → 14:00 UTC) Sample: T=0.0, 1 sample, max_tokens=512 (greedy) §17.5 chain post-§71: SHIP-002 DISCHARGED (no change) SHIP-005 PARTIAL → LIVE-DISCHARGED ← §71 SHIP-006 DISCHARGED (no change) SHIP-007 PARTIAL — multi-PR CUDA cascade (§63 — separate track) SHIP-008 DISCHARGED (no change) MODEL-1 ship %: 94% → 95% (4 of 5 §17.5 PARTIALs LIVE-discharged). Path to 96% requires SHIP-007 multi-PR CUDA cascade. MODEL-2 ship %: unchanged at 57% (independent track). Methodology lesson #18 NEW: §70 → §71 closes the predict-then-verify loop. A fix whose 3/3 smoke flip and whose mechanism-based lift estimate (§70.5 predicted +5-15pp) land within the predicted band (actual +6.10pp) IS the discharge evidence; no further investigation needed. The cascade arc closes when prediction matches empirical. Spec v3.16.0 → v3.17.0. Evidence: - evidence/section-71-ship-005-discharged-2026-05-12/humaneval-164-rc3-gx10.json (full 164-problem JSON, 24KB) - evidence/section-71-ship-005-discharged-2026-05-12/findings.json - evidence/section-70-rc3-fix-2026-05-12/findings.json (3/3 trio) - evidence/section-69-harness-bug-2026-05-12/findings.json (smoking-gun) - evidence/section-67-h4-164-run-result-2026-05-12/findings.json (baseline) Closes task #56 (PMAT-CODE-SHIP-TWO-SECTION-71). Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…-CODE-MBPP-DIAG-001) (#1641) The §69 diagnostic surface (PR #1634) and §70 RC3 fix (PR #1635) closed the harness-bug class for HumanEval. MBPP's path (run_mbpp_inference + run_mbpp_inference_cuda) was not yet instrumented. This PR extends APR_EVAL_DEBUG to MBPP so future investigation of MBPP failures has ground-truth diagnostics on the same surface. What changes: - run_mbpp_inference (CPU path) now calls execute_python_test_with_diagnostics and emits /tmp/apr_eval_debug_MBPP_<task>.json when APR_EVAL_DEBUG=1 is set. - run_mbpp_inference_cuda (CUDA path) gets the same treatment. What does NOT change: - run_mbpp_inference still uses the legacy AprTransformer::forward_with_cache + AprKVCache path. PMAT-CODE- SHIP-005-FIX (PR #1616) replaced this for HumanEval with realizar:: run_inference + OwnedQuantizedModel::from_apr. MBPP needs the same routing fix — but that's a separate multi-PR cascade scope (also includes H4 ChatML wrap + R1+R2 extraction equivalents for MBPP). Out of scope for this PR. - MBPP prompts are natural language (not Python signatures), so the §70 RC3 import-stripping bug does NOT apply to MBPP. Why ship this now: - Pure diagnostic — zero behaviour change for non-APR_EVAL_DEBUG callers - Lets us run a 1-problem MBPP smoke under APR_EVAL_DEBUG=1 to verify the legacy path's failure mode (currently undiagnosed) - Mirrors the pattern that successfully diagnosed §69 RC3 in 5 minutes on gx10 Test plan: - [x] cargo check -p apr-cli --features inference → clean - [x] cargo check -p apr-cli --features "inference,cuda,training" → clean - [x] cargo fmt --all → clean - [ ] gx10 single-MBPP-problem APR_EVAL_DEBUG=1 smoke (next slice; will document MBPP failure mode in a §72-class amendment) Refs: - crates/apr-cli/src/commands/eval/inference.rs::write_apr_eval_debug - contracts/apr-eval-humaneval-harness-invariant-v1.yaml v1.1.0 - PR #1634 (HumanEval diagnostic surface) - PR #1635 (HumanEval RC3 fix; cascade base for this branch) Closes task #53 (MBPP harness diagnostic extension; renamed from "RC3 prompt-preamble fix" since RC3 does not apply to MBPP's NL prompts — that decision recorded in commit body). Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
4 tasks
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…ode-block extraction (PMAT-CODE-MBPP-H4-FIX) Mirrors the §70 HumanEval H4 + R1+R2 cascade (PRs #1616, #1628 squashed via #1634/#1635) for MBPP. The legacy `AprTransformer::forward_with_cache + AprKVCache` path was producing NL-prose continuations on MBPP prompts (see PR #1641 MBPP/11 smoke: SyntaxError on "Example:" prose, 0/1 pass). Changes: - Replace `AprTransformer::forward_with_cache + AprKVCache` loop with `realizar::run_inference + InferenceConfig::with_prompt` (ChatML auto-wrap for instruct models). - Parse `\`\`\`python ... \`\`\`` markdown blocks from the response via `extract_python_code_block_targeted(&result.text, None)`. MBPP has no `entry_point` in the problem schema; first-non-empty-block fallback is appropriate. - Raw-continuation fallback preserved: strip prompt prefix, truncate at next top-level def — used when no markdown block found. Out of scope (vs HumanEval cascade): - §70 RC3 prompt-preamble handling: MBPP prompts are NL ("Write a python function to..."), no Python imports to preserve. `extract_prompt_preamble` not applicable. - §17.5 chain impact: MBPP is not in §17.5; this PR does not move ship %. - Full 500-problem rerun: dispatch as a separate evidence slice. Test plan: - [x] cargo check -p apr-cli --features inference → clean - [x] cargo fmt --all → clean - [ ] gx10 single-MBPP-problem APR_EVAL_DEBUG=1 smoke (next slice) - [ ] gx10 sanitized-subset MBPP rerun for pass@1 measurement Refs: - crates/apr-cli/src/commands/eval/inference.rs::run_humaneval_inference (mirror) - PR #1641 (MBPP diagnostic surface, cascade base) - evidence/section-71-ship-005-discharged-2026-05-12/ (HumanEval cascade pattern) - project_2026_05_12_mbpp_legacy_path_finding.md (cascade scope) Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
noahgift
added a commit
that referenced
this pull request
May 12, 2026
…ode-block extraction (PMAT-CODE-MBPP-H4-FIX) (#1645) Mirrors the §70 HumanEval H4 + R1+R2 cascade (PRs #1616, #1628 squashed via #1634/#1635) for MBPP. The legacy `AprTransformer::forward_with_cache + AprKVCache` path was producing NL-prose continuations on MBPP prompts (see PR #1641 MBPP/11 smoke: SyntaxError on "Example:" prose, 0/1 pass). Changes: - Replace `AprTransformer::forward_with_cache + AprKVCache` loop with `realizar::run_inference + InferenceConfig::with_prompt` (ChatML auto-wrap for instruct models). - Parse `\`\`\`python ... \`\`\`` markdown blocks from the response via `extract_python_code_block_targeted(&result.text, None)`. MBPP has no `entry_point` in the problem schema; first-non-empty-block fallback is appropriate. - Raw-continuation fallback preserved: strip prompt prefix, truncate at next top-level def — used when no markdown block found. Out of scope (vs HumanEval cascade): - §70 RC3 prompt-preamble handling: MBPP prompts are NL ("Write a python function to..."), no Python imports to preserve. `extract_prompt_preamble` not applicable. - §17.5 chain impact: MBPP is not in §17.5; this PR does not move ship %. - Full 500-problem rerun: dispatch as a separate evidence slice. Test plan: - [x] cargo check -p apr-cli --features inference → clean - [x] cargo fmt --all → clean - [ ] gx10 single-MBPP-problem APR_EVAL_DEBUG=1 smoke (next slice) - [ ] gx10 sanitized-subset MBPP rerun for pass@1 measurement Refs: - crates/apr-cli/src/commands/eval/inference.rs::run_humaneval_inference (mirror) - PR #1641 (MBPP diagnostic surface, cascade base) - evidence/section-71-ship-005-discharged-2026-05-12/ (HumanEval cascade pattern) - project_2026_05_12_mbpp_legacy_path_finding.md (cascade scope) Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
4 tasks
noahgift
added a commit
that referenced
this pull request
May 13, 2026
…T-CODE-V0-33-0-RELEASE-PREP) 🎉 v0.33.0 marks **MODEL-1 SHIP % = 100%** for SHIP-TWO-001. All 10 AC-SHIP1-* falsifiers are LIVE-discharged on the canonical 7B Qwen2.5-Coder-Instruct Q4_K_M teacher (lambda-vector RTX 4090, --features cuda). This release prep PR ships: 1. CHANGELOG.md [0.33.0] entry with §69-§75 highlights: - 🎉 MODEL-1 SHIP % = 100% (all 10 AC-SHIP1-* LIVE) - Fixed: SHIP-007 F32 GEMV PTX layout (PR #1651, §75) — 124.6 tok/s - Fixed: SHIP-005 HumanEval RC3 (PR #1635, §70/§71) — pass@1 86.59% - Added: APR_EVAL_DEBUG=1 diagnostic surface (PR #1634) - Added: APR_GPU_STAGE_DUMP=<dir> diagnostic surface (PR #1649) - Added: MBPP harness H4 fix (PR #1645) - Added: 2 new falsifiable contracts (apr-eval-humaneval-harness- invariant v1.1.0, apr-ship-007-gpu-stage-bisection v1.0.0) - Methodology lessons #16-22 captured in MEMORY.md - Spec: v3.13.0 → v3.21.0 across §67-§75 2. Workspace version bump: - [workspace.package].version: 0.32.0 → 0.33.0 - Root [package].version (aprender facade crate): 0.32.0 → 0.33.0 - 28 sub-crate version literals: 0.32.0 → 0.33.0 3. `cargo check -p aprender` → clean (workspace builds at 0.33.0). Out of scope for this PR (separate steps after #1651/1652 land + this PR lands): - Tag release `v0.33.0` on main - Cascade publish to crates.io (per memory project_ship_two_001_v0_32_0_release.md — 15 user-facing crates + 7 internal-tier in topological dependency order; uses `make publish CRATE=<name>`) - Post-publish QA per `feedback_post_publish_qa_required.md` — `cargo install aprender --force` + `/dogfood` GO verdict required before declaring release done (v0.31.1 was yanked for skipping this) - GitHub Release with §75 narrative - HF artifact verification (paiml/qwen2.5-coder-7b-apache-q4k-v1 sha256 already verified by §72 SHIP-010 LIVE evidence; double-check before release announcement) This PR ships ONLY the version-bump + CHANGELOG. Publishing is the next step after merge. Refs: - §75 MODEL-1 100% (PR #1652) - §74 SHIP-007 bug localized (PR #1650) - §73 SHIP-007 cascade reduction (PR #1647) - §72 5-AC LIVE cascade (PR #1646) - §71 SHIP-005 LIVE-DISCHARGED (PR #1642) - §70 RC3 fix (PR #1636) - §69 Q4K hypothesis falsified (PR #1633) - PR #1635 RC3 prepend - PR #1634 diagnostic surface + contract - PR #1648 SHIP-007 contract scaffold - PR #1649 SHIP-007 PR-B stage dump - PR #1651 SHIP-007 PR-E F32 GEMV layout fix Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
noahgift
added a commit
that referenced
this pull request
May 13, 2026
…T-CODE-V0-33-0-RELEASE-PREP) (#1653) 🎉 v0.33.0 marks **MODEL-1 SHIP % = 100%** for SHIP-TWO-001. All 10 AC-SHIP1-* falsifiers are LIVE-discharged on the canonical 7B Qwen2.5-Coder-Instruct Q4_K_M teacher (lambda-vector RTX 4090, --features cuda). This release prep PR ships: 1. CHANGELOG.md [0.33.0] entry with §69-§75 highlights: - 🎉 MODEL-1 SHIP % = 100% (all 10 AC-SHIP1-* LIVE) - Fixed: SHIP-007 F32 GEMV PTX layout (PR #1651, §75) — 124.6 tok/s - Fixed: SHIP-005 HumanEval RC3 (PR #1635, §70/§71) — pass@1 86.59% - Added: APR_EVAL_DEBUG=1 diagnostic surface (PR #1634) - Added: APR_GPU_STAGE_DUMP=<dir> diagnostic surface (PR #1649) - Added: MBPP harness H4 fix (PR #1645) - Added: 2 new falsifiable contracts (apr-eval-humaneval-harness- invariant v1.1.0, apr-ship-007-gpu-stage-bisection v1.0.0) - Methodology lessons #16-22 captured in MEMORY.md - Spec: v3.13.0 → v3.21.0 across §67-§75 2. Workspace version bump: - [workspace.package].version: 0.32.0 → 0.33.0 - Root [package].version (aprender facade crate): 0.32.0 → 0.33.0 - 28 sub-crate version literals: 0.32.0 → 0.33.0 3. `cargo check -p aprender` → clean (workspace builds at 0.33.0). Out of scope for this PR (separate steps after #1651/1652 land + this PR lands): - Tag release `v0.33.0` on main - Cascade publish to crates.io (per memory project_ship_two_001_v0_32_0_release.md — 15 user-facing crates + 7 internal-tier in topological dependency order; uses `make publish CRATE=<name>`) - Post-publish QA per `feedback_post_publish_qa_required.md` — `cargo install aprender --force` + `/dogfood` GO verdict required before declaring release done (v0.31.1 was yanked for skipping this) - GitHub Release with §75 narrative - HF artifact verification (paiml/qwen2.5-coder-7b-apache-q4k-v1 sha256 already verified by §72 SHIP-010 LIVE evidence; double-check before release announcement) This PR ships ONLY the version-bump + CHANGELOG. Publishing is the next step after merge. Refs: - §75 MODEL-1 100% (PR #1652) - §74 SHIP-007 bug localized (PR #1650) - §73 SHIP-007 cascade reduction (PR #1647) - §72 5-AC LIVE cascade (PR #1646) - §71 SHIP-005 LIVE-DISCHARGED (PR #1642) - §70 RC3 fix (PR #1636) - §69 Q4K hypothesis falsified (PR #1633) - PR #1635 RC3 prepend - PR #1634 diagnostic surface + contract - PR #1648 SHIP-007 contract scaffold - PR #1649 SHIP-007 PR-B stage dump - PR #1651 SHIP-007 PR-E F32 GEMV layout fix Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>
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.
Summary
§69 (PR #1633) enumerated 4 candidate root causes (RC1-RC4) for the apr eval HumanEval harness bug. The diagnostic surface (PR #1634 `APR_EVAL_DEBUG=1`) was run live on gx10 (Blackwell GB10) against the canonical 7B Qwen2.5-Coder Q4K APR teacher. HumanEval/1 result:
```
exit_code: 1
stderr: NameError: name 'List' is not defined. Did you mean: 'list'?
```
RC3 CONFIRMED: the ChatML/markdown branch uses the extracted code block AS the program, but
problem.prompt's preamble (e.g.,from typing import List) is NEVER included. Function signature usesList[str]→ python3 NameError at line 1.The 80.49% pass@1 in §67 was a LOWER BOUND, not the model's real capability.
RC disambiguation table
What ships
Code (
crates/apr-cli/src/commands/eval/inference.rs)extract_prompt_preamble(prompt, entry_point)returns everything beforedef {entry_point}(.Contract (
contracts/apr-eval-humaneval-harness-invariant-v1.yaml)validation_result_v1_1records gx10 empirical evidencepv validatePASSUnit tests (7/7 pass)
captures_typing_import_preamblecaptures_multiline_preambleempty_when_def_at_startempty_when_entry_missingempty_when_entry_emptyempty_when_entry_unknownrc3_falsifier_composed_program_is_valid_pythonExpected ship impact
Test plan
pv validate→ contract validcargo check -p apr-cli --features inference→ cleanAPR_EVAL_DEBUG=1rerun on HumanEval/1 → expectjson.success == trueRefs
/tmp/apr_eval_debug_HumanEval_1.jsonon gx10 (raw evidence)🤖 Generated with Claude Code