Bug #13613
closed
[R SDK] document minimum version of R required (and check at runtime)
100%
Description
test-copy.R
library(ArvadosR)
packageVersion("ArvadosR")
arv <- Arvados$new("api-token-redacted", "e51c5.arvadosapi.com", numRetries = 3)
previous_collection <- Collection$new(arv, "e51c5-4zz18-ko243dg1k5c6hid")
message("accessed previous collection")
previous_collection$getFileListing()
file_to_be_transferred <- previous_collection$get("foo")
bcosc@shell:~$ Rscript test-copy.R
[1] ‘0.0.4’
accessed previous collection
[1] "foo"
Error in trimFromStart(relativePath, "./") :
could not find function "startsWith"
Calls: <Anonymous> -> <Anonymous> -> trimFromStart
Execution halted
Updated by Bryan Cosca over 6 years ago
shell.e51c5 has R version 3.2.3 installed, and this should be fixed in 3.3.0
Updated by Tom Morris over 6 years ago
- Subject changed from R SDK collection$get errors with 'could not find function "startsWith"' to [R SDK] document minimum version of R required (and check at runtime)
- Assigned To set to Fuad Muhic
- Target version set to To Be Groomed
The startsWith method was new in R 3.3.0, so our use of it makes that the minimum R version.
We need to document that minimum and check at run time using getRVersion() or R.version$minor to make sure that our minimum is met.
Bioconductor requires the current R release (3.5 today), so that will become the requirement going forward when we're in Bioconductor.
Updated by Tom Morris over 6 years ago
- Target version changed from To Be Groomed to 2018-07-03 Sprint
Updated by Fuad Muhic over 6 years ago
- Status changed from New to In Progress
Users running R version less than 3.3.0 (for now) will get a warning message every time they try to load the package. Package will still load, however. I don't think we can prevent package from loading if version is less than required (I'm not sure if that's what we want anyway). You can test this feature by bumping minAllowedRVersion to "3.6.0" in file "sdk/R/R/zzz.R" on line 3. I also added a note in README file.
Updated by Fuad Muhic over 6 years ago
- Status changed from In Progress to Closed