Skip to content

Remove daily survers from outro pages#503

Merged
iamzoltan merged 31 commits intostagingfrom
remove-daily-surveys
Mar 30, 2026
Merged

Remove daily survers from outro pages#503
iamzoltan merged 31 commits intostagingfrom
remove-daily-surveys

Conversation

@iamzoltan
Copy link
Copy Markdown
Contributor

No description provided.

courtneydean33 and others added 30 commits March 18, 2026 11:40
removed 2025 schedule update language
fix: convert JB1 sections→children in TOC, add book/ symlinks in work…
fix: add skip-execution tag to NotImplementedError stub cells for JB2…
…rip error divs (matches JB1 allow_errors behaviour)
Update generate_book_v2 and error parsing v2
Update generate_book_v2 and error parsing v2 to fix bugs
- generate_book_v2.py: add &autoplay=0 to Bilibili iframe URL so the
  player does not autoplay when the tab becomes visible
- parse_html_for_errors_v2.py: rewrite path discovery to walk
  book/_build/html/** instead of reconstructing input-derived paths;
  JB2/MyST flattens output to slug/index.html so the old
  materials.yml-based paths never matched, leaving all stub errors
  visible in the built site
Print count of index.html files found (or fail fast with clear error
if the HTML root directory doesn't exist) so we can confirm the walk
is finding the right path in CI.
- generate_book_v2.py: move logo/favicon/logo_text into site.options
  (MyST book-theme requires these as options, not top-level site keys;
  favicon was rendering as /favicon.ico instead of the ai-logo.png)
- schedule_intro.md, tech_intro.md: add short_title: Overview frontmatter
  so sidebar shows 'Overview' instead of 'Schedule'/'Technical Help'
  (which duplicated the parent section folder label)
generate_book_v2.py:
- hide_title_block: true — MyST keeps the notebook H1 in the cell body
  for .ipynb files (unlike .md), so the auto-rendered myst-fm-block-title
  appeared as a second identical heading above the article; suppress it
- edit_url: null — auto-computed edit URL gets a spurious book/ prefix
  from the book/tutorials symlink, producing a 404 GitHub path

publish-book-v2.yml:
- Bump cache key from jb2-exec- to jb2-exec-v2- to bust the execution
  cache; prior cached executions predate the raises-exception tagging
  and error-stripping fixes, leaving visible NotImplementedError tracebacks
  on pages that were cache-hit rather than re-executed (W1D2 and others)
MyST renders pages as a React SPA — the browser ignores the static HTML
and re-renders everything from window.__remixContext, which is populated
from per-page .json files in book/_build/html/.  Stripping the static
index.html had no visible effect.

Rewrite parse_html_for_errors_v2.py to walk every .json page data file,
find 'output' nodes in the mdast tree whose jupyter_data.ename matches
NotImplementedError or NameError, remove them from their parent 'outputs'
node, and drop the 'outputs' node entirely if it becomes empty.
The script no longer parses HTML — it strips error outputs from the
JSON build artifacts. Update the filename and all references.
@iamzoltan iamzoltan changed the base branch from main to staging March 30, 2026 00:14
Copy link
Copy Markdown
Contributor Author

@iamzoltan iamzoltan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@iamzoltan iamzoltan merged commit 9cba943 into staging Mar 30, 2026
0 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants