Closed bzkrouse closed 2 months ago
Filename Stmts Miss Cover Missing
--------------------------- ------- ------ ------- --------------------------
R/apply_fmt_fn.R 95 24 74.74% 36-43, 109-116, 198-205
R/ard_attributes.R 35 1 97.14% 56
R/ard_categorical.R 342 16 95.32% 233, 393-394, 550-565, 599
R/ard_complex.R 34 0 100.00%
R/ard_continuous.R 176 4 97.73% 118, 200-201, 299
R/ard_dichotomous.R 80 8 90.00% 62, 150-155, 178
R/ard_hierarchical.R 83 0 100.00%
R/ard_missing.R 56 0 100.00%
R/ard_stack.R 84 7 91.67% 107, 186-191
R/ard_total_n.R 7 0 100.00%
R/as_nested_list.R 41 0 100.00%
R/bind_ard.R 22 3 86.36% 51-53
R/check_ard_structure.R 39 6 84.62% 31, 55-59
R/default_stat_labels.R 18 0 100.00%
R/eval_capture_conditions.R 18 0 100.00%
R/get_ard_statistics.R 16 1 93.75% 75
R/nest_for_ard.R 71 1 98.59% 60
R/print_ard_conditions.R 75 66 12.00% 60-176
R/print.R 72 0 100.00%
R/process_selectors.R 125 1 99.20% 334
R/replace_null_statistic.R 11 0 100.00%
R/round5.R 1 0 100.00%
R/selectors.R 14 0 100.00%
R/shift_ard_columns.R 77 13 83.12% 61, 69, 144-147, 150-156
R/shuffle_ard.R 102 0 100.00%
R/summary_functions.R 25 1 96.00% 59
R/tidy_ard_order.R 25 0 100.00%
R/tidy_as_ard.R 39 11 71.79% 88-98
R/utils.R 24 0 100.00%
TOTAL 1807 163 90.98%
Filename Stmts Miss Cover
---------- ------- ------ --------
TOTAL 0 0 +100.00%
Results for commit: e3184e4fb6063f09d1ebd6ee1ba41e6060077d72
Minimum allowed coverage is 80%
:recycle: This comment has been updated with latest results
1 files 90 suites 13s :stopwatch: 88 tests 55 :white_check_mark: 33 :zzz: 0 :x: 178 runs 130 :white_check_mark: 48 :zzz: 0 :x:
Results for commit e3184e4f.
:recycle: This comment has been updated with latest results.
@bzkrouse this is great, thank you for adding!
What do you think about simplifying the AE table to show either the patient-level OR the AE counts? I think the FDA specs separate them, right?
@bzkrouse this is great, thank you for adding!
What do you think about simplifying the AE table to show either the patient-level OR the AE counts? I think the FDA specs separate them, right?
@ddsjoberg Made this update, please let me know what you think!
What changes are proposed in this pull request?
NEWS.md
. (#Provide more detail here as needed.
Reference GitHub issue associated with pull request. e.g., 'closes #'
Resolves #209
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.md
with 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.md
for examples).