udacity / CarND-Kidnapped-Vehicle-Project

MIT License
121 stars 1.3k forks source link

Incorrect comment on updateWeights #9

Closed mithi closed 7 years ago

mithi commented 7 years ago

in the updateWeights function it says:

//   Note that you'll need to switch the minus sign in that equation to a plus to account
//   for the fact that the map's y-axis actually points downwards.

It is agreed that

That comment is incorrect, you shouldn’t change the sign. they still haven’t fixed it.

Discussion from: https://carnd.slack.com/archives/C4TSP7BFU/p1494255995984903

beeva-ricardoguerrero commented 7 years ago

This issue seems to be already solved.

mithi commented 7 years ago

@beeva-ricardoguerrero Thanks!