[GH-ISSUE #1825] [Bug] Vol2 Ch2 Definition 1.5 typo #35280

Closed
opened 2026-07-15 22:41:19 -05:00 by GiteaMirror · 4 comments
Owner

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

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_
GiteaMirror added the area: booktype: bug labels 2026-07-15 22:41:19 -05:00
Author
Owner

@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

<!-- gh-comment-id:4592866693 --> @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
Author
Owner

@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.md
  • Main README.md

We love recognizing our contributors! ❤️

<!-- gh-comment-id:4592904001 --> @github-actions[bot] commented on GitHub (Jun 1, 2026): I've added @tddg as a contributor to **book**! :tada: **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.md` - Main `README.md` We love recognizing our contributors! :heart:
Author
Owner

@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

<!-- gh-comment-id:4867686180 --> @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
Author
Owner

@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.md
  • Main README.md

We love recognizing our contributors! ❤️

<!-- gh-comment-id:4867723242 --> @github-actions[bot] commented on GitHub (Jul 2, 2026): I've added @tddg as a contributor to **book**! :tada: **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.md` - Main `README.md` We love recognizing our contributors! :heart:
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#35280