Repository navigation
Comprensive AI review #106
Description
Generated by Claude Opus 4.8 and a workflow. (Fun fact if we go on a token-based plan: this would have cost about $25 in tokens. Though if I was on a token-based plan, I would have triggered the edits with a simpler model, which would have helped (and Sonet might have been fine for the review, too).)
Full report:
🤖 AI Summary 🤖
Course material review
A multi-agent review of all course material — one reviewer per chapter (45
chapters) across four dimensions (technical accuracy/currency, pedagogy, prose,
code correctness), with adversarial verification of every accuracy/code
finding.
189 findings → 171 confirmed (18 rejected by verification). All 171 are
applied in #107.
- By category: 🎯 accuracy 32 · 💻 code 31 · 📚 pedagogy 13 · ✍️ prose 95
- By severity: 🔴 high 8 · 🟠 medium 56 · ⚪ low 107
- 42 of 45 chapters had at least one finding
High-severity items (all fixed in #107)
week13_misc/shell.md— empty stub (in thetocbut no content) → written as a full chapterweek02_testing/pytest.md— wrong@pytest.fixture("platform", …)syntax (×2) →params=…week07_design/designpatt.md— dataclass field missing type annotationweek10_binding/03-pybind/03b-pybind.ipynb—includemissing#in a C++ exampleweek11_omp/threading.md—worker.thread.Threadtypo →threading.Thread- Malformed sentences in
week02_testing/testing.mdandweek04_package/packaging.md
Recurring themes
- Code that didn't run: infinite recursion (
MyList.append), missing@dataclasses.dataclassdecorators, missing imports/class defs, undefined vars in the Matlab/JSexists()examples - Conceptual errors:
asyncio.to_threaddescribed as running async (it runs sync) code; a C++ move-assignment that recursed instead of stealing the pointer;__neq__→__ne__;parameter()→perimeter() - Currency: Python 3.9 (EOL) → recommend 3.12+; git output
master→main - Dead links:
github.com/pypy/pipx(typo→pypa),pypa.github.io/pipx→pipx.pypa.io, a 404 pipx noxfile link, and two unreachable references inpytest.md
Per-chapter breakdown
| Chapter | Total | 🎯 | 💻 | 📚 | ✍️ | 🔴 |
|---|---|---|---|---|---|---|
week01_intro/python_setup.md |
12 | 1 | 0 | 0 | 11 | |
week02_testing/pytest.md |
10 | 0 | 8 | 0 | 2 | 2 |
week08_static_typing/typing.md |
10 | 1 | 4 | 0 | 5 | |
week09_compiled/rust_example/rust.md |
9 | 0 | 1 | 0 | 8 | |
week06_oop/oodesign.md |
8 | 4 | 1 | 0 | 3 | |
week06_oop/introoo.md |
7 | 0 | 4 | 0 | 3 | |
week07_design/designpatt.md |
7 | 0 | 3 | 0 | 4 | 1 |
week09_compiled/rust_example/compiled.md |
6 | 1 | 0 | 0 | 5 | |
week09_compiled/debugging.md |
6 | 1 | 0 | 1 | 4 | |
week10_binding/02-cpython/02-cpython.ipynb |
6 | 2 | 0 | 0 | 4 | |
week11_omp/threading.md |
6 | 0 | 3 | 1 | 2 | 1 |
week05_ci/docs.md |
5 | 0 | 1 | 0 | 4 | |
week07_design/functional.md |
5 | 1 | 1 | 0 | 3 | |
week04_package/using_packages.md |
4 | 0 | 0 | 0 | 4 | |
week05_ci/ci.md |
4 | 1 | 0 | 0 | 3 | |
week09_compiled/Compiled_Languages.ipynb |
4 | 1 | 0 | 0 | 3 | |
week11_omp/concepts.md |
4 | 0 | 0 | 0 | 4 | |
| …26 more chapters with 1–3 findings each |
Full per-finding detail (location, issue, fix, verifier reasoning) was captured
in the review artifact behind #107.
Activity