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

Improved clarity for Advanced Data vignette #9

Open
cherz4 opened this issue Feb 10, 2025 · 0 comments
Open

Improved clarity for Advanced Data vignette #9

cherz4 opened this issue Feb 10, 2025 · 0 comments

Comments

@cherz4
Copy link

cherz4 commented Feb 10, 2025

Based on comment in PR #6 , #6 (comment)

To improve clarity and understanding for the reader of the Advanced Data vignette, could you address the following?

  1. In Treat it as non-extinct section, you refer to chainsR package, when I think you mean to refer to nbbp
  2. You may wish to link to some Stan documentation on what divergences mean and when some amounts of divergence are reasonable or unreasonable. I am familiar with divergences, but the Stan message in your vignette is not the same as the one I worked with before (e.g. X # divergent transitions after warmup: How to Diagnose and Resolve Convergence Problems – Stan)
  3. Can you explain / point out where you got the probability of 7% in the Comparison section more clearly?
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

No branches or pull requests

1 participant