insightsengineering / cards

CDISC Analysis Results Data
https://insightsengineering.github.io/cards/
30 stars 2 forks source link

Function to rename ard columns #254

Closed bzkrouse closed 3 months ago

bzkrouse commented 4 months ago

What changes are proposed in this pull request?

Provide more detail here as needed.

Reference GitHub issue associated with pull request. e.g., 'closes #' Resolves #213


Pre-review Checklist (if item does not apply, mark is as complete)

Reviewer Checklist (if item does not apply, mark is as complete)

When the branch is ready to be merged:

github-actions[bot] commented 4 months ago

badge

Code Coverage Summary

Filename                       Stmts    Miss  Cover    Missing
---------------------------  -------  ------  -------  --------------------------
R/apply_fmt_fn.R                  94      23  75.53%   36-42, 109-116, 198-205
R/ard_attributes.R                33       0  100.00%
R/ard_categorical.R              348      16  95.40%   223, 383-384, 540-555, 589
R/ard_complex.R                   34       0  100.00%
R/ard_continuous.R               176       4  97.73%   118, 200-201, 299
R/ard_dichotomous.R               66       2  96.97%   60, 146
R/ard_hierarchical.R              83       0  100.00%
R/ard_missing.R                   21       0  100.00%
R/ard_stack.R                     70       1  98.57%   103
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             58       1  98.28%   91
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                           1773     149  91.60%

Diff against main

Filename                 Stmts    Miss  Cover
---------------------  -------  ------  -------
R/shift_ard_columns.R      +77     +13  +83.12%
R/shuffle_ard.R            -32      -1  +0.75%
TOTAL                      +45     +12  -0.48%

Results for commit: f2a7e6d6b9d4d73d97d7ac2c1b9e839477bf74d9

Minimum allowed coverage is 80%

:recycle: This comment has been updated with latest results

github-actions[bot] commented 4 months ago

Unit Tests Summary

  1 files   88 suites   13s :stopwatch:  86 tests  54 :white_check_mark: 32 :zzz: 0 :x: 180 runs  133 :white_check_mark: 47 :zzz: 0 :x:

Results for commit cc5f1ef8.

github-actions[bot] commented 4 months ago

Unit Tests Summary

  1 files   88 suites   13s :stopwatch:  86 tests  54 :white_check_mark: 32 :zzz: 0 :x: 176 runs  129 :white_check_mark: 47 :zzz: 0 :x:

Results for commit f2a7e6d6.

:recycle: This comment has been updated with latest results.

github-actions[bot] commented 4 months ago

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
shift_ard_columns 👶 $+0.32$ $+2$ $+1$ $0$ $0$
Additional test case details | Test Suite | $Status$ | Time on `main` | $±Time$ | Test Case | |:-----|:----:|:----:|:----:|:-----| | shift_ard_columns | 👶 | | $+0.32$ | rename_ard_columns_works |

Results for commit 0e0507a40f3c23e08d155f9314b6b6fc5c8b21d2

♻️ This comment has been updated with latest results.