-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Clarified parameterization of AR1 model in AR notebook #4063
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 |
Codecov Report
@@ Coverage Diff @@
## master #4063 +/- ##
==========================================
- Coverage 88.91% 86.83% -2.08%
==========================================
Files 89 88 -1
Lines 14429 14165 -264
==========================================
- Hits 12829 12300 -529
- Misses 1600 1865 +265
|
View / edit / reply to this conversation on ReviewNB twiecki commented on 2020-08-23T17:21:42Z nice to have the version output. |
View / edit / reply to this conversation on ReviewNB twiecki commented on 2020-08-23T17:21:43Z How does it sample with default tuning?
|
View / edit / reply to this conversation on ReviewNB twiecki commented on 2020-08-23T17:21:44Z also here good to have output |
View / edit / reply to this conversation on ReviewNB AlexAndorra commented on 2020-08-24T11:13:57Z This small text needs to be upated too |
+ the PyMC3 version is already shown in the watermark output at the end + a few small changes in the text of the cells that
I just made the change requested in the review and removed the print of the PyMC3 version, because that's already covered by the watermark at the end of the notebook. Hope this pushes it over the finish line? |
@michaelosthege LGTM but merge conflict :-/ |
The AR notebook conflates the precision and variance in the description and data simulation steps. I've updated the notebook to make it consistent.
Closes #4059