Skip to content

Fix chapter titles and filenames to match TOC order#50

Merged
tyfolino merged 1 commit into
ProjectPythia:mainfrom
tyfolino:fix-chapter-titles
Mar 13, 2026
Merged

Fix chapter titles and filenames to match TOC order#50
tyfolino merged 1 commit into
ProjectPythia:mainfrom
tyfolino:fix-chapter-titles

Conversation

@tyfolino
Copy link
Copy Markdown
Collaborator

Summary

  • Renames bnf-mrms-qpe-hourly.ipynbch4_bnf-mrms-qpe-hourly.ipynb for consistent naming
  • Renames ch4_realtimeData.ipynbch5_realtimeData.ipynb to match its actual chapter number
  • Adds missing "Chapter 4:" prefix to the BNF notebook title
  • Updates myst.yml TOC to use new filenames
  • Verified all cross-references between notebooks are correct

Closes #41

Test plan

  • Check that the build passes
  • Verify chapter URLs in the built site match the new filenames

🤖 Generated with Claude Code

- Rename bnf-mrms-qpe-hourly.ipynb -> ch4_bnf-mrms-qpe-hourly.ipynb
- Rename ch4_realtimeData.ipynb -> ch5_realtimeData.ipynb
- Add "Chapter 4:" prefix to BNF notebook title (was missing)
- Update myst.yml TOC to use new filenames
- Chapter 5 (real-time) title was already correct

Closes ProjectPythia#41

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 6a2d03b
✅ Deployment Preview URL: In Progress

@tyfolino
Copy link
Copy Markdown
Collaborator Author

The build check is timing out due to a pre-existing issue in Chapter 5 (ch5_realtimeData.ipynb), which fetches live MRMS data from AWS and renders a full CONUS interactive plot (~24.5M data points via hvplot/Bokeh). This is non-deterministic in CI — it works some runs and hangs others depending on network conditions and runner load. This is unrelated to the changes in this PR. A separate branch will address the Chapter 5 memory/execution issue.

@tyfolino tyfolino merged commit 6529cc6 into ProjectPythia:main Mar 13, 2026
1 of 2 checks passed
@tyfolino tyfolino deleted the fix-chapter-titles branch March 13, 2026 20:09
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.

Change chapter titles

1 participant