[PR #788] [MERGED] swap dimension order for W^L (in #4800

New Issue

GiteaMirror · 2026-04-19T12:38:59-05:00

GiteaMirror commented

2026-04-19 12:38:59 -05:00

📋 Pull Request Information

Original PR: https://github.com/harvard-edge/cs249r_book/pull/788
Author: @TheHiddenLayer
Created: 4/1/2025
Status: ✅ Merged
Merged: 4/1/2025
Merged by: @profvjreddi

Base: dev ← Head: dev

📝 Commits (2)

c4f474a swap dimension order for W^L
8cdaa4d Auto-update changelog

📊 Changes

2 files changed (+26 additions, -72 deletions)

View changed files

📝 CHANGELOG.md (+25 -71)
📝 contents/core/dl_primer/dl_primer.qmd (+1 -1)

📄 Description

Should the dimensions for W^L be swapped in the Mathematical Representation section? Since there are 10 neurons in the final output layer, then the dimensions of the final layer's weights in matrix form would be n_{L-1} rows and 10 columns.

The text has been proofread for grammar and spelling errors.
All images, figures, and tables render properly without any glitches.
All images have a source or they are properly linked to external sites.
The chapter's formatting is consistent with the rest of the book.

The chapter has been locally built and tested using Quarto.

Sorry, I didn't do this.

_{🔄 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/788 **Author:** [@TheHiddenLayer](https://github.com/TheHiddenLayer) **Created:** 4/1/2025 **Status:** ✅ Merged **Merged:** 4/1/2025 **Merged by:** [@profvjreddi](https://github.com/profvjreddi) **Base:** `dev` ← **Head:** `dev` --- ### 📝 Commits (2) - [`c4f474a`](https://github.com/harvard-edge/cs249r_book/commit/c4f474a7082bb357fbdcc995bb99972126cf3d0c) swap dimension order for W^L - [`8cdaa4d`](https://github.com/harvard-edge/cs249r_book/commit/8cdaa4d243cd6f8bce54e315b8224bcd99a49614) Auto-update changelog ### 📊 Changes **2 files changed** (+26 additions, -72 deletions) <details> <summary>View changed files</summary> 📝 `CHANGELOG.md` (+25 -71) 📝 `contents/core/dl_primer/dl_primer.qmd` (+1 -1) </details> ### 📄 Description Should the dimensions for $$W^L$$ be swapped in the [Mathematical Representation](https://mlsysbook.ai/contents/core/dl_primer/dl_primer.html#mathematical-representation) section? Since there are 10 neurons in the final output layer, then the dimensions of the final layer's weights in matrix form would be $$n_{L-1}$$ rows and 10 columns. --- - [x] The text has been proofread for grammar and spelling errors. - [x] All images, figures, and tables render properly without any glitches. - [x] All images have a source or they are properly linked to external sites. - [x] The chapter's formatting is consistent with the rest of the book. > - [ ] The chapter has been locally built and tested using Quarto. Sorry, I didn't do this. --- <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-04-19 12:38:59 -05:00

GiteaMirror closed this issue

2026-04-19 12:39:01 -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#4800