mirror of
https://github.com/harvard-edge/cs249r_book.git
synced 2026-07-22 15:11:24 -05:00
Closed
opened 2026-07-15 22:41:19 -05:00 by GiteaMirror
·
4 comments
No Branch/Tag Specified
dev
audit/mlperf-edu-readiness
feat/mlperf-edu-precondition
gh-pages
vol1/all-final
main
vol1/appendices-final
vol1/ch16-final
vol1/ch15-final
vol1/ch14-final
vol1/ch13-final
vol1/ch11-final
vol1/ch12-final
vol1/ch10-final
vol1/ch9-final
vol1/ch8-final
vol1/ch7-final
vol1/ch6-final
vol1/ch5-final
vol1/ch4-final
vol1/ch3-final
vol1/ch2-final
vol1/frontmater-final
kai/fixing-profile-setting-and-map
chore/staffml-ci-path
fix/callout-flow
vol1/ch10-pass4
vol1/ch9-pass4
vol1/ch8-pass4
vol1/ch7-pass4
vol1/ch6-pass4
vol1/ch5-pass4
vol1/apC-pass3
vol1/ch4-pass4
vol1/ch3-pass4
vol1/ch2-pass4
vol1/ch1-pass4
vol1/frontmatter
vol1/apE-pass3
vol1/apD-pass3
fmt-fix
vol1/ch14-pass3
kai/clarify-community-map-totals
vol1/ch13-pass3
vol1/ch12-pass3
vol1/ch11-pass3
vol1/ch10-pass3
vol1/ch7-pass3
vol1/ch9-pass3
vol1/ch8-pass3
vol1/ch6-pass3
vol1/ch5-pass3
vol1/ch4-pass3
vol1/ch3-pass3
vol1/ch2-pass3
vol1/ch1-pass3
vol1/ch6-pass2
vol1/ch5-pass2
vol1/ch4-pass2
vol1/ch3-pass2
vol1/ch2-pass2
fix/badge-fixes
chore/precommit-cleanup
cleanup/book-validate-paths
fix/staffml-trigger-on-workflow-edits
fix/staffml-reusable-concurrency
feat/container-preflight-urls
instructors-v0.1.1
vol1-pr1957-merged
vol2-v0.2.1
vol1-v0.7.1
vol1-v0.7.0+vol2-v0.2.0
slides-v0.1.0
vol2-v0.2.0
vol1-v0.7.0
tinytorch-v0.1.13
mlsysim-v0.2.0
vol2-v0.1.2
vol1-v0.6.2
tinytorch-v0.1.12
vol2-v0.1.1
vol1-v0.6.1
mlsysim-v0.1.3
vol2-v0.1.0
vol1-v0.6.0
mlsysim-v0.1.2
staffml-v0.1.1
tinytorch-v0.1.11
labs-v0.1.0
kits-v0.1.0
slides-latest
instructors-v0.1.0
staffml-v0.1.0
mlsysim-v0.1.1
mlsysim-v0.1.0
tinytorch-v0.1.10
build-verified-windows-v1
tinytorch-v0.1.9
tinytorch-v0.1.8
mit-submission-v1
tinytorch-v0.1.7
tinytorch-v0.1.6
tinytorch-v0.1.5
tinytorch-slides-v0.1.0
tinytorch-v0.1.3
tinytorch-v0.1.4
tinytorch-v0.1.2
tinytorch-v0.1.1
tinytorch-v0.1.0
book-v0.5.1
tinytorch-audio-v0.1.1
tinytorch-audio-assets-v0.1
book-v0.5.0
book-v0.4.2
book-v0.4.1
book-v0.4.0
book-v0.3.0
book-v0.2.0
book-v0.1.0
Labels
Clear labels
area: book
area: collabs
area: kits
area: labs
area: socratiq
area: staffml
area: tinytorch
area: tools
area: website
bug
dependencies
format: epub
format: pdf
javascript
link-health
link-rot
priority-high
pull-request
staffml
type: bug
type: citation
type: code
type: errata
type: improvement
type: new
type: question
vault-sli
Mirrored from GitHub Pull Request
Milestone
No items
No Milestone
Projects
Clear projects
No projects
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: github-starred/cs249r_book#35280
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Originally created by @tddg on GitHub (May 31, 2026).
Original GitHub issue: https://github.com/harvard-edge/cs249r_book/issues/1825
Area
Book — Volume I
Location
Vol2, Computing Infrastructure chapter, Definition 1.5
Description
Bullet 1:
Significance (quantitative): ... For a 7B-parameter model on H100 (R_peak = 312 TFLOPS)
It should be A100, as H100 peak TFLOPS is 1,979 .
Expected Behavior
For a 7B-parameter model on A100 (instead of H100)
Environment (TinyTorch bugs only)
No response
@profvjreddi commented on GitHub (Jun 1, 2026):
Hi @tddg - thanks so much for catching this.
Coincidentally, I just fixed this in my local dev because I was rewriting the mlsysIM (https://mlsysbook.ai/mlsysim) to fix issues -- it's basically the calculator for the book, and I needed to refactor it, so I will resolve the issue in there and ping you when its pushed live, which will happen in a few days. I've been cleaning up a lot of things.
Appreciate your help!
@all-contributors please add @tddg as a contributor for 🪲 Bug in Book
@github-actions[bot] commented on GitHub (Jun 1, 2026):
I've added @tddg as a contributor to book! 🎉
Recognized for: bug, code
Project(s): book (explicitly mentioned in comment)
Based on: @all-contributors please add @tddg as a contributor for 🪲 Bug in Book
The contributor list has been updated in:
book/.all-contributorsrc,book/README.mdREADME.mdWe love recognizing our contributors! ❤️
@profvjreddi commented on GitHub (Jul 2, 2026):
Thanks again @tddg. This is now fixed. As part of the mlsysIM refactor, the Computing Infrastructure chapter no longer hardcodes accelerator specs; the A100 and H100 numbers are pulled directly from the hardware zoo. The old definition that mislabeled the 312 TFLOPS FP16 peak as H100 has been rewritten, and 312 TFLOPS is now correctly attributed to the A100 throughout. The change is merged to main and will carry into the next release. Closing this out.
@all-contributors please add @tddg as a contributor for 🪲 Bug in Book
@github-actions[bot] commented on GitHub (Jul 2, 2026):
I've added @tddg as a contributor to book! 🎉
Recognized for: bug, code
Project(s): book (explicitly mentioned in comment)
Based on: @all-contributors please add @tddg as a contributor for 🪲 Bug in Book
The contributor list has been updated in:
book/.all-contributorsrc,book/README.mdREADME.mdWe love recognizing our contributors! ❤️