udacity / dlnd-issue-reports

5 stars 0 forks source link

CNN math error #575

Open lcrucks opened 6 years ago

lcrucks commented 6 years ago

From student: "Under CNN section, math in one of the solution for parameter sharing is incorrect.

There are 3860 total parameters. That's 196 times fewer parameters! Here's how the answer is calculated:

(8 8 3 + 1) 20 = 3840 + 20 = 3860 => Incorrect (8 8 3) 20 = 3840 + 20 = 3860"