Repository navigation
Run the automata chapter's Racket examples at render time - #146
Conversation
pairs-table, step, run, accepts? and the numeral checker were plain fences, and the runs and #t and #f results were typed in the prose (#131). They are now {racket} chunks, the typed results are chunk output, and the solution to exr-racket-step runs the lookup one step at a time, as its reasoning does. Part of #131. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FWgYrEUFVHj2qKjKncHg77
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…d-2wtrwq-automata
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
🤖 Antigravity agent reportCross-harness review (Gemini 3.8 Flash (Medium) via Antigravity
|
One chunk held the two runs and the three acceptance tests, introduced by the sentence about acceptance alone, and it tested "ba" before the empty string while the sentence named them the other way round. Split it so each sentence introduces its own calls, and test the strings in the order the sentence names them. Raised by the cross-harness read of #146. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FWgYrEUFVHj2qKjKncHg77
|
Claude finished review — View run The live fetch changes one thing in my review: #139 has already merged, so the "contingent on #139" condition is met.
The review found nothing blocking at VerdictReady for merge. #139 is merged, so nothing blocks #146. Structured Review Data (JSON){
"schema_version": "1.1",
"reviewer": "claude",
"commit_sha": "9411a494c17a49c72d7746db34204f3867dd937e",
"verdict": "CLEAN",
"findings": [],
"detailed_assessment": "In _racket-run_exm.qmd the run calls and accepts? calls are now separate chunks matching the prose order, with unchanged results and code-fold options, so no defect exists.",
"holistic_assessment": "The change is scoped to one subfile, preserves the engine's replay dependencies for later chunks, adds no new math or dependencies, and the stacking dependency on PR 139 is now merged."
}Reviewed commit: 9411a49 |
|
Reply to the Flash read at b468011, now at 9411a49: Fixed. Each sentence now introduces its own chunk: the two
Posted by Claude Code (AI agent) --- not written by a human. Generated by Claude Code |
🤖 Antigravity agent reportCross-harness re-read (Gemini 3.8 Flash (Medium) via Antigravity
|
Built to branch |
Requested by Ezra · project thread
Important
Stacked on #139, which adds the racket engine. Merge #139 first. Independent of the other chapter PRs.
Part of #131.
What changes
In
chapters/automata.qmd, the "Automata in Racket" section:_subfiles/_racket-setup.qmd;pairs-tableandstep,runandaccepts?,numeral-table,classifyandnumeral?are{racket}chunks;exm-racket-runruns the two runs and the threeaccepts?calls instead of typing their results;exm-racket-numeralruns(map numeral? ...)on its eight strings instead of typing'(#t #t #f #f #f #f #f #f);exr-racket-stepkeeps its reasoning and ends with a chunk that does the lookup one step at a time.Testing
Rendered the chapter to HTML locally. The outputs match the values the prose stated before:
'q0,'(q0 q1 q0 q1 q0),'(q0 q1 q2 q2),#t,#f,#tand'(#t #t #f #f #f #f #f #f).Generated by Claude Code