-
Notifications
You must be signed in to change notification settings - Fork 757
Chapter 9 of Statistical Rethinking 2nd Edition #91
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
Check out this pull request on Review Jupyter notebook visual diffs & provide feedback on notebooks. Powered by ReviewNB |
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:54Z nitpicking: no need for the legend "Statistical Rethinking, 2nd Edition", right?
|
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:54Z If you use az.plot_kde you will be able to remove "ax.set_xlim(0, 36)". |
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:55Z fix typo --> "Evaluate potential and kinetic energies at start and end of trajectory" |
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:56Z We may ask Richard for guidance. |
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:56Z This is already supported, right? (or only on master?) |
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:57Z maybe set alpha to better mimic the plot in the book? |
View / edit / reply to this conversation on ReviewNB aloctavodia commented on 2020-06-12T21:04:58Z This is already possible |
Hi @oscarbranson ! |
Thanks @aloctavodia and @AlexAndorra - I haven't had time to look at it yet... I hope to get to it this weekend. |
Hi @oscarbranson ! Do you think you'll have some time to move this across the finish line? 🏁 |
Yes! Apologies for the delay, @AlexAndorra... I've been away. Should get to it early this week. |
Let's move this one along: merging now so that people can already use it, and when someone has time he can implement the changes @aloctavodia mentioned above 😉 |
Apologies @AlexAndorra and @aloctavodia - it's been a hectic couple of months. I'll try to revisit this soon, but probably a good idea to merge for now - it mostly works as-is! |
Do not worry @oscarbranson we are living crazy times. And you already helped a lot! Thanks for your contributions. |
Mangled the last one (#89) so badly that I created a new PR... hope this is better @aloctavodia!