Repository navigation
[Docs] Publish the 0.2.13 fork-vs-stock measurement, corrected and verified - #3
Merged
Merged
Conversation
…ified The repository had no front-page statement of what the fork is actually worth against the release it was ported to, and the 0.2.13 comparison existed only as a branch report. This publishes it as the evidence it is, with every number re-derived from the raw JSON first. Validation of the existing 0.2.13 report, from its own artifacts: - battery: recomputed from battery/*.json -> reproduces all 8 legs and both metrics; engine parity holds, largest gap 1.1 % (long decode). - GuideLLM: recomputed from guidellm/*.json -> TTFT medians 739/2,622/ 6,688/9,641 ms against stock 949/3,163/7,500/9,659 ms, so the fork is 22.1 / 17.1 / 10.8 / 0.2 % sooner to first token at 1/2/4/8 streams. - three defects found and fixed: the "errored 22/21" column was the successful count (errored_requests is 0 in both arms; cancelled is 4/5/8/14 and identical); "tok/s" was a mean of per-iteration samples, not throughput; and the stock ref named the annotated tag object rather than the commit it points at (fb57f74 -> d2f0964). - one unsupported claim dropped: "12,288/12,288 slots, 0 experts over PCIe" has no shipped artifact behind it. - fairness re-verified on disk: both configs identical apart from paths (conv_route on in both, ignored by stock), expert profiles byte-equal. Front page: a measured block under "What this fork changes" with the chart, the TTFT table, the throughput-in-window row, the engine-parity line, and a Measured-on note that names the drift (2-4 %) and says which differences exceed it. The chart and report live in dirigo-results/0.2.13-port/ with the raw JSON, the harnesses, and scripts/make-chart.py, which recomputes every figure in the chart from that folder's data.
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.
Problem and result
The repository could not answer "what is this fork actually worth against the release it was ported to". The 0.2.13
comparison existed only as a branch report, and re-deriving its numbers from the shipped JSON found three labelling
errors and one unsupported claim. This publishes the comparison as a front-page block with the chart, and makes the
report match its own evidence.
Before: no fork-vs-stock statement on the default branch; the 0.2.13 report said "errored 22 / 21" (those are
successes; the arms errored 0), reported a mean of per-iteration samples as
tok/s, and cited "tag commitfb57f747" (the annotated tag object, not the commitd2f0964). After: the corrected report, the chart, and afront-page table saying exactly which differences exceed measurement drift and which do not.
Related discussion
none — documentation and evidence only. No source, flag, default or interface changed.
Approach
Validation first, then publication.
battery/*.jsonreproduces all eight legs and both metrics;guidellm/*.jsongives the TTFT medians the report claims (739 / 2,622 / 6,688 / 9,641 ms against stock949 / 3,163 / 7,500 / 9,659 ms). Three defects fell out (below).
from paths (
conv_route: "on"in both, which stock ignores;thinking_policy: "on"in both) and the twodata/expert-profile-mine.binfiles have the same SHA-256. The measured fork tree isv0.2.13+ 2 commits; thethree commits after it on
port/0.2.13touchbench/only.10.8–22.1 % sooner to first token at 1–4 streams, equal at 8. The README states the mechanism (single-shot
traffic served with the prompt cache off), the drift (2–4 % in this rig), that errors were 0 in both arms, and that
conversational traffic gets no such win — so a reader cannot mistake this for an engine speed-up.
dirigo-results/0.2.13-port/holds the report, the chart, the raw JSON,the harnesses and
scripts/make-chart.py, which recomputes every figure in the chart from that folder.Corrections made to the original report (also listed in it): the
erroredcolumn was the successful count; thetok/scolumn was a mean of per-iteration samples, not throughput; the stock ref named the tag object instead of thecommit. Dropped: "12,288 of 12,288 slots pre-filled, 0 experts over PCIe", which no shipped file supports.
Validation
Passed:
BENCH=. python3 scripts/make-chart.pyrecomputes the chart fromdirigo-results/0.2.13-port/'s ownJSON; the printed deltas match the README table and the chart labels (−22.1 / −17.1 / −10.8 / −0.2 %; parity worst
1.14 %).
(
v0.2.13annotated objectfb57f747→ commitd2f0964);git diff v0.2.13 3512b4ccis 38 files, +5,062/−264,as stated.
#what-it-is-worth-…anchor and thedirigo-results/0.2.13-port/links resolve.through ImageMagick and was reviewed visually for label collisions.
suites/confirm 1,537OKand 1,512 with one failure.Failed:
none observed.
Not run:
arithmetic was redone. What stays unverified is whether the gap reproduces on a second session — the report says so
in its limits.
Limits
not re-measured here.
so rather than counting it as a win.
like stock; that is stated on the front page.