Skip to content

Commit 73aa275

Browse files
committed
fix issues with match on R devel
1 parent 4206fb8 commit 73aa275

File tree

4 files changed

+10
-12
lines changed

4 files changed

+10
-12
lines changed

DESCRIPTION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
Package: dynamichazard
22
Type: Package
33
Title: Dynamic Hazard Models using State Space Models
4-
Version: 1.0.0
4+
Version: 1.0.1
55
Authors@R: c(person("Benjamin", "Christoffersen",
66
email = "boennecd@gmail.com",
77
role = c("cre", "aut"),

NEWS.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
1+
# dynamichazard 1.0.1
2+
* fix issues with `match` after changes in R devel.
3+
4+
# dynamichazard 1.0.0
5+
* add citation information before JSS article is published.
6+
17
# dynamichazard 0.6.8
28
* fix issue due to changes in speedglm.
39
* add documentation and fix other issues raised by CRAN.

cran-comments.md

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
11
## Test environments
22
* Ubuntu 20.04 LTS with gcc 10.1.0
33
R version 4.1.1
4-
* Ubuntu 20.04 LTS with gcc 10.1.0
5-
R devel 2021-09-06 r80861 with LTO checks
64
* Ubuntu 20.04 LTS with gcc 10.1.0
75
R version 4.1.1 with valgrind
86
* Ubuntu 20.04 LTS with gcc 10.1.0
9-
R devel 2021-09-05 r80859 with ASAN and UBSAN
7+
R devel 2021-10-09 r81024 with ASAN and UBSAN
108
* Github actions on windows-latest (release), macOS-latest (release),
119
ubuntu-20.04 (release), and ubuntu-20.04 (devel)
1210
* win-builder (devel, oldrelease, and release)
@@ -17,9 +15,3 @@
1715
There were no WARNINGs or ERRORs.
1816

1917
There is a NOTE about the package size in some cases.
20-
21-
There is a NOTE about a 404 error for one of the DOIs in the description. This
22-
is for a Journal of Statistical Software paper that will be published soon.
23-
The paper is the reason for this update.
24-
25-
There is NOTE that Christoffersen is possibly misspelled. It is not.

tests/testthat/test_design_mat_and_risk_obj.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -580,7 +580,7 @@ test_that("Permutating gives the data frame and risk set permutated", {
580580
expect_equal(nrow(suppressMessages(plyr::match_df(X_Y_org[[i]], X_Y[[i]]))),
581581
nrow(X_Y_org[[i]]))
582582
}
583-
expect_true(sum(is.na(match(X_Y_org$Y, X_Y$Y))) == 0)
583+
expect_true(sum(is.na(match(unclass(X_Y_org$Y), unclass(X_Y$Y)))) == 0)
584584

585585
expect_true(setequal(w_org, w))
586586
expect_true(any(w_org != w))
@@ -641,7 +641,7 @@ test_that("Ordering gives the data frame and risk set orded", {
641641
nrow(X_Y_org[[i]]))
642642
}
643643
}
644-
expect_true(sum(is.na(match(X_Y_org$Y, X_Y$Y))) == 0)
644+
expect_true(sum(is.na(match(unclass(X_Y_org$Y), unclass(X_Y$Y)))) == 0)
645645

646646
expect_true(setequal(w_org, w))
647647
expect_true(any(w_org != w))

0 commit comments

Comments
 (0)