Skip to content

Conversation

mrwizard82d1
Copy link
Contributor

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.

Thank you for opening a pull request!

Please check our style guide, and also make sure that the notebooks you've modified pass the pre-commit checks. If, for example, you modified notebook1.ipynb and notebook2.ipynb, you could do this by running:

pre-commit run --files notebook1.ipynb notebook2.ipynb

You may need to run it a couple of times before all the checks pass.

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.
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@canyon289 canyon289 merged commit e8b4928 into pymc-devs:master Mar 20, 2022
@canyon289
Copy link
Member

Thank you!

@mrwizard82d1
Copy link
Contributor Author

Quite welcome. Very glad to help.

@mrwizard82d1 mrwizard82d1 deleted the laj/task/vampire-markdown branch March 21, 2022 01:22
@mrwizard82d1
Copy link
Contributor Author

mrwizard82d1 commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants