Skip to content

Commit

Permalink
CRAN submission v1.1.3
Browse files Browse the repository at this point in the history
  • Loading branch information
mikolajsp committed Sep 6, 2023
1 parent 2ea3365 commit f1b2d34
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 27 deletions.
2 changes: 2 additions & 0 deletions .Rbuildignore
Original file line number Diff line number Diff line change
Expand Up @@ -12,3 +12,5 @@ LICENSE
^misc$
^cran-comments\.md$
^vignettes/articles$
^tests$
^CRAN-SUBMISSION$
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: survex
Title: Explainable Machine Learning in Survival Analysis
Version: 1.1.1
Version: 1.1.3
Authors@R:
c(
person("Mikołaj", "Spytek", email = "[email protected]", role = c("aut", "cre"), comment = c(ORCID = "0000-0001-7111-2286")),
Expand Down
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# survex 1.1.0
# survex 1.1.3

* fixed not being able to plot or print SurvLIME results for the cph model sometimes. ([#72](https://github.com/ModelOriented/survex/issues/72))
* added global explanations via the SurvSHAP(t) method (see `model_survshap()` function)
Expand Down
6 changes: 0 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -20,12 +20,6 @@ Note that an explainer can be created for **any** survival model, using the `exp

## Installation

```diff
! Please note that that version v1.1.1 with new functionalities is still being processed by CRAN.
+ To use all functionalities, for the time being, please install the most current version from GitHub!
```


The package is available on [CRAN](https://cran.r-project.org/package=survex):

```r
Expand Down
23 changes: 4 additions & 19 deletions cran-comments.md
Original file line number Diff line number Diff line change
@@ -1,25 +1,10 @@
## Resubmission 30/08/2023 v.1.1.1
## Resubmission 05/09/2023 v.1.1.3

* Fix notes about long running examples by wrapping
in \donttest{} blocks.
* Previous submission (v.1.1.0) also had the following
notes from `r-devel-linux-x86_64-debian-gcc` flavor
check:
- Running R code in 'testthat.R' had CPU time
7.6 times elapsed time
- Re-building vignettes had CPU time
10.8 times elapsed time
I could not replicate these in any other environment:
- locally using R CMD Check (Windows),
- on r_hub (Windows, macos, and linux)
- using R CMD Check in GitHub Actions runners
(Windows, Ubuntu, macos)
- using `devtools::check_win_devel()` and
`check_macos_release()`
I believe these to be false positives likely due to
hardware or current load at the time of the check
in case they are not, please provide advice how to fix.

* Fix notes about CPU time being much longer than elapsed
time when running tests and building vignettes.

## R CMD check results

0 errors | 0 warnings | 1 note
Expand Down

0 comments on commit f1b2d34

Please sign in to comment.