-
-
Notifications
You must be signed in to change notification settings - Fork 26
[markov_chains] Integrate Comments #67
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
Conversation
✅ Deploy Preview for taupe-gaufre-c4e660 ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
|
@mmcky there seem to be false positives in the line checker: Many thanks if you could take a look :) |
|
Thanks @HumphreyYang , it's looking good. Small comments above. |
|
thanks @HumphreyYang would you mind to review the merge conflict? It looks like there have been updates made on the |
Many thanks! It seems like the main branch has some building issues. |
|
Thanks @HumphreyYang . @mmcky , please merge when ready (noting the build failure). |
|
@mmcky I couldn't see what the build issue was but I'm going to go ahead and merge this. We can iterate from there. |
Similar to the LLN and CLT lecture, this PR resolves some of the comments proposed this morning. Due to the audio quality, I missed some of the valuable comments from the team. Please kindly comment on this PR so that I can further improve this lecture.