Skip to content

remove warning and delete vignette - #154

Draft
audreyyeoCH wants to merge 7 commits into
mainfrom
150_warnings_h_getBetamixPost
Draft

audreyyeoCH wants to merge 7 commits into
mainfrom
150_warnings_h_getBetamixPost

Conversation

@audreyyeoCH

Copy link
Copy Markdown
Collaborator

close #153

@github-actions

github-actions Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
ocPostprob 💚 $27.44$ $-1.24$ $0$ $0$ $0$ $0$
ocPredprob 💔 $205.75$ $+12.68$ $0$ $0$ $0$ $0$
ocPredprobDist 💔 $95.96$ $+4.09$ $0$ $0$ $0$ $0$
ocRctPredprobDist 💔 $28.95$ $+1.92$ $0$ $0$ $0$ $0$
plotOc 💔 $213.19$ $+8.39$ $0$ $0$ $0$ $0$
predprobDist 💚 $40.01$ $-38.76$ $-2426$ $0$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
ocPostprob 💚 $3.13$ $-1.88$ h_get_looks_together_with_h_get_distance_always_gives_unique_looks
ocPredprob 💔 $202.91$ $+12.49$ ocPredprob_correctly_shows_maximum_sample_size_when_no_decision_reached
ocPredprobDist 💔 $24.76$ $+1.26$ ocPredprobDist_gives_higher_PrEfficacy_with_more_efficacy_looks
ocPredprobDist 💔 $61.58$ $+2.81$ ocPredprobDist_gives_higher_PrFutility_with_more_futility_looks
plotOc 💔 $56.55$ $+2.03$ h_get_dataframe_oc_gives_correct_results_for_ocPredprobDist_when_relativeDelta_FALSE
plotOc 💔 $48.83$ $+2.10$ h_get_dataframe_oc_gives_correct_results_for_ocPredprobDist_when_relativeDelta_TRUE
plotOc 💔 $104.74$ $+4.01$ plotOc_gives_expected_results_for_ocPredprobDist_with_different_relativeDelta_status
predprob 💀 $0.02$ $-0.02$ Warning_length_equal_to_length_of_x
predprobDist 💚 $4.42$ $-4.23$ h_predprobdist_gives_correct_list
predprobDist 💚 $11.30$ $-10.97$ h_predprobdist_gives_higher_predictive_probability_when_thetaT_is_lower
predprobDist 💚 $16.20$ $-15.84$ predprobDist_gives_higher_predictive_probability_when_thetaT_is_lower_in_a_single_arm_trial
predprobDist 💚 $7.06$ $-6.88$ predprobDist_gives_the_correct_results_in_a_two_arm_study

Results for commit 28098a8

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                 Stmts    Miss  Cover    Missing
---------------------  -------  ------  -------  -----------
R/betadiff.R                78       0  100.00%
R/boundsPostprob.R          44       0  100.00%
R/boundsPredprob.R          56       1  98.21%   83
R/dbetabinom.R              96       3  96.88%   32, 62, 147
R/oc2.R                    162     162  0.00%    93-326
R/oc3.R                    146     146  0.00%    91-308
R/ocPostprob.R             117       0  100.00%
R/ocPostprobDist.R         122       0  100.00%
R/ocPredprob.R             205       0  100.00%
R/ocPredprobDist.R         257       5  98.05%   363-367
R/ocRctPostprobDist.R      174       0  100.00%
R/ocRctPredprobDist.R      302       0  100.00%
R/plotBeta.R                97       9  90.72%   124-132
R/plotBounds.R              83      83  0.00%    42-130
R/plotDecision.R            73       0  100.00%
R/plotOc.R                  55       2  96.36%   99-100
R/postprob.R                34       1  97.06%   113
R/postprobDist.R            77       1  98.70%   204
R/predprob.R                36       0  100.00%
R/predprobDist.R           140       1  99.29%   269
R/runShinyPhase1b.R          4       4  0.00%    8-13
R/sumBetaDiff.R             80      15  81.25%   102-120
R/sumTable.R                36       0  100.00%
TOTAL                     2474     433  82.50%

Diff against main

Filename          Stmts    Miss  Cover
--------------  -------  ------  -------
R/dbetabinom.R       -3       0  -0.09%
TOTAL                -3       0  -0.02%

Results for commit: b3c94f1

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Oct 17, 2026 •

Copy link
Copy Markdown
Contributor

Unit Tests Summary

    1 files     20 suites   10m 6s ⏱️
  139 tests   137 ✅ 0 💤 2 ❌
1 323 runs  1 320 ✅ 0 💤 3 ❌

For more details on these failures, see this check.

Results for commit b3c94f1.

♻️ This comment has been updated with latest results.

@audreyyeoCH
audreyyeoCH marked this pull request as draft October 19, 2026 08:03
@audreyyeoCH audreyyeoCH self-assigned this Oct 19, 2026
@audreyyeoCH
audreyyeoCH requested review from danielinteractive and removed request for danielinteractive October 19, 2026 08:10
@audreyyeoCH

audreyyeoCH commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator Author

Opening another PR to fix recurring snap fails, see #154, converting this PR to draft for now

This branch has not been deployed

No deployments
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.

150_warnings_h_getBetamixPost

1 participant