lostecho/grokking_algorithms
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

link to errata page

Browse Source
This commit is contained in:
Aditya Bhargava
2016-10-30 06:23:05 -07:00
parent 0aec97dbf2
commit c3396b8b0a
2 changed files with 4 additions and 72 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
README.md
View File

@@ -4,6 +4,10 @@ This is the code in my book [Grokking Algorithms](https://www.manning.com/bharga
Also check out [Python Tutor](http://pythontutor.com/), a great website that helps you step through Python code line by line.
## Errata
[Here's the errata page](http://adit.io/errata.html).
## Images
This repo also contains every image in Grokking Algorithms, in hi-res. These images are available for free for non-commercial use. If you use an image, please add "copyright Manning Publications, drawn by adit.io". You are welcome to use these images in any non-commercial teaching materials, presentations, etc.

72
errata.markdown
View File

@@ -1,72 +0,0 @@
Page 41 [Image]:
if i <= 1
WE ARE DONE
Code sample is `i <= 0`. Should match?
Page 43 [Snippet 2]:
def greet2(name):
...
def bye():
print "ok bye!"
`def bye()` should be aligned with `def greet2(name)` not nested?
Chapter 8 (Code):
best_station = None
states_covered = set()
for station, states_for_station in stations.items():
final loop has:
best_station = None
states_covered = set()
for station, states in stations.items():
should be states for both?
~
Chapter 8 (final algorithm):
Can't seem to get Jupyter Notebook, Kernel: Python 2.7 to run the while loop? Times out?
Might just be me? Or one of my typos?
~
Chapter 9 (Image/Example):
Knapsack items are 4lbs, 3lbs, 1lbs.
Chapter 8 items are 30lbs, 20lbs, 15lbs.
Values should be the same?
~
Suggestion:
Make the difference between pseudo-code, intermediate code, and executable code more explicit?
~
Suggestion:
Full executable python algorithms for Chapter 9 and 10? Given the awesome pace of the early chapters I was disappointed that there wasn't any executable code in 9 and 10 :(
---
So you started out with a 1680 × 640 farm that needed to be split up. But now you need to split up a smaller segment, 640 × 400. If you nd the biggest box that will work for this size, that will be the biggest box that will work for the entire farm. You just reduced the problem from a 1680 × 640 farm to a 640 × 400 farm!
I think it's confusing as it's using W X H for 1680 x 640 but H X W for 640 x 400
So should it not be?
So you started out with a 1680 × 640 farm that needed to be split up. But now you need to split up a smaller segment, 400 × 680. If you nd the biggest box that will work for this size, that will be the biggest box that will work for the entire farm. You just reduced the problem from a 1680 × 640 farm to a 400 × 680 farm!