Skip to content

fix(introduction): calculation and punctuation. - #69

Merged
llpeterson merged 3 commits into
SystemsApproach:mainfrom
Walentalien:fix/chapter1-text
Oct 6, 2026
Merged

llpeterson merged 3 commits into
SystemsApproach:mainfrom
Walentalien:fix/chapter1-text

Conversation

@Walentalien

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>

@drbruced12 drbruced12 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The punctuation change is correct.
The calculation isn't exactly wrong, given that 1KB = 1024 bytes. It's probably better to write it as (352 × 240 × 24) / (8 × 1024) = 247.5 KB.
In older editions we had some explanation about the difference between bandwidths expressed as Mbps/kbps (where the conventional assumption is M = 10^6 and k = 10^3) versus computer memory sizes in MB/KB where we assume powers of 2. I think putting the 1024 explicitly in the calculation here should remove the confusion.

Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
@Walentalien

Copy link
Copy Markdown
Contributor Author

The punctuation change is correct. The calculation isn't exactly wrong, given that 1KB = 1024 bytes. It's probably better to write it as (352 × 240 × 24) / (8 × 1024) = 247.5 KB. In older editions we had some explanation about the difference between bandwidths expressed as Mbps/kbps (where the conventional assumption is M = 10^6 and k = 10^3) versus computer memory sizes in MB/KB where we assume powers of 2. I think putting the 1024 explicitly in the calculation here should remove the confusion.

Made the change. Just to clarify: the book will continue to use KB/MB rather than Kib/Mib for binary units?

@llpeterson

Copy link
Copy Markdown
Contributor

Accepting this PR, and then I'll add a clarifying footnote.

@llpeterson
llpeterson merged commit b019287 into SystemsApproach:main Oct 6, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants