Repository navigation
Spec 001 T005 and T006: a no-go, and the figures drawn in a step of their own - #84
Closed
adamjohnwright wants to merge 2 commits into
Closed
adamjohnwright wants to merge 2 commits into
adamjohnwright wants to merge 2 commits into
Conversation
…heir own T005 measured the render service in the book's mix on an 8-core host: at most 1.41 figures a second (concurrency 8), limited by the processors at about 3 core-seconds a figure, against the 7.3 a second SC-003's figure budget needs. The script is ci/measure/throughput.ts. T006 takes a third route, the user's choice: `pdf-export figures` draws every figure the book needs into a folder before the book (SC-010, contracts/figures.md), and `book --figures` reads them, asking no render service. SC-003 is now the book's time from figures already drawn. T046 adds the command; T024, T029, T037 and T039 follow; T005's single-figure measurement moves to T007. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…e folder tied to its release The measuring script now chooses a selection's ancestor as the book does (nearestAncestor, a top-level pathway last), samples the container's CPU without stalling the client, rejects bad numbers, and keeps the mix when a selection has no ancestor. Re-run on all four settings: 1.523 figures a second at best, 3.5-4.6 core-seconds a figure, rising with concurrency. The figures' folder gets a manifest (release; each figure's kind, source and selection): `book` refuses a stale or other release's folder, and `figures` redraws only what changed. `figures` takes --files and runs the book's release-file and character checks before drawing. The book's own run, contracts, research R3/R7/R9/R10, data-model, quickstart, plan and tasks no longer say the book asks the render service. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
Author
|
Replaced by the follow-up PR with this review's findings fixed. |
This was referenced Nov 2, 2026
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.
Spec 001, T005 and T006. Replaces #83, with its review's findings fixed.
T005: a no-go. The render service, in the book's mix (85% reactions, 6% diagrams, 1% illustrations, 8% selections), cold, on a private copy of release 98's graph, on an 8-core host:
SC-003's figure budget needs 7.3 a second; the processors are the limit, and the cost per figure rises with concurrency. What T005 did not run, and why, is in research R7. Script:
ci/measure/throughput.ts.T006: the route taken. The figures are drawn by a step of their own before the book:
pdf-export figureswrites them to a folder with a manifest tying it to its release (newcontracts/figures.md), after running the book's release-file and character checks;book --figuresreads them, refuses a stale folder, and asks no render service. SC-003 is the book's time from figures already drawn; new SC-010 covers the figures' step. New task T046; T024, T029, T031, T037, T039 and T045 follow; contracts, research, data model, quickstart and plan updated to match.From #83's review: the script chose a selection's ancestor differently from the book and stalled its client while sampling CPU; both fixed and all four settings re-run. R7's per-figure cost claims now follow from the table.
🤖 Generated with Claude Code