Repository navigation
chore(squad): scribe round 8 — async-offload/correctness wave consolidation - #555
Merged
Merged
Conversation
…wave Merge 6 inbox notes into decisions.md (20458->20332 B, under gate); archive two historical wave records to decisions-archive/2026-07.md. Update mary/cohaagen/harry/melina histories (mary compacted to history-archive). Wave: #544 async page-in increment-1, #552 capture observability, #554 native-CUDA session-reuse recurrent-state reset (closes #553), 27B offload A/B proof (2.9x VRAM, byte-exact). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
justinchuby
enabled auto-merge (squash)
July 31, 2026 09:14
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #555 +/- ##
==========================================
+ Coverage 80.59% 81.16% +0.57%
==========================================
Files 315 315
Lines 123446 123446
Branches 123446 123446
==========================================
+ Hits 99489 100198 +709
+ Misses 19907 19191 -716
- Partials 4050 4057 +7
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
🔴 Benchmark Regression DetectedComparison of criterion micro-benchmarks: PR head vs merge-base, measured on the same runner in the same job (base first → PR second).
Visual flags: Host infoWhat this cannot catch
|
justinchuby
added a commit
that referenced
this pull request
Aug 23, 2026
…-starvation, stale-figure disclaimers)
Adds to the same EXPERIMENTAL spike test file:
- QWEN15_MOE_A27B fixture shape (experts=60, hidden=2048, inter=1408,
top_k=4, cited from Qwen1.5-MoE-A2.7B's published config.json) and a
new #[ignore]d test exercising it, since prior architecture notes name
this exact fixture ("qwen15-moe-qmoe") and it had not been measured
directly before.
- A deterministic "real cold bytes/step, measured from fixture, NOT
modeled" print emitted before any control run or timing number, for
every shape, per measurement-discipline's bytes/counters-first rule.
- A new resize-starvation stress test
(qmoe_routed_residency_guard_resize_starvation_stress_1000_steps)
driving the real CudaWeightResidency::acquire_routed_residency /
resize_safe_point / execute_resize machinery through 1000 back-to-back
acquire-hold-release cycles simulating sequential decode-step dispatch:
every resize attempted while a guard is held is confirmed rejected
(NotSafePoint), every resize attempted in the inter-step gap is
confirmed accepted and executed -- no starvation, no guard-count drift
over 1000 cycles. Explicitly scoped: single-threaded/sequential only;
concurrent-stream starvation remains a separate, larger follow-up spike.
No new allocator/cache, no kernel ABI change, no sub-weight VMM/host-map
composition attempted or built -- per the reviewing instruction, this
spike remains bounded to what today's whole-tensor-granularity zero-copy
mechanism and existing RoutedResidencyGuard/resize machinery already
support. Verdict unchanged: NO-GO for whole-tensor-granularity A' as a
production per-expert cold path.
Correctness: still bit-identical on every arm across all 3 shapes, 2 more
independent idle-A100 runs (GPU 5 and GPU 6). Bandwidth close to and
consistent with the original 2-shape run (~31.5-35.6 GB/s all-cold,
~25.8-29.1 GB/s mixed fc1-cold across all 3 shapes) -- these are freshly
measured on A100/Linux/real-QMoE-GEMV and are NOT #880's RTX4060/WDDM
5.6 GB/s figure nor #925's H200/Linux dense-qwen14b 6.795 GB ceiling;
neither prior figure is reused or blended into these numbers.
Full updated report in
.squad/decisions/inbox/deckard-a-prime-spike-results-cycle7.md, including
the correction that Mobius #550/#555 now close the DeepSeek-V4/GLM-5.2
exporter prerequisite (artifacts still absent from this box).
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.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.
Consolidates 6 inbox decision notes into decisions.md (20458->20332 B, under the 20480 gate) and archives two historical wave records. Updates agent histories.
Wave summary (all merged):
State-only change (decisions/histories/archive). Logs are gitignored.
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com