Add pk helper functions - #289
Conversation
|
✅ All contributors have signed the CLA |
|
Hi @shajoezhu @wzfrank, Could you please help with the tests on the functions? Is there a certain format I need to follow? So some simple tests below are fine. ` test_that("geom_mean() returns NA for non-positive values", { test_that("fmt_3sig() formats correctly", { |
Unit Tests Summary 1 files 260 suites 2m 15s ⏱️ Results for commit 8dbb153. ♻️ This comment has been updated with latest results. |
Unit Test Performance DifferenceAdditional test case details
Results for commit 408ec74 ♻️ This comment has been updated with latest results. |
Code Coverage SummaryDiff against mainResults for commit: 8dbb153 Minimum allowed coverage is ♻️ This comment has been updated with latest results |
What changes are proposed in this pull request?
NEWS.md. (#, @)Provide more detail here as needed.
Add cv(), geom_cv(), geom_mean(), fmt_3sig() and impuation_rules for the PK TLGs.
Close #287
Pre-review Checklist (if item does not apply, mark is as complete)
usethis::pr_merge_main()devtools::test_coverage()Reviewer Checklist (if item does not apply, mark is as complete)
pkgdown::build_site(). Check the R console for errors, and review the rendered website.devtools::test_coverage()When the branch is ready to be merged:
NEWS.mdwith the changes from this pull request under the heading "# cards (development version)". If there is an issue associated with the pull request, reference it in parentheses at the end update (seeNEWS.mdfor examples).