docs: clear, grouped prerequisites in the handbook and both READMEs - #57
Merged
Conversation
project-delphi
force-pushed
the
feat/issue-51-prerequisites
branch
from
September 3, 2026 17:41
1c7a4b0 to
5ba92c9
Compare
) The prerequisites were one flat list that mixed what you must do beforehand with what you need on the day, and buried the required pre-work in the last bullet. Group them instead — do before the session / recommended / have ready on the day — and give every item its link: the book chapter, the pre-work repo, notebook 00 in Colab, a Google account, Discord, Kahoot. Adds 3Blue1Brown's "Essence of linear algebra" (16 videos, ~3 h) as a recommended prerequisite, in _variables.yml as workshop.prereq_video_url so both landing pages read it from the one source. Discord stays prose-only: the invite comes from the facilitator and there is no public link to point at. The same four surfaces carry it: index.qmd, es/index.qmd, both halves of README.md and the handbook, which hard-codes its links as it already does and speaks in Blocks rather than section numbers. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0125vs3ARnz2pLp8D5vQ6iKJ
project-delphi
force-pushed
the
feat/issue-51-prerequisites
branch
from
September 3, 2026 17:43
5ba92c9 to
ec99c1f
Compare
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.
Closes #51.
What changed
The prerequisites were a single flat list that mixed do this beforehand with have this on the day, and left the required pre-work as the last bullet. They are now three labelled groups, every item linked:
linear-algebra-deep-learning(still marked required), and running notebook 00 in Colab.workshop.prereq_video_urlin_variables.yml, so both landing pages read it from the one source.Four surfaces carry the same list:
index.qmd,es/index.qmd, both halves ofREADME.md, and the handbook (which hard-codes its links, as it already did, and speaks in Blocks rather than section numbers).Discord is deliberately prose-only — "the invite comes from the facilitator". There is no public invite in the repo, and a placeholder URL is worse than none.
On the diff size
Two things beyond the prose:
docs/notebooks/had gone stale againstnotebooks/again — all twelve. Re-rendering syncs them, which is most of the line count. This is exactly the failure CLAUDE.md documents under Publishing: the regenerate gate never looks indocs/, andcompare_render.pywalks only*.html, so nothing catches it.compare_render.pynormalizes hashed asset names, so the publish gate is unaffected.Verified
gen_tables.pyandgen_notebooks.pyrerun with no tree drift.quarto renderwith the pinned 1.6.40.check_links.py: all eight failable checks pass (688 internal links, both landing pages, the 195-minute clock). Check 9 still prints the pre-existing Kahoot TODO, which is not a failure.🤖 Generated with Claude Code
https://claude.ai/code/session_0125vs3ARnz2pLp8D5vQ6iKJ