Repository navigation
[Docs] Put the fork-vs-stock comparison in "Why it is fast" - #7
Merged
Merged
Conversation
The section was named for the question a reader brings - why is this fast - and answered it with the engine's design and a table of what upstream's past releases gained (0.1.13, 0.2.0, 0.2.4, 0.2.7). Past versions are not what a reader comparing this fork to the current release needs, and the actual comparison sat 180 lines below, under "What this fork changes". Now the section runs: the four engine decisions, one absolute-speed line for this card, then "Against stock v0.2.13, on the same card" with the short version, the chart, the two-way TTFT table, the throughput sentence, why the gap exists, the short-answer and suite results, what the numbers do not claim, and the Measured-on block. The release-history table is gone, replaced by a link to the history table in docs/DETAILS.md, so the page stops re-litigating other people's older releases. The later section keeps the list of changes and now points back up for the numbers, so there is one copy of the claim and one chart. Numbers unchanged; the chart is referenced once.
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
## Why it is fastanswered a different question than its heading asks. It explained the engine's design and thenspent a six-row table on what upstream's past releases gained (0.1.13, 0.2.0, 0.2.4, 0.2.7) — a reader comparing
this fork against the current release learns nothing from those rows. The actual comparison lived 180 lines further
down, under
What this fork changes.After: the section states the fork's gain against stock v0.2.13 directly, with the chart and the table, right
after the four engine decisions that explain why the engine is fast at all. The past-release table is replaced by a
link to the history table in
docs/DETAILS.md.Related discussion
none — documentation only.
Approach
absolute speed for this card (1,404 tok/s prompt at 4K, 80.8 tok/s decode), then the comparison as a subsection.
### Against stock v0.2.13, on the same cardcarries the claim and its evidence: the short-version paragraph,the chart, the TTFT table with absolute and reciprocal columns, one sentence for work finished in the window, the
contrast between the two servers at a fresh prompt, the 24-token extraction and the green suite, three sentences
on what the numbers do not claim, and the Measured-on block.
What this fork changeskeeps the list of levers and a three-line pointer back up,instead of a second, longer copy of the same tables.
docs/DETAILS.md; itis not repeated here."
Validation
Passed:
dirigo-results/0.2.13-port/: 739/2,622/6,688/9,641 ms against949/3,163/7,500/9,659; absolute gaps 210/541/812/18 ms; reciprocals 28.4/20.6/12.1 %; window work 22/22/22/23
against 21/20/19/21 and 62.6/62.6/62.6/65.4 against 59.7/56.9/54.0/59.7.
#anchorresolves against its heading (including the new#against-stock-v0213-on-the-same-card); no broken relative links. The chart is referenced exactly once.Failed:
none observed.
Not run:
Limits