github-starred/cs249r_book
2
0
Fork 0
mirror of https://github.com/harvard-edge/cs249r_book.git synced 2026-05-08 09:57:21 -05:00
Code Issues 81 Packages Projects Releases 24 Wiki Activity

[PR #1068] [MERGED] Repository Restructuring: Prepare for TinyTorch Integration #8872

New Issue
Closed
opened 2026-05-03 01:02:46 -05:00 by GiteaMirror · 0 comments
GiteaMirror commented 2026-05-03 01:02:46 -05:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1068
Author: @profvjreddi
Created: 12/5/2025
Status: ✅ Merged
Merged: 12/5/2025
Merged by: @profvjreddi

Base: dev ← Head: feature/repo-restructure

📝 Commits (6)

  • 5d2cc55 Restructure: Move book content to book/ subdirectory
  • 2da1f5a Update pre-commit hooks for book/ subdirectory
  • 25ff7f3 Update .gitignore for book/ subdirectory structure
  • 17d5448 Update GitHub workflows for book/ subdirectory
  • c1de0d6 Update CLI config to support book/ subdirectory
  • f8324a6 Create new root and book READMEs for dual structure

📊 Changes

100 files changed (+0 additions, -0 deletions)

View changed files

📝 .github/workflows/build-linux-container.yml (+7 -7)
📝 .github/workflows/build-windows-container.yml (+7 -7)
📝 .github/workflows/container-health-check.yml (+2 -2)
📝 .github/workflows/deploy-preview.yml (+2 -2)
📝 .github/workflows/link-check.yml (+5 -5)
📝 .github/workflows/publish-live.yml (+5 -5)
📝 .github/workflows/quarto-build-baremetal.yml (+19 -19)
📝 .github/workflows/quarto-build-container.yml (+9 -9)
📝 .github/workflows/update-contributors.yml (+3 -3)
📝 .gitignore (+17 -17)
📝 .pre-commit-config.yaml (+40 -40)
📝 README.md (+50 -340)
➕ book/README.md (+56 -0)
📝 book/binder (+0 -0)
📝 book/cli/README.md (+0 -0)
📝 book/cli/__init__.py (+0 -0)
📝 book/cli/commands/__init__.py (+0 -0)
📝 book/cli/commands/build.py (+0 -0)
📝 book/cli/commands/clean.py (+0 -0)
📝 book/cli/commands/doctor.py (+0 -0)

...and 80 more files

📄 Description

Summary

This PR restructures the repository to support both MLSysBook and TinyTorch projects in a unified codebase:

  • Moves all book content to book/ subdirectory
  • Updates all paths in workflows, configs, and CLI
  • Preserves full Git history for all moved files (2,263 files)
  • Prepares structure for TinyTorch addition

Changes

Directory Structure

  • quarto/ → book/quarto/
  • cli/ → book/cli/
  • docker/ → book/docker/
  • socratiQ/ → book/socratiQ/
  • tools/ → book/tools/
  • scripts/ → book/scripts/
  • config/ → book/config/
  • docs/ → book/docs/
  • binder → book/binder

Configuration Updates (6 commits)

  1. ✅ Moved all content to book/ (git history preserved - 100% renames)
  2. ✅ Updated pre-commit hooks
  3. ✅ Updated .gitignore
  4. ✅ Updated all GitHub workflows (11 files)
  5. ✅ Updated CLI configuration
  6. ✅ Created new root and book READMEs

Testing

  • Binder CLI works from new location
  • Binder status shows correct paths
  • Git history verified (git log --follow works)
  • All configuration files updated
  • Working tree clean

Rollback Plan

If issues arise, rollback branch exists:

git checkout dev
git reset --hard backup-pre-restructure-20251205-135059

Next Steps

After merge:

  1. Monitor CI/CD workflows on dev branch
  2. Verify preview deployment works
  3. Test PDF/EPUB builds
  4. Add TinyTorch content to tinytorch/ directory

Deployment Impact

  • Book deployment paths updated in workflows
  • Netlify redirects unchanged (still work)
  • GitHub Pages deployment will use new structure
  • All existing URLs will continue to work

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/harvard-edge/cs249r_book/pull/1068 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 12/5/2025 **Status:** ✅ Merged **Merged:** 12/5/2025 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `feature/repo-restructure` --- ### 📝 Commits (6) - [`5d2cc55`](https://github.com/harvard-edge/cs249r_book/commit/5d2cc5592246b07ec1e9817b90d5fc7e416bd532) Restructure: Move book content to book/ subdirectory - [`2da1f5a`](https://github.com/harvard-edge/cs249r_book/commit/2da1f5ad307517083536aa7d97e34fdc8e01a1a5) Update pre-commit hooks for book/ subdirectory - [`25ff7f3`](https://github.com/harvard-edge/cs249r_book/commit/25ff7f333232f7aadfb8d1728c8c31dcbeee4268) Update .gitignore for book/ subdirectory structure - [`17d5448`](https://github.com/harvard-edge/cs249r_book/commit/17d544845a84df1b34fcc1e3f7e1198ed7a17299) Update GitHub workflows for book/ subdirectory - [`c1de0d6`](https://github.com/harvard-edge/cs249r_book/commit/c1de0d62b709a22c1acdc224d753c94d64888c23) Update CLI config to support book/ subdirectory - [`f8324a6`](https://github.com/harvard-edge/cs249r_book/commit/f8324a6be0c974c08b23cbf4900c43283d596ed7) Create new root and book READMEs for dual structure ### 📊 Changes **100 files changed** (+0 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/build-linux-container.yml` (+7 -7) 📝 `.github/workflows/build-windows-container.yml` (+7 -7) 📝 `.github/workflows/container-health-check.yml` (+2 -2) 📝 `.github/workflows/deploy-preview.yml` (+2 -2) 📝 `.github/workflows/link-check.yml` (+5 -5) 📝 `.github/workflows/publish-live.yml` (+5 -5) 📝 `.github/workflows/quarto-build-baremetal.yml` (+19 -19) 📝 `.github/workflows/quarto-build-container.yml` (+9 -9) 📝 `.github/workflows/update-contributors.yml` (+3 -3) 📝 `.gitignore` (+17 -17) 📝 `.pre-commit-config.yaml` (+40 -40) 📝 `README.md` (+50 -340) ➕ `book/README.md` (+56 -0) 📝 `book/binder` (+0 -0) 📝 `book/cli/README.md` (+0 -0) 📝 `book/cli/__init__.py` (+0 -0) 📝 `book/cli/commands/__init__.py` (+0 -0) 📝 `book/cli/commands/build.py` (+0 -0) 📝 `book/cli/commands/clean.py` (+0 -0) 📝 `book/cli/commands/doctor.py` (+0 -0) _...and 80 more files_ </details> ### 📄 Description ## Summary This PR restructures the repository to support both MLSysBook and TinyTorch projects in a unified codebase: - Moves all book content to `book/` subdirectory - Updates all paths in workflows, configs, and CLI - Preserves full Git history for all moved files (2,263 files) - Prepares structure for TinyTorch addition ## Changes ### Directory Structure - `quarto/` → `book/quarto/` - `cli/` → `book/cli/` - `docker/` → `book/docker/` - `socratiQ/` → `book/socratiQ/` - `tools/` → `book/tools/` - `scripts/` → `book/scripts/` - `config/` → `book/config/` - `docs/` → `book/docs/` - `binder` → `book/binder` ### Configuration Updates (6 commits) 1. ✅ Moved all content to book/ (git history preserved - 100% renames) 2. ✅ Updated pre-commit hooks 3. ✅ Updated .gitignore 4. ✅ Updated all GitHub workflows (11 files) 5. ✅ Updated CLI configuration 6. ✅ Created new root and book READMEs ## Testing - [x] Binder CLI works from new location - [x] Binder status shows correct paths - [x] Git history verified (git log --follow works) - [x] All configuration files updated - [x] Working tree clean ## Rollback Plan If issues arise, rollback branch exists: ```bash git checkout dev git reset --hard backup-pre-restructure-20251205-135059 ``` ## Next Steps After merge: 1. Monitor CI/CD workflows on dev branch 2. Verify preview deployment works 3. Test PDF/EPUB builds 4. Add TinyTorch content to `tinytorch/` directory ## Deployment Impact - Book deployment paths updated in workflows - Netlify redirects unchanged (still work) - GitHub Pages deployment will use new structure - All existing URLs will continue to work --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-03 01:02:47 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area: book
area: collabs
area: kits
area: socratiq
area: tinytorch
area: tools
area: website
bug
dependencies
format: epub
format: pdf
link-health
link-rot
priority-high
pull-request
Mirrored from GitHub Pull Request
 staffml
type: bug
type: citation
type: code
type: errata
type: improvement
type: new
type: question
vault-sli
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
GiteaMirror ninjasurge
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#8872
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?