1475: Add unconditional exact CI for proportion differences#1476
Conversation
Code Coverage SummaryDiff against mainResults for commit: 96ec5ff Minimum allowed coverage is ♻️ This comment has been updated with latest results |
Unit Tests Summary 1 files 85 suites 1m 41s ⏱️ Results for commit 96ec5ff. ♻️ This comment has been updated with latest results. |
Unit Test Performance DifferenceAdditional test case details
Results for commit 8b7bc83 ♻️ This comment has been updated with latest results. |
|
@danielinteractive lets merge first #1477 so we get rid of the roxygen update |
Melkiades
left a comment
There was a problem hiding this comment.
Hey Daniel! this is a great addition :) — the Santner & Snell tail method is exactly the right approach for unconditional exact CIs, and the implementation reads very cleanly.
Added a couple of comments
…nsightsengineering/tern into 1475_uncond_exact_prop_diff_ci
|
For roxygen2 at 8.0.0 there is a different PR: #1477. Not sure which one should go first |
Signed-off-by: Davide Garolini <davide.garolini@roche.com>
merged the roxygen2 PR |
Signed-off-by: Davide Garolini <davide.garolini@roche.com>
Melkiades
left a comment
There was a problem hiding this comment.
Hey Daniel, this is looking really good after the updates! The tests are thorough — I especially like the SAS reference values and the edge cases (all successes, all failures, zero observations).
Thank you so much for the update. Ready to go for me
closes #1475
I did not commit a full roxygen run because my installed 8.0.0 version seems to change a lot of unrelated stuff.