Skip to content

add robin_mh#114

Open
clarkliming wants to merge 4 commits into
mainfrom
113_mh
Open

add robin_mh#114
clarkliming wants to merge 4 commits into
mainfrom
113_mh

Conversation

@clarkliming

Copy link
Copy Markdown
Collaborator

close #113

implement the robin_mh method

@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  -------------------------------------
R/bias.R                        34       0  100.00%
R/find_data.R                    3       0  100.00%
R/mh_effect.R                   32       0  100.00%
R/predict_couterfactual.R       77       0  100.00%
R/prediction_cf.R               22       0  100.00%
R/robin_glm.R                   49       1  97.96%   40
R/robin_lm.R                    28       0  100.00%
R/robin_mh.R                   205       8  96.10%   92, 101, 105, 110, 116, 236, 254, 333
R/surv_effect.R                 67       0  100.00%
R/survival_cov_adj.R           146       0  100.00%
R/survival_score.R             294       0  100.00%
R/survival.R                   317       1  99.68%   547
R/treatment_effect.R           101       1  99.01%   51
R/utils.R                      204       3  98.53%   83, 115, 119
R/variance_anhecova.R           44       0  100.00%
R/variance_hc.R                 10       0  100.00%
TOTAL                         1633      14  99.14%

Diff against main

Filename         Stmts    Miss  Cover
-------------  -------  ------  --------
R/mh_effect.R      +32       0  +100.00%
R/robin_mh.R      +205      +8  +96.10%
TOTAL             +237      +8  -0.43%

Results for commit: 30aeafc

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
robin_mh 👶 $+0.29$ $+31$ $+1$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
robin_mh 👶 $+0.01$ confint.mh_effect_returns_Wald_confidence_intervals_consistent_with_the_SE
robin_mh 👶 $+0.01$ robin_mh_accepts_a_custom_contrast_against_ref_
robin_mh 👶 $+0.01$ robin_mh_emits_a_warning_if_randomization_strata_are_not_analysis_strata
robin_mh 👶 $+0.02$ robin_mh_extends_to_multi_arm_pairwise_comparisons_and_is_consistent_with_two_arm_fits
robin_mh 👶 $+0.01$ robin_mh_handles_unstratified_analysis_y_1_and_reduces_to_plain_risk_diff
robin_mh 👶 $+0.14$ robin_mh_matches_reference_formulas_across_estimand_and_variance_choices
robin_mh 👶 $+0.01$ robin_mh_print_method_produces_the_expected_output
robin_mh 👶 $+0.03$ robin_mh_returns_mh_effect_with_the_expected_components
robin_mh 👶 $+0.02$ robin_mh_validates_inputs
robin_mh 👶 $+0.03$ table.mh_effect_returns_the_events_table_invisibly

Results for commit d13b8b1

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   14 suites   12s ⏱️
149 tests 102 ✅ 47 💤 0 ❌
372 runs  307 ✅ 65 💤 0 ❌

Results for commit 30aeafc.

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.

implement MH method

1 participant