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

sec 17.2 has some issues #162

Open
murphyk opened this issue Jul 7, 2023 · 0 comments
Open

sec 17.2 has some issues #162

murphyk opened this issue Jul 7, 2023 · 0 comments

Comments

@murphyk
Copy link

murphyk commented Jul 7, 2023

In sec 17.2.1 (rank test), the calculation (1+200)/2 = 100.5 might seem obscure to some.
Later you write this more clearly as shown below. Maybe move here?

Screenshot 2023-07-07 at 10 00 36 AM

In sec 17.2.2(vaccines), the pargraph below is hard to understand

Screenshot 2023-07-07 at 10 05 52 AM

The numbers in table 17.1 are obscure - where does 293 come from? (I assume it's ceil(585/2), but this needs to be explained.)

Finally you say that the multivariate hypergeometric distribution is explained in ch 3, but that only covers the univariate case, IIUC. So the code may not be very clear to some readers.

@murphyk murphyk changed the title sec 17.2.2 (bootstrap test for vaccine efficacy) has issues sec 17.2 has some issues Jul 7, 2023
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