Skip to content

Commit 0dad5c5

Browse files
Merge pull request datashield#578 from StuartWheater/v6.3.2-test
V6.3.2 test
2 parents 6d66ea5 + 07e32d9 commit 0dad5c5

175 files changed

Lines changed: 699 additions & 567 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.Rbuildignore

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@
1717
^R/secure.global.ranking.md$
1818
^_pkgdown\.yml$
1919
^docs$
20-
^dsBase_6.3.1.tar.gz$
21-
^dsBase_6.3.1-permissive.tar.gz$
20+
^dsBase_6.3.2.tar.gz$
21+
^dsBase_6.3.2-permissive.tar.gz$
2222
^dsDanger_6.3.1.tar.gz$
2323
^\.circleci$
2424
^\.circleci/config\.yml$

.circleci/config.yml

Lines changed: 18 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,8 @@ version: 2.1
55
jobs:
66
dsbaseclient:
77
docker:
8-
- image: cimg/base:current
8+
# - image: cimg/base:current
9+
- image: cimg/base:2024.11
910
resource_class: small
1011
steps:
1112
- checkout
@@ -18,6 +19,10 @@ jobs:
1819
echo " Tag: " $CIRCLE_TAG
1920
- run:
2021
command: |
22+
sudo apt-get install --no-install-recommends software-properties-common dirmngr
23+
wget -qO- https://cloud.r-project.org/bin/linux/ubuntu/marutter_pubkey.asc | sudo tee -a /etc/apt/trusted.gpg.d/cran_ubuntu_key.asc
24+
sudo add-apt-repository -y "deb https://cloud.r-project.org/bin/linux/ubuntu $(lsb_release -cs)-cran40/"
25+
2126
sudo apt-get update -y
2227
sudo apt-get install -y r-base-core cmake
2328
- run:
@@ -33,6 +38,9 @@ jobs:
3338
sudo apt-get install -y libjpeg-dev
3439
- run:
3540
command: |
41+
sudo Rscript -e "install.packages('devtools', dependencies=TRUE)"
42+
sudo Rscript -e "install.packages('covr', dependencies=TRUE)"
43+
3644
sudo Rscript -e "install.packages('fields', dependencies=TRUE)"
3745
sudo Rscript -e "install.packages('metafor', dependencies=TRUE)"
3846
sudo Rscript -e "install.packages('meta', dependencies=TRUE)"
@@ -42,6 +50,15 @@ jobs:
4250
sudo Rscript -e "install.packages('panelaggregation', dependencies=TRUE)"
4351
sudo Rscript -e "install.packages('methods', dependencies=TRUE)"
4452
sudo Rscript -e "install.packages('dplyr', dependencies=TRUE)"
53+
54+
sudo Rscript -e "install.packages('DSI', dependencies=TRUE)"
55+
sudo Rscript -e "install.packages('DSOpal', dependencies=TRUE)"
56+
sudo Rscript -e "install.packages('DSLite', dependencies=TRUE)"
57+
sudo Rscript -e "install.packages('MolgenisAuth', dependencies=TRUE)"
58+
sudo Rscript -e "install.packages('MolgenisArmadillo', dependencies=TRUE)"
59+
sudo Rscript -e "install.packages('DSMolgenisArmadillo', dependencies=TRUE)"
60+
sudo Rscript -e "install.packages('DescTools', dependencies=TRUE)"
61+
sudo Rscript -e "install.packages('e1071', dependencies=TRUE)"
4562
- run:
4663
command: |
4764
sudo Rscript -e 'library(covr); covr::codecov(token = "'$CODECOV_TOKEN'", quiet=FALSE)'

DESCRIPTION

Lines changed: 46 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,42 @@
11
Package: dsBaseClient
22
Title: DataSHIELD Client Functions
3-
Version: 6.3.1
4-
Author: DataSHIELD Developers <datashield@liverpool.ac.uk>
5-
Maintainer: DataSHIELD Developers <datashield@liverpool.ac.uk>
3+
Version: 6.3.2
4+
Authors@R: c(person(given = "Paul",
5+
family = "Burton",
6+
role = c("aut")),
7+
person(given = "Rebecca",
8+
family = "Wilson",
9+
role = c("aut")),
10+
person(given = "Olly",
11+
family = "Butters",
12+
role = c("aut")),
13+
person(given = "Patricia",
14+
family = "Ryser-Welch",
15+
role = c("aut")),
16+
person(given = "Alex",
17+
family = "Westerberg",
18+
role = c("aut")),
19+
person(given = "Leire",
20+
family = "Abarrategui",
21+
role = c("aut")),
22+
person(given = "Roberto",
23+
family = "Villegas-Diaz",
24+
role = c("aut"),
25+
comment = c(ORCID = "0000-0001-5036-8661")),
26+
person(given = "Demetris",
27+
family = "Avraam",
28+
role = c("aut"),
29+
comment = c(ORCID = "0000-0001-8908-2441")),
30+
person(given = "Yannick",
31+
family = "Marcon",
32+
role = c("aut", "cre"),
33+
email = "yannick.marcon@obiba.org",
34+
comment = c(ORCID = "0000-0003-0138-2023")),
35+
person(given = "Stuart",
36+
family = "Wheater",
37+
role = c("aut", "cre"),
38+
email = "stuart.wheater@arjuna.com",
39+
comment = c(ORCID = "0009-0003-2419-1964")))
640
Description: DataSHIELD client functions for the client side.
741
License: GPL-3
842
Depends:
@@ -19,6 +53,14 @@ Imports:
1953
methods,
2054
dplyr
2155
Suggests:
22-
testthat
56+
lme4,
57+
httr,
58+
tibble,
59+
testthat,
60+
e1071,
61+
DescTools,
62+
DSOpal,
63+
DSMolgenisArmadillo,
64+
DSLite
2365
RoxygenNote: 7.3.2
2466
Encoding: UTF-8

R/ds.glm.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
#'
1818
#' Many GLMs can be fitted very simply using a formula such as:
1919
#'
20-
#' \deqn{y~a+b+c+d}
20+
#' \eqn{y~a+b+c+d}
2121
#'
2222
#' which simply means fit a GLM with \code{y} as the outcome variable and
2323
#' \code{a}, \code{b}, \code{c} and \code{d} as covariates.
@@ -26,7 +26,7 @@
2626
#' Instead, if you need to fit a more complex
2727
#' model, for example:
2828
#'
29-
#' \deqn{EVENT~1+TID+SEXF*AGE.60}
29+
#' \eqn{EVENT~1+TID+SEXF*AGE.60}
3030
#'
3131
#' In the above model the outcome variable is \code{EVENT}
3232
#' and the covariates

R/ds.glmSLMA.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
#'
6262
#' Many glms can be fitted very simply using a formula such as:
6363
#'
64-
#' \deqn{y~a+b+c+d}
64+
#' \eqn{y~a+b+c+d}
6565
#'
6666
#' which simply means fit a glm with \code{y} as the outcome variable and
6767
#' \code{a}, \code{b}, \code{c} and \code{d} as covariates.
@@ -70,7 +70,7 @@
7070
#' Instead, if you need to fit a more complex
7171
#' model, for example:
7272
#'
73-
#' \deqn{EVENT~1+TID+SEXF*AGE.60}
73+
#' \eqn{EVENT~1+TID+SEXF*AGE.60}
7474
#'
7575
#' In the above model the outcome variable is \code{EVENT}
7676
#' and the covariates

R/ds.glmerSLMA.R

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,17 +18,17 @@
1818
#' In \code{formula} most shortcut notation allowed by \code{glmer()} function is
1919
#' also allowed by \code{ds.glmerSLMA}.
2020
#' Many GLMEs can be fitted very simply using a formula like:
21-
#' \deqn{y~a+b+(1|c)}
21+
#' \eqn{y~a+b+(1|c)}
2222
#' which simply means fit an GLME with \code{y} as the outcome variable (e.g.
2323
#' a binary case-control using a logistic regression model or a count or a survival
2424
#' time using a Poisson regression model), \code{a} and \code{b}
2525
#' as fixed effects, and \code{c} as a random effect or grouping factor.
2626
#'
2727
#' It is also possible to fit models with random slopes by specifying a model such as
28-
#' \deqn{y~a+b+(1+b|c)}
28+
#' \eqn{y~a+b+(1+b|c)}
2929
#' where the effect of \code{b} can vary randomly between groups defined by \code{c}.
3030
#' Implicit nesting can be specified with formulas such as: \eqn{y~a+b+(1|c/d)}
31-
#' or \eqn{y~a+b+(1|c)+(1|c:d)}.
31+
#' or \eqn{y~a+b+(1|c)+(1|c:d)}.
3232
#'
3333
#'
3434
#' The \code{dataName} argument avoids you having to specify the name of the

R/ds.lmerSLMA.R

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@
1818
#'
1919
#' In \code{formula} most shortcut notation allowed by \code{lmer()} function is
2020
#' also allowed by \code{ds.lmerSLMA}. Many LMEs can be fitted very simply using a formula like:
21-
#' \deqn{y ~ a + b + (1 | c)}
21+
#' \eqn{y ~ a + b + (1 | c)}
2222
#' which simply means fit an LME with \code{y} as the outcome variable with \code{a} and \code{b}
2323
#' as fixed effects, and \code{c} as a random effect or grouping factor.
2424
#'
2525
#' It is also possible to fit models with random slopes by specifying a model such as
26-
#' \deqn{y ~ a + b + (1 + b | c)}
26+
#' \eqn{y ~ a + b + (1 + b | c)}
2727
#' where the effect of \code{b} can vary randomly between groups defined by \code{c}.
2828
#' Implicit nesting can be specified with formulae such as \eqn{y ~ a + b + (1 | c / d)}
2929
#' or \eqn{y ~ a + b + (1 | c) + (1 | c : d)}.

R/ds.rBinom.R

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,10 +216,11 @@ single.integer.seed<-c(single.integer.seed,seed.as.integer.study.specific)
216216
if(seed.as.text=="NULL"){
217217
cat("NO SEED SET IN STUDY",study.id,"\n\n")
218218

219-
}
219+
} else {
220220
calltext <- paste0("setSeedDS(", seed.as.text, ")")
221221
ssDS.obj[[study.id]] <- DSI::datashield.aggregate(datasources[study.id], as.symbol(calltext))
222222
}
223+
}
223224
cat("\n\n")
224225

225226

R/ds.rUnif.R

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,9 +234,10 @@ single.integer.seed<-c(single.integer.seed,seed.as.integer.study.specific)
234234
if(seed.as.text=="NULL"){
235235
cat("NO SEED SET IN STUDY",study.id,"\n")
236236

237-
}
237+
} else {
238238
calltext <- paste0("setSeedDS(", seed.as.text, ")")
239239
ssDS.obj[[study.id]] <- DSI::datashield.aggregate(datasources[study.id], as.symbol(calltext))
240+
}
240241
}
241242

242243

armadillo_azure-pipelines.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -58,10 +58,10 @@ schedules:
5858
- master
5959
always: true
6060
- cron: "0 2 * * *"
61-
displayName: Nightly build - v6.3.1-dev
61+
displayName: Nightly build - v6.3.2-dev
6262
branches:
6363
include:
64-
- v6.3.1-dev
64+
- v6.3.2-dev
6565
always: true
6666

6767
#########################################################################################
@@ -185,7 +185,7 @@ jobs:
185185
# If this step fails still mark as failed, but don't stop the rest of the steps running.
186186
- bash: |
187187
188-
R -q -e "library('devtools'); devtools::check(args = c('--no-examples'))" | tee azure-pipelines_check.Rout
188+
R -q -e "library('devtools'); devtools::check(args = c('--no-examples', '--no-tests'))" | tee azure-pipelines_check.Rout
189189
grep --quiet "^0 errors" azure-pipelines_check.Rout && grep --quiet " 0 warnings" azure-pipelines_check.Rout && grep --quiet " 0 notes" azure-pipelines_check.Rout
190190
191191
workingDirectory: $(Pipeline.Workspace)/dsBaseClient
@@ -235,7 +235,7 @@ jobs:
235235
236236
curl -u admin:admin -X GET http://localhost:8080/packages
237237
238-
curl -u admin:admin --max-time 300 -v -H 'Content-Type: multipart/form-data' -F "file=@dsBase_6.3.1-permissive.tar.gz" -X POST http://localhost:8080/install-package
238+
curl -u admin:admin --max-time 300 -v -H 'Content-Type: multipart/form-data' -F "file=@dsBase_6.3.2-permissive.tar.gz" -X POST http://localhost:8080/install-package
239239
sleep 60
240240
241241
docker container restart dsbaseclient_armadillo_1
@@ -263,7 +263,7 @@ jobs:
263263
# best guess is that there is an implicit build or similar that happens. Although
264264
# I cannot replicate that directly with build etc directly.
265265
266-
sudo R --verbose -e 'devtools::check()'
266+
sudo R --verbose -e 'devtools::reload()'
267267
268268
mkdir $(Pipeline.Workspace)/logs
269269
@@ -364,7 +364,7 @@ jobs:
364364
- bash: |
365365
curl -u admin:admin http://localhost:8080/whitelist
366366
367-
curl -u admin:admin -v -H 'Content-Type: multipart/form-data' -F "file=@dsDanger_6.3.1.tar.gz" -X POST http://localhost:8080/install-package
367+
curl -u admin:admin -v -H 'Content-Type: multipart/form-data' -F "file=@dsDanger_6.3.2.tar.gz" -X POST http://localhost:8080/install-package
368368
369369
docker container restart dsbaseclient_armadillo_1
370370
sleep 60
@@ -387,7 +387,7 @@ jobs:
387387
- bash: |
388388
389389
# See, 'Code coverage and JUnit report output' for issues with the approach and improvement needed.
390-
sudo R --verbose -e 'devtools::check()'
390+
sudo R --verbose -e 'devtools::reload()'
391391
392392
pwd
393393
mkdir $(Pipeline.Workspace)/logs

0 commit comments

Comments
 (0)