-
Notifications
You must be signed in to change notification settings - Fork 160
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
CN-W0D5 Statistics Tutorial 2 Video 5: Statistics #114
Comments
@ogulyurdakul thank you for reporting this. I have added a comment below the video to clarify this. #117 resolves this issue. |
I don't think I see it yet, is it on the notebooks, video comments or the video captions? |
@ogulyurdakul have you checked the code and the PR? https://github.com/NeuromatchAcademy/precourse/pull/117/files#diff-7ae65b771274f7cb2925fa50a44cbd11b047f203d6296e9c5d7f99c539c1980c |
I hadn't, I checked the published notebooks directly from the website those on Colab. I guess the changes are not published yet, then? |
You have checked the jupyterbook, not the published notebooks. You need to open the notebook by clicking the badge to check it. Anyways, the issue has been fixed. Thus, I posted the PR that fixes the issue to be able to check and track the changes. Thank you for your contribution. After a PR merge, you can also read about book generation and the time needed to get a new book. |
On the video at this link (https://youtu.be/1Q3VqcpfvBk?t=35) the Bayes' rule is stated incorrectly, as a division by the joint density/probability as opposed to the marginal. Here's the screenshot:
The text was updated successfully, but these errors were encountered: