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

Suppress warning in trialplanning.Rmd #108

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Suppress warning in trialplanning.Rmd #108

merged 1 commit into from
Jan 10, 2024

Conversation

holgstr
Copy link
Contributor

@holgstr holgstr commented Dec 27, 2023

Fixes #107

@holgstr holgstr added the bug Something isn't working label Dec 27, 2023
@holgstr holgstr self-assigned this Dec 27, 2023
Copy link
Contributor

badge

Code Coverage Summary

Filename                     Stmts    Miss  Cover    Missing
-------------------------  -------  ------  -------  ---------
R/assertions.R                  15       0  100.00%
R/corPFSOS.R                    95      19  80.00%   317-335
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          157       0  100.00%
R/transitionParameters.R        42       0  100.00%
TOTAL                          821      23  97.20%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: 311fcad

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

Unit Tests Summary

  1 files   16 suites   2m 11s ⏱️
104 tests 103 ✅ 1 💤 0 ❌
230 runs  229 ✅ 1 💤 0 ❌

Results for commit 311fcad.

Copy link
Collaborator

@danielinteractive danielinteractive left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @holgstr 🙏

@holgstr holgstr merged commit aa1f7ce into main Jan 10, 2024
23 checks passed
@holgstr holgstr deleted the issue107-warning branch January 10, 2024 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warning displayed in Trial Planning Vignette
2 participants