Repository navigation
book: Handwired (handwired) - #155
Merged
Merged
Conversation
|
|
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
⛔ Files ignored due to path filters: 14
📒 Files selected for processing: 14
✨ Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Five narrated chapters on chenglou/handwired @ 4ea7fcb2 (MIT): an if-else as two ramps and a bit, the toggle template as a fixed program, the host loop, loops as counts/trees/tables, and exactness, gradients and the three demos. - Scenes: apps/bookshelf/src/viz/books/handwired/chapter-1..5 (+ stories). Visualizations authored by Claude Opus 5.5 from a Claude Sonnet 5.5 plan; research, review and blog prose by Claude Haiku 5.5. - Narration: ElevenLabs default voice at speed 0.94 (manifest voiceSpeed). - sources.json: 78 claims, 175 links pinned to 4ea7fcb2; every quote checked against the clone. Companion blog.md with 10 live viz sections. - Cover: title-only. The gpt-image call failed with insufficient_quota, so no animal.webp ships and the manifest carries no animal field. - generator: opt-in `video.mjs --speed <0.7-1.2>` (ElevenLabs voice_settings.speed), merged onto stored voice settings when readable, speed-only override otherwise; default narration requests are unchanged. Tests: generator/tts-speed.test.mjs. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
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.
Handwired — a new O'RLY? book
Five narrated, animated chapters on chenglou/handwired by Cheng Lou (MIT), pinned at
4ea7fcb2: how a program's if/else, loops and state are written by hand as a fixed program of weighted sums and relu over 32-bit floats.apps/bookshelf/src/viz/books/handwired/chapter-1..5.tsx(+ stories). Book data:public/generated/handwired/.sources.jsonhas 78 claims and 175 links, all pinned to4ea7fcb2. Every repo quote is checked against the clone.insufficient_quota. Noanimal.webpships, and the manifest andlibrary.jsoncarry noanimalfield. Note: the shelf'sresolveAnimalborrows another book's animal (picked by a hash of the slug) for books without one.video.mjs --speed <0.7–1.2>(ElevenLabsvoice_settings.speed). It merges onto the voice's stored settings when they can be read. This key lacksvoices_read, so it falls back to a speed-only override with a warning. Default requests are unchanged. Tests are ingenerator/tts-speed.test.mjs. The book was narrated at 0.94, recorded asvoiceSpeedin the manifest.Models (requested → actual)
--speedpipeline flag (2 passes)Every run went through
generator/run-fable.mjs, and each result'smodelUsagewas checked.Checks run
npm run typecheck✓ ·npm run build -- --skip-nx-cache✓ ·npm run book:sources -- --slug handwired✓ ·ORLY_VERIFY_PORT=5277 node generator/verify-book.mjs --slug handwired✓ (all 5 chapters) ·node generator/blog-viz.mjs --slug handwired✓ ·node --test generator/tts-speed.test.mjs generator/pronunciation.test.mjs generator/narration-timing.test.mjs✓Self-grade (fresh-eyes Haiku reviewer, docs/book-quality.md §4)
>=within 0..16,384).Mean 7.44: FAIL against the bar (every dimension ≥ 8, mean ≥ 8.5). No critical issue. Per the rubric, sync and sound stay at 6.5–7.5 until a person watches and listens, so the self-grade cannot pass without that.
What I could not judge
Known remaining issues
🤖 Generated with Claude Code
Summary by CodeRabbit