Skip to content
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

Repair "typo" in vampire markdown #172

Merged
merged 1 commit into from
Mar 20, 2022

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