FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Correction in the formula for mean squared error by krishnaw14 · Pull Request #845 · aimacode/aima-python · GitHub

Repository navigation

Correction in the formula for mean squared error - #845

Closed
krishnaw14 wants to merge 5 commits into
aimacode:masterfrom
krishnaw14:master
Closed

krishnaw14 wants to merge 5 commits into
aimacode:masterfrom
krishnaw14:master

Conversation

krishnaw14 commented Mar 14, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor

There was a minor error in the formula for mean squared error in neural_nets.ipynb

Comment thread neural_nets.ipynb Outdated
"In both the Perceptron and the Neural Network, we are using the Backpropagation algorithm to train our weights. Basically it achieves that by propagating the errors from our last layer into our first layer, this is why it is called Backpropagation. In order to use Backpropagation, we need a cost function. This function is responsible for indicating how good our neural network is for a given example. One common cost function is the *Mean Squared Error* (MSE). This cost function has the following format:\n",
"\n",
"$$MSE=\\frac{1}{2} \\sum_{i=1}^{n}(y - \\hat{y})^{2}$$\n",
"$$MSE=\\frac{1}{2n} \\sum_{i=1}^{n}(y - \\hat{y})^{2}$$\n",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

should be \frac{1}{n}

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I guess both conventions are used. In this website, it uses 2n and in this, it uses n.

Anyways, I made the changes.

norvig commented Mar 15, 2018

Copy link
Copy Markdown
Member

Oops, there are conflicts agin, due to another PR. These are really hard to review for .ipynb

Copy link
Copy Markdown
Contributor Author

Should I send another PR after pulling the latest changes from the repository? @norvig

Changes from original repo

ad71 commented Mar 15, 2018 •
edited
Loading

Copy link
Copy Markdown
Contributor

This PR contains duplicated changes. It will be easier if you close this PR, update your working directory, create a new branch and open a fresh pull request. Be sure to make a branch before you commit changes. The master branch is for updating the local directory from the upstream repo.

Bringing up to date with original repo
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL