[PR #950] [MERGED] fix: Correct biological neuron mapping and add optical interconnects context #1024

Closed
opened 2026-03-22 15:58:09 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/950
Author: @profvjreddi
Created: 8/23/2025
Status: Merged
Merged: 8/23/2025
Merged by: @profvjreddi

Base: devHead: fix/issue-947-dendrite-synapse-correction


📝 Commits (2)

  • 79e73a5 fix: Correct biological neuron to artificial neuron mapping
  • f3eafb0 feat: Add optical interconnects context to sustainability chapter

📊 Changes

6 files changed (+135 additions, -10 deletions)

View changed files

📝 .gitignore (+1 -0)
CLAUDE.md (+102 -0)
📝 quarto/contents/core/dl_primer/dl_primer.qmd (+11 -7)
📝 quarto/contents/core/dl_primer/dl_primer_quizzes.json (+3 -3)
📝 quarto/contents/core/sustainable_ai/sustainable_ai.bib (+14 -0)
📝 quarto/contents/core/sustainable_ai/sustainable_ai.qmd (+4 -0)

📄 Description

Summary

Thank you @formlsysbookissue for this excellent feedback! Your observations about the biological neuron mapping needing clarification and optical interconnects deserving more attention are spot on. Really appreciate you taking the time to help improve the book's technical accuracy.

Changes Made

This PR addresses issue #947 with two focused improvements:

1. Corrected Biological-Artificial Neuron Mapping (Chapter 3)

  • Fixed the analogy: Dendrites now correctly map to inputs (receivers), synapses to weights (modulators)
  • Updated Table 3.1 to reflect accurate biological-artificial correspondence
  • Revised explanatory text throughout the section to maintain this distinction
  • Updated quiz question to test the corrected understanding

2. Added Optical Interconnects Context (Chapter 18)

  • Integrated naturally into the existing material resources discussion
  • Provided concrete examples: TPUv4 (400 Gbps), Mellanox technologies
  • Added academic citation: Jouppi et al. 2023 paper on TPUv4
  • Connected to sustainability theme: Linked optical technology adoption to germanium supply concerns
  • Included explanatory footnote about Mellanox's role in the industry

Why These Changes Matter

The dendrite/synapse correction helps students build accurate mental models from the foundation, preventing conceptual confusion as they progress to more complex neural network architectures.

The optical interconnects addition provides crucial context about a technology that's transitioning from optimization to necessity as AI models scale toward trillion parameters.

Testing

  • Content reviewed for technical accuracy
  • Build tested locally
  • Cross-references verified (@tbl- format used)
  • Academic tone maintained throughout
  • Changes integrate seamlessly with existing content

Fixes #947


Thank you again for your thoughtful feedback - contributions like yours help make this textbook a better learning resource for everyone!


🔄 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/950 **Author:** [@profvjreddi](https://github.com/profvjreddi) **Created:** 8/23/2025 **Status:** ✅ Merged **Merged:** 8/23/2025 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `fix/issue-947-dendrite-synapse-correction` --- ### 📝 Commits (2) - [`79e73a5`](https://github.com/harvard-edge/cs249r_book/commit/79e73a530c58de054b4101f0c9176902e3367dfd) fix: Correct biological neuron to artificial neuron mapping - [`f3eafb0`](https://github.com/harvard-edge/cs249r_book/commit/f3eafb01f4ac1b14845d9a5fb2e5ad0652080260) feat: Add optical interconnects context to sustainability chapter ### 📊 Changes **6 files changed** (+135 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+1 -0) ➕ `CLAUDE.md` (+102 -0) 📝 `quarto/contents/core/dl_primer/dl_primer.qmd` (+11 -7) 📝 `quarto/contents/core/dl_primer/dl_primer_quizzes.json` (+3 -3) 📝 `quarto/contents/core/sustainable_ai/sustainable_ai.bib` (+14 -0) 📝 `quarto/contents/core/sustainable_ai/sustainable_ai.qmd` (+4 -0) </details> ### 📄 Description ## Summary Thank you @formlsysbookissue for this excellent feedback! Your observations about the biological neuron mapping needing clarification and optical interconnects deserving more attention are spot on. Really appreciate you taking the time to help improve the book's technical accuracy. ## Changes Made This PR addresses issue #947 with two focused improvements: ### 1. Corrected Biological-Artificial Neuron Mapping (Chapter 3) - **Fixed the analogy**: Dendrites now correctly map to inputs (receivers), synapses to weights (modulators) - **Updated Table 3.1** to reflect accurate biological-artificial correspondence - **Revised explanatory text** throughout the section to maintain this distinction - **Updated quiz question** to test the corrected understanding ### 2. Added Optical Interconnects Context (Chapter 18) - **Integrated naturally** into the existing material resources discussion - **Provided concrete examples**: TPUv4 (400 Gbps), Mellanox technologies - **Added academic citation**: Jouppi et al. 2023 paper on TPUv4 - **Connected to sustainability theme**: Linked optical technology adoption to germanium supply concerns - **Included explanatory footnote** about Mellanox's role in the industry ## Why These Changes Matter The dendrite/synapse correction helps students build accurate mental models from the foundation, preventing conceptual confusion as they progress to more complex neural network architectures. The optical interconnects addition provides crucial context about a technology that's transitioning from optimization to necessity as AI models scale toward trillion parameters. ## Testing - [x] Content reviewed for technical accuracy - [x] Build tested locally - [x] Cross-references verified (@tbl- format used) - [x] Academic tone maintained throughout - [x] Changes integrate seamlessly with existing content Fixes #947 --- Thank you again for your thoughtful feedback - contributions like yours help make this textbook a better learning resource for everyone! --- <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-03-22 15:58: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#1024