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

Article informative priors #10

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

cetagostini-wise
Copy link

@cetagostini-wise cetagostini-wise commented Apr 20, 2024

Changes log:

@juanitorduz
Copy link

@twiecki would it be possible to allow the ReviewNB for this repository so that we can provide feedback on the notebook directly 🙏 ?

@aloctavodia
Copy link

If adding ReviewNB is not possible a workaround is to use jupytext to automatically generate .md files. Not at nice as ReviewNB, but reading and commenting .md files is better than just notebooks.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@twiecki
Copy link
Contributor

twiecki commented Apr 21, 2024

added

@cetagostini-wise
Copy link
Author

cetagostini-wise commented Apr 21, 2024

@juanitorduz You ask to add uncertainty in the prior plots but, I didn't add it in the plots because the intervals are to wide and makes the images look very strange, and pick a lower interval could be seeing as guiding the information in a "convenient" way at best.

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.

4 participants