-
-
Notifications
You must be signed in to change notification settings - Fork 26
[markov_chain_I] Update animation for different initial distributions #515
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
Dear John, This pull request updates the Markov chain animation for different initial distributions. In addition, this pull request also updates the descriptions of the code. The previous pull request #492 can be closed as this one has no merge conflict. Best, Longye
✅ Deploy Preview for taupe-gaufre-c4e660 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
This reverts commit f699174.
Thanks @longye-tian , great animations! @mmcky , perhaps we should hide the code? Also, should we / can we set the default to "loop" rather than "once" for both of these animations? Please merge when ready. |
thanks @jstac I agree with hiding the code. @longye-tian can you please add |
@jstac I think once is best -- sometimes I find when I go to a website and it loops eternally it is distracting. I think the play button would be a clear way to run it again. |
Hi Matt, I have added the hide input tag to both the animation code. Best, |
thanks @longye-tian and @jstac. |
Dear John @jstac,
This pull request updates the Markov chain animation for different initial distributions.
In addition, this pull request also updates the descriptions of the code.
The previous pull request, #492, can be closed as this one has no merge conflict.
Best ❤️
Longye