-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LLN and CLT #67
Comments
Yep, good catch on that one. Thanks for reporting. If you would like to submit a PR for the mistake you found in illustrates_lln.py that would be great. I will fix the problems in the website now -- although I'm not sure when the updates will appear online. |
closed by #68 |
@oyamad Much appreciated @spencerlyon2 Please go ahead and merge the fix-lln-clt branch into master in the private repo and close the issue. |
Done, thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
I (re)started a study group with students on quant-econ.
Today we found the following:
Regarding illustrates_lln.py, lines 44-46, if I am not mistaken,
should be
http://nbviewer.ipython.org/gist/oyamad/4789a8bcaf5aa867e92d
(Should I submit a pull request myself?)
In the code shown in Simulation 1, the following lines are missing:
Compare the original code.
The text was updated successfully, but these errors were encountered: