Repository navigation
[Docs] Rename to "Why this fork is faster than stock Strata" and demonstrate it - #11
Merged
Merged
Conversation
The section is the fork's case, so it should be named for it and built like an argument rather than a description. It is now "Why this fork is faster than stock Strata", and it demonstrates in this order: 1. what is NOT faster - both trees build engine 0.1.59 and the 8-leg engine battery lands within 1.1 %, so the reader knows the claim rests on the server before seeing a single gain; 2. the claim and its evidence - 739 vs 949 ms at one stream, the chart, the table with absolute and reciprocal columns, the work finished in the same window; 3. where it comes from - the two servers' behaviour at a fresh prompt; 4. a control - the same router measured shadow (934 ms, stock's number) against on (727 ms) on one build, so the switch rather than the build is shown to move the first token; 5. how to reproduce it - the harness, the per-level JSON, make-chart.py; 6. what it does not show - engine parity, conversations, 8 streams, the one-request granularity at 1 stream; 7. what both versions share underneath - one paragraph, since the engine decisions are stock's and belong to the shared explanation. The drafting table moved to "What this fork changes", where a list of differences belongs, under "the same drafter, more control"; the extraction and suite results moved with it as "two measured differences that are not speed". The old "Why it is fast" bullets are gone: the three engine decisions now appear once, as the closing paragraph of the new section.
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 fastdescribed the engine. The page's job in that slot is to make the fork's case against the release itwas ported to, so the section is renamed "Why this fork is faster than stock Strata" and rebuilt as an argument:
what is not faster, what is, by how much, why, the control that isolates the cause, how to reproduce it, and what it
does not show.
Related discussion
none — documentation only.
Approach
reader learns the claim rests on the server before seeing any gain, which is what makes the rest credible.
conv_route: shadow(classify and log, never switch) lands on stock'snumber, 934 ms at one stream, and
conv_route: onlands on 727 ms. Same build, same card, same workload — theswitch, not the build, moves the first token. Cited to
DIRIGO.md, which carries both arms.make-chart.py, which recomputes every figure onthe page from those files.
stock's and are explained in
docs/HOW_IT_WORKS.md.What this fork changes, under "the same drafter, more control"; the extraction andsuite results moved there too, as "two measured differences that are not speed".
Validation
Passed:
DIRIGO.mdcarries for the 0.2.12 pair (934 / 3 004 / 7 583 / 9 807 ms shadowagainst 727 / 2 640 / 6 114 / 9 758 ms on); the 934 vs 727 ms quoted is the one-stream pair, and the section says it
is a different build from the 0.2.13 comparison.
dirigo-results/0.2.13-port/.#against-stock-v0213-on-the-same-cardis gone from the page (0 occurrences); the newheading anchor resolves from
Start hereand fromWhat this fork changes; no broken relative links.lookup=0|1cell stays a single cell.Failed:
none observed.
Not run:
Limits
and is there to show the mechanism, not to add to the percentages.