Mlsysbook_chapter_9_effTradeoff #433

New Issue

GiteaMirror · 2026-03-22T15:39:32-05:00

GiteaMirror commented

2026-03-22 15:39:32 -05:00

Originally created by @formlsysbookissue on GitHub (Sep 3, 2025).

(Minor and not necessarily suggestion) The "Efficiency tradeoff" heading appears twice in the document: once as a main heading and once as a subheading. The contents are also quite similar and feel repetitive.
The first (subheading) occurrence should be left without a title, and some part of its content should be merged into the main heading section to avoid redundancy and improve flow.

Originally created by @formlsysbookissue on GitHub (Sep 3, 2025). (Minor and not necessarily suggestion) The "Efficiency tradeoff" heading appears twice in the document: once as a main heading and once as a subheading. The contents are also quite similar and feel repetitive. The first (subheading) occurrence should be left without a title, and some part of its content should be merged into the main heading section to avoid redundancy and improve flow.

GiteaMirror added the area: book label 2026-03-22 15:39:32 -05:00

GiteaMirror closed this issue

2026-03-22 15:39:33 -05:00

GiteaMirror commented

2026-03-22 15:39:33 -05:00

@profvjreddi commented on GitHub (Sep 3, 2025):

✅ Issue Fully Resolved

After careful analysis of the original feedback, I've now completely addressed all aspects of the issue:

Original feedback analysis:

✅ Duplicate headings: Removed redundant 'Efficiency Trade-offs' subheading
✅ Repetitive content: Eliminated multiple similar introductions about trade-offs between algorithmic, compute, and data efficiency
✅ Content consolidation: Merged resource-constrained perspective into main section
✅ Improved flow: Streamlined transitions and removed formulaic repetitions

Key changes made:

Removed the redundant subheading entirely (not just renamed)
Consolidated repetitive explanations of efficiency trade-offs
Made the Tiny ML example more concise and integrated
Eliminated duplicate conceptual introductions
Improved overall document structure and readability

Commits:

ed2bf0a5: Initial heading rename
2c349dbe: Section label consistency
3ec047a5: Content consolidation
114cf2fb: Final redundancy elimination

The chapter now has a cleaner structure with no duplicate headings or repetitive content, fully addressing the original feedback. Thanks again @formlsysbookissue, please take a look at and let mek now if you are ok with it, and again you can cehck the ddev version here: dev

@profvjreddi commented on GitHub (Sep 3, 2025): ✅ **Issue Fully Resolved** After careful analysis of the original feedback, I've now completely addressed all aspects of the issue: **Original feedback analysis:** 1. ✅ **Duplicate headings**: Removed redundant 'Efficiency Trade-offs' subheading 2. ✅ **Repetitive content**: Eliminated multiple similar introductions about trade-offs between algorithmic, compute, and data efficiency 3. ✅ **Content consolidation**: Merged resource-constrained perspective into main section 4. ✅ **Improved flow**: Streamlined transitions and removed formulaic repetitions **Key changes made:** - Removed the redundant subheading entirely (not just renamed) - Consolidated repetitive explanations of efficiency trade-offs - Made the Tiny ML example more concise and integrated - Eliminated duplicate conceptual introductions - Improved overall document structure and readability **Commits:** - ed2bf0a5: Initial heading rename - 2c349dbe: Section label consistency - 3ec047a5: Content consolidation - 114cf2fb: Final redundancy elimination The chapter now has a cleaner structure with no duplicate headings or repetitive content, fully addressing the original feedback. Thanks again @formlsysbookissue, please take a look at and let mek now if you are ok with it, and again you can cehck the ddev version here: [dev](https://harvard-edge.github.io/cs249r_book_dev/)

GiteaMirror commented

2026-03-22 15:39:34 -05:00

@profvjreddi commented on GitHub (Sep 3, 2025):

Closing as completed. All aspects of the original feedback have been addressed with proper content consolidation and structure improvements.

@profvjreddi commented on GitHub (Sep 3, 2025): Closing as completed. All aspects of the original feedback have been addressed with proper content consolidation and structure improvements.

GiteaMirror commented

2026-03-22 15:39:34 -05:00

@formlsysbookissue commented on GitHub (Sep 4, 2025):

I have no additional comments regarding Issue 960. However, I plan to review the final version of the book in terms of Issue 959 and if others, in its entirety. If I identify any areas for further enhancement or adjustment, I will provide feedback accordingly. Thank you for your efforts and attention to detail."

@formlsysbookissue commented on GitHub (Sep 4, 2025): I have no additional comments regarding Issue 960. However, I plan to review the final version of the book in terms of Issue 959 and if others, in its entirety. If I identify any areas for further enhancement or adjustment, I will provide feedback accordingly. Thank you for your efforts and attention to detail."

GiteaMirror commented

2026-03-22 15:39:35 -05:00

@profvjreddi commented on GitHub (Sep 4, 2025):

Sounds great! Thanks @formlsysbookissue

@profvjreddi commented on GitHub (Sep 4, 2025): Sounds great! Thanks @formlsysbookissue

GiteaMirror referenced this issue

2026-03-22 15:49:40 -05:00

[PR #433] [MERGED] 431 Notes in "privacy_security.qmd" #738

GiteaMirror referenced this issue

2026-04-11 08:12:14 -05:00

[PR #433] [MERGED] 431 Notes in "privacy_security.qmd" #1961

GiteaMirror referenced this issue