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

92: Add unit tests for correlation functions #98

Merged
merged 7 commits into from
Dec 7, 2023

Conversation

danielinteractive
Copy link
Collaborator

closes #92

Copy link
Contributor

github-actions bot commented Dec 7, 2023

Unit Test Performance Difference

Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
corPFSOS 👶 $+0.01$ PFSOSInteg_works_as_expected
corPFSOS 👶 $+0.01$ PFSOSInteg_works_as_expected_for_vectors
corPFSOS 👶 $+0.16$ corTrans_works_as_expected
corPFSOS 👶 $+0.01$ expvalOSInteg_works_as_expected
corPFSOS 👶 $+0.01$ expvalPFSInteg_works_as_expected
corPFSOS 👶 $+0.01$ log_p11_works_as_expected
corPFSOS 👶 $+0.01$ log_p11_works_as_expected_for_multiple_time_points
corPFSOS 👶 $+0.01$ p11Integ_works_as_expected
corPFSOS 👶 $+0.01$ survOS_works_as_expected_for_Weibull_transition_hazards
corPFSOS 👶 $+0.01$ survOS_works_as_expected_for_exponential_transition_hazards
corPFSOS 👶 $+0.01$ survOS_works_as_expected_for_piecewise_exponential_transition_hazards
corPFSOS 👶 $+0.02$ survPFSOS_works_as_expected_for_Weibull_transitions
corPFSOS 👶 $+0.02$ survPFSOS_works_as_expected_for_exponential_transition
corPFSOS 👶 $+0.03$ survPFSOS_works_as_expected_for_piecewise_constant_hazard_transitions
corPFSOS 👶 $+0.01$ survPFS_works_as_expected_for_Weibull_transition_hazards
corPFSOS 👶 $+0.03$ survPFS_works_as_expected_for_exponential_transition_hazards
corPFSOS 👶 $+0.01$ survPFS_works_as_expected_for_piecewise_exponential_transition_hazards

Results for commit 758b570

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Dec 7, 2023

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  ---------
R/assertions.R                  15       0  100.00%
R/corPFSOS.R                    95      19  80.00%   314-332
R/empSignificant.R              59       0  100.00%
R/estimateParams.R              80       0  100.00%
R/eventTracking.R               88       2  97.73%   48, 244
R/getClinicalTrials.R           71       0  100.00%
R/getSimulatedData.R           109       0  100.00%
R/getWaitTimeSum.R              29       2  93.10%   45, 48
R/hazardFunctions.R             26       0  100.00%
R/piecewiseDistribution.R       32       0  100.00%
R/piecewiseHazards.R            18       0  100.00%
R/survivalFunctions.R          111       0  100.00%
R/transitionParameters.R        42       0  100.00%
TOTAL                          775      23  97.03%

Diff against main

Filename              Stmts    Miss  Cover
------------------  -------  ------  -------
R/corPFSOS.R             +4     -18  +20.66%
R/estimateParams.R       +5      -4  +5.33%
TOTAL                    +9     -22  +2.91%

Results for commit: abe6d1b

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Dec 7, 2023

Unit Tests Summary

    1 files    16 suites   2m 14s ⏱️
  98 tests   97 ✔️ 1 💤 0
219 runs  218 ✔️ 1 💤 0

Results for commit abe6d1b.

♻️ This comment has been updated with latest results.

@danielinteractive danielinteractive merged commit bbe7658 into main Dec 7, 2023
25 checks passed
@danielinteractive danielinteractive deleted the 92_test_cor_funs branch December 7, 2023 15:45
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.

Add unit tests for correlation functions
2 participants