[GH-ISSUE #402] Note on first 4 chapter of "benchmarking.qmd" #5368

New Issue

GiteaMirror · 2026-04-21T21:20:49-05:00

GiteaMirror commented

2026-04-21 21:20:49 -05:00

Originally created by @BravoBaldo on GitHub (Aug 26, 2024).
Original GitHub issue: https://github.com/harvard-edge/cs249r_book/issues/402

The "Baseline" and "Baseline Models" sections start with the same 2 sentences and are too close for that.
In "11.4.3 Training vs. Inference"
Not sure (probably just a translation flaw):
"On the other hand, benchmarking inference evaluates"...
"benchmarking inference" vs "Inference benchmarking" ? (different places)
See "** Energy Consumption: **" maybe missing format tags (I suspect spaces in .qmd file)
In section "Benchmarks" maybe "MLPerf Training Benchmark" is differently formatted compared to previous "DAWNBench" and "Fathom"
Search "if a model takes less time to predict,".... maybe opposite!
In section "11.4.5 Inference Benchmarks"... sub-sec. "Metrics" points 5 and 6 are doubled "Energy Efficiency: "
Search "OpenVINO toolkit provides a benchmark tool" ... The whole paragraph is duplicated

Originally created by @BravoBaldo on GitHub (Aug 26, 2024). Original GitHub issue: https://github.com/harvard-edge/cs249r_book/issues/402 - The "Baseline" and "Baseline Models" sections start with the same 2 sentences and are too close for that. - In "11.4.3 Training vs. Inference" Not sure (probably just a translation flaw): "On the other hand, benchmarking inference evaluates"... "benchmarking inference" vs "Inference benchmarking" ? (different places) - See "** Energy Consumption: **" maybe missing format tags (I suspect spaces in .qmd file) - In section "Benchmarks" maybe "MLPerf Training Benchmark" is differently formatted compared to previous "DAWNBench" and "Fathom" - Search "if a model takes less time to predict,".... maybe opposite! - In section "11.4.5 Inference Benchmarks"... sub-sec. "Metrics" points 5 and 6 are doubled "Energy Efficiency: " - Search "OpenVINO toolkit provides a benchmark tool" ... The whole paragraph is duplicated

GiteaMirror added the area: book label 2026-04-21 21:20:49 -05:00

GiteaMirror closed this issue

2026-04-21 21:20:50 -05:00

GiteaMirror commented

2026-04-21 21:20:51 -05:00

@profvjreddi commented on GitHub (Aug 26, 2024):

Thank you, @BravoBaldo, for catching these mistakes. Your feedback is on point, and I really appreciate it. Also, thank you for sending these in small pieces, it makes it so much easier for me to quickly fix! So keep them coming 🙌

It seems that some duplicates were left in during one of the merges. All your other points are accurate, so I'm grateful that you pointed them out.

I have now updated the material. Thank you for reading through everything carefully.

@profvjreddi commented on GitHub (Aug 26, 2024): Thank you, @BravoBaldo, for catching these mistakes. Your feedback is on point, and I really appreciate it. Also, thank you for sending these in small pieces, it makes it so much easier for me to quickly fix! So keep them coming 🙌 It seems that some duplicates were left in during one of the merges. All your other points are accurate, so I'm grateful that you pointed them out. I have now updated the material. Thank you for reading through everything carefully.

GiteaMirror commented

2026-04-21 21:20:52 -05:00

@profvjreddi commented on GitHub (Aug 26, 2024):

I deleted likely spam-related comments to keep things clean.

@profvjreddi commented on GitHub (Aug 26, 2024): I deleted likely spam-related comments to keep things clean.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/cs249r_book#5368