Skip to content

Commit 6f3797c

Browse files
committed
improve software/setParameters
1 parent d7427f7 commit 6f3797c

File tree

2 files changed

+5
-7
lines changed

2 files changed

+5
-7
lines changed

R/geoflow_software.R

+4-6
Original file line numberDiff line numberDiff line change
@@ -122,9 +122,8 @@ geoflow_software <- R6Class("geoflow_software",
122122
},
123123

124124
#'@description Set properties. Function to call to pass argument values for a given \code{geoflow_software}
125-
#'@param ... named list of properties
126-
setProperties = function(...){
127-
props <- list(...)[[1]]
125+
#'@param props named list of properties
126+
setProperties = function(props){
128127
propNames <- names(props)
129128
if(length(propNames)>0){
130129
for(propName in propNames){
@@ -144,9 +143,8 @@ geoflow_software <- R6Class("geoflow_software",
144143
},
145144

146145
#'@description Set parameters. Function to call to pass argument values for a given \code{geoflow_software}
147-
#'@param ... named list of parameters
148-
setParameters = function(...){
149-
params <- list(...)[[1]]
146+
#'@param params named list of parameters
147+
setParameters = function(params){
150148
paramNames <- names(params)
151149
if(length(paramNames)>0){
152150
for(paramName in paramNames){

R/initWorkflow.R

+1-1
Original file line numberDiff line numberDiff line change
@@ -221,7 +221,7 @@ initWorkflow <- function(file, dir = ".", jobDirPath = NULL, handleMetadata = TR
221221
config$logger.info(sprintf("Configuring %s software '%s' (%s)", software$type, software$id, software$software_type))
222222
target_software$setId(software$id)
223223
target_software$setType(software$type)
224-
if(!is.null(software$parameters)) target_software$setParameters(unlist(software$parameters))
224+
if(!is.null(software$parameters)) target_software$setParameters(software$parameters)
225225

226226
#check software dependencies
227227
target_software$checkPackages()

0 commit comments

Comments
 (0)