github-starred/cs249r_book
2
0
Fork 0
mirror of https://github.com/harvard-edge/cs249r_book.git synced 2026-05-21 13:31:55 -05:00
Code Issues 114 Packages Projects Releases 24 Wiki Activity

[PR #1606] [MERGED] fix(kits/raspi/llm): wrap context-token array to prevent page overflow #9217

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

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/1606
Author: @farhan523
Created: 4/30/2026
Status: ✅ Merged
Merged: 4/30/2026
Merged by: @profvjreddi

Base: dev ← Head: fix/kits-llm-token-array-overflow

📝 Commits (1)

  • 86c48d0 fix(kits/raspi/llm): wrap context-token array to prevent page overflow

📊 Changes

1 file changed (+6 additions, -1 deletions)

View changed files

📝 kits/contents/raspi/llm/llm.qmd (+6 -1)

📄 Description

Summary

  • On kits/contents/raspi/llm/llm.qmd, the 26-token context array was authored as inline code (single backtick) under a sub-bullet. Inline code does not wrap, so the array stretched the page wider than the viewport and forced horizontal scroll on the entire page (see Before).
  • Moved the array into a fenced ```text code block with the values broken across three lines. Fenced code blocks scroll within their own width, so the page itself no longer overflows.
  • Content of the array is unchanged; only formatting.

Before

Screenshot 2026-04-30 101044

After

Screenshot 2026-04-30 101116

Test plan

  • cd kits && quarto preview → open /contents/raspi/llm/llm.html.
  • Scroll to the section after "As we can see, several pieces of information are generated".
  • Confirm the page itself has no horizontal scrollbar at standard widths (e.g. 1280px, 1024px).
  • Confirm the token array is rendered inside a code block and scrolls within its own bounds if the viewport is narrowed.
  • Verify other code blocks on the page still render correctly.

🔄 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/1606 **Author:** [@farhan523](https://github.com/farhan523) **Created:** 4/30/2026 **Status:** ✅ Merged **Merged:** 4/30/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `fix/kits-llm-token-array-overflow` --- ### 📝 Commits (1) - [`86c48d0`](https://github.com/harvard-edge/cs249r_book/commit/86c48d019f2d349a4594d40e21d933f2804d4f8d) fix(kits/raspi/llm): wrap context-token array to prevent page overflow ### 📊 Changes **1 file changed** (+6 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `kits/contents/raspi/llm/llm.qmd` (+6 -1) </details> ### 📄 Description ## Summary - On `kits/contents/raspi/llm/llm.qmd`, the 26-token `context` array was authored as **inline code** (single backtick) under a sub-bullet. Inline code does not wrap, so the array stretched the page wider than the viewport and forced horizontal scroll on the entire page (see *Before*). - Moved the array into a fenced ```` ```text ```` code block with the values broken across three lines. Fenced code blocks scroll within their own width, so the page itself no longer overflows. - Content of the array is unchanged; only formatting. ## Before <img width="1769" height="318" alt="Screenshot 2026-04-30 101044" src="https://github.com/user-attachments/assets/81cdfcb5-d020-47ff-8599-1b23460d5b07" /> ## After <img width="1638" height="387" alt="Screenshot 2026-04-30 101116" src="https://github.com/user-attachments/assets/45aad65e-f245-4735-ad1e-1bbc286195cc" /> ## Test plan - [ ] `cd kits && quarto preview` → open `/contents/raspi/llm/llm.html`. - [ ] Scroll to the section after "*As we can see, several pieces of information are generated*". - [ ] Confirm the page itself has no horizontal scrollbar at standard widths (e.g. 1280px, 1024px). - [ ] Confirm the token array is rendered inside a code block and scrolls within its own bounds if the viewport is narrowed. - [ ] Verify other code blocks on the page still render correctly. --- <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:29:09 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area: book
area: collabs
area: kits
area: labs
area: socratiq
area: tinytorch
area: tools
area: website
bug
dependencies
format: epub
format: pdf
javascript
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#9217
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?