-
-
Notifications
You must be signed in to change notification settings - Fork 745
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
Rename pymc3 -> pymc #165
Rename pymc3 -> pymc #165
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@@ -10,7 +10,7 @@ | |||
"import matplotlib.pyplot as plt\n", | |||
"import numpy as np\n", | |||
"import pandas as pd\n", | |||
"import pymc3 as pm\n", | |||
"import pymc as pm\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With this change, I get
AttributeError: module 'pymc' has no attribute 'trace_to_dataframe'
in code 4.32
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried moving over to
trace_df = az.from_pymc3(trace=trace_4_1)
but got
ModuleNotFoundError: No module named 'pymc3'
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh I see pr #141 is working on this
@@ -9,7 +9,7 @@ | |||
"import arviz as az\n", | |||
"import matplotlib.pyplot as plt\n", | |||
"import numpy as np\n", | |||
"import pymc3 as pm\n", | |||
"import pymc as pm\n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Issue with this chapter using pymc too: pymc-devs/pymc#5443
Previous commits contained the expression `Pr(positive|mortal)1 - Pr(vampire)` as part of the normalizing constant. I believe the correct expression is `Pr(positive|mortal)(1 - Pr(vampire))`. This commit corrects the markdown of the vampire normalizing constant. Co-authored-by: laj <[email protected]>
Should this one be merged sooner rather than later? |
Add BMCP to resources
* Add bayes_rule env * Rename prefix * Add seaborn
* Update README.md * Update README.md Co-authored-by: Oriol Abril-Pla <[email protected]> * Update README.md Co-authored-by: Oriol Abril-Pla <[email protected]>
Good point, need to resolve conflicts @fonnesbeck |
In advance of the PyMC4 beta, this PR is renaming
pymc3
topymc
throughout, taking care to preserve cases likearviz.from_pymc3
andbcm_pymc3
which have not yet changed.