Skip to content

Commit c6f475a

Browse files
committed
pre-release updates
1 parent 243f90f commit c6f475a

File tree

8 files changed

+137
-50
lines changed

8 files changed

+137
-50
lines changed

.gitignore

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,5 +16,5 @@ temp/
1616
tests/testthat/_snaps/**/*.new.md
1717
tests/testthat/_snaps/**/*.new.svg
1818

19-
# reverse dep cehcks
19+
# reverse dep checks
2020
checked/

DESCRIPTION

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ BugReports: https://github.com/insightsengineering/cardx/issues
2020
Depends:
2121
R (>= 4.2)
2222
Imports:
23-
cards (>= 0.6.0.9011),
23+
cards (>= 0.6.1),
2424
cli (>= 3.6.1),
2525
dplyr (>= 1.1.2),
2626
glue (>= 1.6.2),
@@ -44,8 +44,6 @@ Suggests:
4444
survival (>= 3.6-4),
4545
testthat (>= 3.2.0),
4646
withr (>= 2.5.0)
47-
Remotes:
48-
insightsengineering/cards
4947
Config/Needs/website: insightsengineering/nesttemplate
5048
Config/testthat/edition: 3
5149
Config/testthat/parallel: true

NEWS.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
11
# cardx 0.2.4.9008
22

3+
* Adding `strata` argument to `ard_categorical_max()`. (#445, @jtalboys)
4+
5+
* Added function `ard_incidence_rate()` to calculate ARDs for incidence rate estimation. (#234)
6+
37
* Updating any `fmt_fn` references to `fmt_fun` for consistency.
48

59
* Any function with an argument `cardx::foo(fmt_fn)` has been updated to `cardx::foo(fmt_fun)`. The old syntax will continue to function, but with a deprecation warning to users.
610

711
* Importantly, the ARD column named `"fmt_fn"` has been updated to `"fmt_fun"`. This change cannot be formally deprecated. For users who were accessing the ARD object directly to modify this column instead of using functions like `cards::update_ard_fmt_fun()`, this will be a breaking change.
812

9-
* Adding `strata` argument to `ard_categorical_max()`. (#445, @jtalboys)
10-
11-
* Added function `ard_incidence_rate()` to calculate ARDs for incidence rate estimation. (#234)
12-
1313
* Fix in `ard_survival_survfit.data.frame()` method where the stratifying variable was not correctly converted back to its original type.
1414

1515
* Fix in `ard_total_n.survey.design()` to use `update()` instead of `dplyr::mutate()`, which sometimes caused a downstream issue.

README.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ cards::ADSL |>
7777
## 13 ARM AGE stats_t_… var.equal Equal Va… FALSE
7878
## 14 ARM AGE stats_t_… conf.level CI Confi… 0.95
7979

80-
## ℹ 3 more variables: fmt_fn, warning, error
80+
## ℹ 3 more variables: fmt_fun, warning, error
8181

8282
Note that the returned ARD contains the analysis results in addition to
8383
the function parameters used to calculate the results allowing for
@@ -108,13 +108,13 @@ construct_model(
108108

109109
## {cards} data frame: 6 x 8
110110

111-
## variable context stat_name stat_label stat fmt_fn
112-
## 1 (Intercept) aod_wald… df Degrees … 1 1
113-
## 2 (Intercept) aod_wald… statistic Statistic 7126.713 1
114-
## 3 (Intercept) aod_wald… p.value p-value 0 1
115-
## 4 ARM aod_wald… df Degrees … 2 1
116-
## 5 ARM aod_wald… statistic Statistic 1.046 1
117-
## 6 ARM aod_wald… p.value p-value 0.593 1
111+
## variable context stat_name stat_label stat fmt_fun
112+
## 1 (Intercept) aod_wald… df Degrees … 1 1
113+
## 2 (Intercept) aod_wald… statistic Statistic 7126.713 1
114+
## 3 (Intercept) aod_wald… p.value p-value 0 1
115+
## 4 ARM aod_wald… df Degrees … 2 1
116+
## 5 ARM aod_wald… statistic Statistic 1.046 1
117+
## 6 ARM aod_wald… p.value p-value 0.593 1
118118

119119
## ℹ 2 more variables: warning, error
120120

cran-comments.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,11 @@
1010

1111
## revdepcheck results
1212

13-
We checked 4 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package.
13+
We checked 6 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package.
1414

1515
* We saw 1 new problem in the 'gtsummary' package. I maintain this package and the break is expected. I will submit a new version as soon as 'cardx' is on CRAN.
16+
17+
* We failed to check 0 packages
1618

1719
## Additional Comments
1820

revdep/README.md

Lines changed: 43 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -1,38 +1,51 @@
11
# Platform
22

3-
|field |value |
4-
|:--------|:------------------------------------------|
5-
|version |R version 4.4.2 (2024-10-31) |
6-
|os |macOS Ventura 13.6.9 |
7-
|system |aarch64, darwin20 |
8-
|ui |RStudio |
9-
|language |(EN) |
10-
|collate |en_US.UTF-8 |
11-
|ctype |en_US.UTF-8 |
12-
|tz |America/Los_Angeles |
13-
|date |2025-02-17 |
14-
|rstudio |2024.12.0+467.pro1 Kousa Dogwood (desktop) |
15-
|pandoc |3.4 @ /usr/local/bin/pandoc |
3+
|field |value |
4+
|:--------|:--------------------------------------------|
5+
|version |R version 4.5.0 (2025-04-11) |
6+
|os |macOS Ventura 13.6.9 |
7+
|system |aarch64, darwin20 |
8+
|ui |RStudio |
9+
|language |(EN) |
10+
|collate |en_US.UTF-8 |
11+
|ctype |en_US.UTF-8 |
12+
|tz |America/Los_Angeles |
13+
|date |2025-07-03 |
14+
|rstudio |2025.05.0+496.pro5 Mariposa Orchid (desktop) |
15+
|pandoc |3.4 @ /usr/local/bin/pandoc |
1616

1717
# Dependencies
1818

19-
|package |old |new |Δ |
20-
|:----------|:-----|:----------|:--|
21-
|cardx |0.2.2 |0.2.2.9023 |* |
22-
|cards |0.4.0 |0.5.0.9000 |* |
23-
|cli |NA |3.6.4 |* |
24-
|cpp11 |NA |0.5.1 |* |
25-
|fansi |NA |1.0.6 |* |
26-
|glue |NA |1.8.0 |* |
27-
|pillar |NA |1.10.1 |* |
28-
|purrr |NA |1.0.4 |* |
29-
|R6 |NA |2.6.1 |* |
30-
|rlang |NA |1.1.5 |* |
31-
|stringi |NA |1.8.4 |* |
32-
|tidyr |NA |1.3.1 |* |
33-
|tidyselect |NA |1.2.1 |* |
34-
|vctrs |NA |0.6.5 |* |
35-
|withr |NA |3.0.2 |* |
19+
|package |old |new |Δ |
20+
|:----------|:------|:----------|:--|
21+
|cardx |0.2.4 |0.2.4.9008 |* |
22+
|cards |0.6.0 |0.6.1.9000 |* |
23+
|cli |3.6.5 |3.6.5 | |
24+
|cpp11 |0.5.2 |0.5.2 | |
25+
|dplyr |1.1.4 |1.1.4 | |
26+
|generics |0.1.4 |0.1.4 | |
27+
|glue |1.8.0 |1.8.0 | |
28+
|lifecycle |1.0.4 |1.0.4 | |
29+
|magrittr |2.0.3 |2.0.3 | |
30+
|pillar |1.10.2 |1.10.2 | |
31+
|pkgconfig |2.0.3 |2.0.3 | |
32+
|purrr |1.0.4 |1.0.4 | |
33+
|R6 |2.6.1 |2.6.1 | |
34+
|rlang |1.1.6 |1.1.6 | |
35+
|stringi |1.8.7 |1.8.7 | |
36+
|stringr |1.5.1 |1.5.1 | |
37+
|tibble |3.3.0 |3.3.0 | |
38+
|tidyr |1.3.1 |1.3.1 | |
39+
|tidyselect |1.2.1 |1.2.1 | |
40+
|utf8 |1.2.6 |1.2.6 | |
41+
|vctrs |0.6.5 |0.6.5 | |
42+
|withr |3.0.2 |3.0.2 | |
3643

3744
# Revdeps
3845

46+
## New problems (1)
47+
48+
|package |version |error |warning |note |
49+
|:---------|:-------|:------|:-------|:----|
50+
|[gtsummary](problems.md#gtsummary)|2.2.0 |__+2__ | | |
51+

revdep/cran.md

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,16 @@
11
## revdepcheck results
22

3-
We checked 4 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package.
3+
We checked 6 reverse dependencies, comparing R CMD check results across CRAN and dev versions of this package.
44

5-
* We saw 0 new problems
5+
* We saw 1 new problems
66
* We failed to check 0 packages
77

8+
Issues with CRAN packages are summarised below.
9+
10+
### New problems
11+
(This reports the first line of each new failure)
12+
13+
* gtsummary
14+
checking examples ... ERROR
15+
checking tests ...
16+

revdep/problems.md

Lines changed: 66 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,66 @@
1-
*Wow, no problems at all. :)*
1+
# gtsummary
2+
3+
<details>
4+
5+
* Version: 2.2.0
6+
* GitHub: https://github.com/ddsjoberg/gtsummary
7+
* Source code: https://github.com/cran/gtsummary
8+
* Date/Publication: 2025-04-14 10:30:02 UTC
9+
* Number of recursive dependencies: 204
10+
11+
Run `revdepcheck::revdep_details(, "gtsummary")` for more info
12+
13+
</details>
14+
15+
## Newly broken
16+
17+
* checking examples ... ERROR
18+
```
19+
Running examples in ‘gtsummary-Ex.R’ failed
20+
The error most likely occurred in:
21+
22+
> ### Name: add_ci
23+
> ### Title: Add CI Column
24+
> ### Aliases: add_ci add_ci.tbl_summary
25+
>
26+
> ### ** Examples
27+
>
28+
> ## Don't show:
29+
...
30+
29. │ └─dplyr:::mutate_col(dots[[i]], data, mask, new_columns)
31+
30. │ └─mask$eval_all_mutate(quo)
32+
31. │ └─dplyr (local) eval()
33+
32. ├─base::ifelse(...)
34+
33. ├─fmt_fn
35+
34. ├─rlang:::`$.rlang_data_pronoun`(.data, fmt_fn)
36+
35. │ └─rlang:::data_pronoun_get(...)
37+
36. └─rlang:::abort_data_pronoun(x, call = y)
38+
37. └─rlang::abort(msg, "rlang_error_data_pronoun_not_found", call = call)
39+
Execution halted
40+
```
41+
42+
* checking tests ...
43+
```
44+
Running ‘spelling.R’
45+
Comparing ‘spelling.Rout’ to ‘spelling.Rout.save’ ... OK
46+
Running ‘testthat.R’
47+
ERROR
48+
Running the tests in ‘tests/testthat.R’ failed.
49+
Last 13 lines of output:
50+
+ Condition
51+
+ Warning:
52+
+ The `fmt_fn` argument of `ard_categorical()` is deprecated as of cards 0.6.1.
53+
+ i Please use the `fmt_fun` argument instead.
54+
...
55+
+ The `fmt_fn` argument of `ard_dichotomous()` is deprecated as of cards 0.6.1.
56+
+ i Please use the `fmt_fun` argument instead.
57+
and 15 more ...
58+
59+
* Run `testthat::snapshot_accept('theme_gtsummary')` to accept the change.
60+
* Run `testthat::snapshot_review('theme_gtsummary')` to interactively review the change.
61+
62+
[ FAIL 233 | WARN 2367 | SKIP 0 | PASS 1361 ]
63+
Error: Test failures
64+
Execution halted
65+
```
66+

0 commit comments

Comments
 (0)