[PR #1312] [MERGED] fix(mlsysim): address reviewer feedback + improve landing page #2470

New Issue

GiteaMirror · 2026-04-11T08:29:08-05:00

GiteaMirror commented

2026-04-11 08:29:08 -05:00

📋 Pull Request Information

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

Base: dev ← Head: feat/mlsysim

📝 Commits (1)

e0d64da fix(mlsysim): address reviewer feedback + improve landing page

📊 Changes

3 files changed (+131 additions, -136 deletions)

View changed files

📝 mlsysim/docs/config/_quarto-html.yml (+10 -10)
📝 mlsysim/docs/index.qmd (+108 -113)
📝 mlsysim/paper/paper.tex (+13 -13)

📄 Description

Summary

Paper (paper.tex)

Define "bind/binding" at first use with footnote linking to LP and Roofline analogies
Clarify Table 2 caption — define Scope, Dist., etc.; add footnote defining "single accelerator" vs "node"
Rename "Progressive Lowering" to "Layered Input Stack" throughout (abstract, intro, heading, figure caption, conclusion)

Website (index.qmd + config)

Remove stats bar that hits visitors with decontextualized numbers
Move interactive carousel right under hero tagline
Reorder carousel for narrative flow
Fix broken tutorial links on landing page
Fix sidebar: tutorials/11_full_stack_audit → 12_full_stack_audit
Comment out Interactive Apps sidebar section (files pending Marimo integration)

Test plan

Build paper: cd mlsysim/paper && make
Build site: cd mlsysim/docs && quarto preview
All tutorial links resolve to existing files

_{🔄 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/1312 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 4/10/2026 **Status:** ✅ Merged **Merged:** 4/10/2026 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `feat/mlsysim` --- ### 📝 Commits (1) - [`e0d64da`](https://github.com/harvard-edge/cs249r_book/commit/e0d64da7f94e78452c726dd654411342a96c95b1) fix(mlsysim): address reviewer feedback + improve landing page ### 📊 Changes **3 files changed** (+131 additions, -136 deletions) <details> <summary>View changed files</summary> 📝 `mlsysim/docs/config/_quarto-html.yml` (+10 -10) 📝 `mlsysim/docs/index.qmd` (+108 -113) 📝 `mlsysim/paper/paper.tex` (+13 -13) </details> ### 📄 Description ## Summary ### Paper (paper.tex) - Define "bind/binding" at first use with footnote linking to LP and Roofline analogies - Clarify Table 2 caption — define Scope, Dist., etc.; add footnote defining "single accelerator" vs "node" - Rename "Progressive Lowering" to "Layered Input Stack" throughout (abstract, intro, heading, figure caption, conclusion) ### Website (index.qmd + config) - Remove stats bar that hits visitors with decontextualized numbers - Move interactive carousel right under hero tagline - Reorder carousel for narrative flow - Fix broken tutorial links on landing page - Fix sidebar: `tutorials/11_full_stack_audit` → `12_full_stack_audit` - Comment out Interactive Apps sidebar section (files pending Marimo integration) ## Test plan - [ ] Build paper: `cd mlsysim/paper && make` - [ ] Build site: `cd mlsysim/docs && quarto preview` - [ ] All tutorial links resolve to existing files --- <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-04-11 08:29:09 -05:00

GiteaMirror closed this issue

2026-04-11 08:29:09 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#2470