Repository navigation
docs(squad): record what the VMM/offload/streaming/batching push established - #808
Merged
Merged
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…blished A durable record of the load-bearing results, including the claims that were refuted. Three of this push's most useful outcomes were corrections: seq-major had not realised the 8x floor (bindings ignored the layout descriptor), the decoder was not declining capture (a cached env var in a long-lived test process), and a fixed KV stride does not remove re-capture (the engine invalidates unconditionally on growth). Also records the #736 audit's recurring finding -- four of five slices found over-reservation rather than an ungoverned allocation -- and the guidance that follows from it. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: d60eb808-7cc6-4abc-b48d-2a6dd3841624
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #808 +/- ##
===========================================
+ Coverage 79.43% 83.55% +4.12%
===========================================
Files 331 12 -319
Lines 137783 5192 -132591
Branches 137783 5192 -132591
===========================================
- Hits 109442 4338 -105104
+ Misses 23994 677 -23317
+ Partials 4347 177 -4170
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
|
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.
A durable record of the load-bearing results from this push, with every claim tied to a merged executable test — and, deliberately, the claims that were refuted.
Three of the most useful outcomes were corrections to things I had stated as achieved:
Also records the #736 audit's recurring finding — four of five slices found over-reservation rather than an ungoverned allocation — and the guidance that follows: start from use, not from allocation, because governing a bypass without sizing it to use converts invisible waste into charged waste, which tightens admission and costs concurrency.
Docs only, no code paths touched.
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com