MarcLavielle / mlxR

mlxR
Other
18 stars 5 forks source link

no visible bindings in readdatamlx #2

Closed romainfrancois closed 9 years ago

romainfrancois commented 9 years ago

R CMD check gives:

readdatamlx: no visible binding for global variable ‘icov’
readdatamlx: no visible binding for global variable ‘icat’
readdatamlx: no visible binding for global variable ‘iid’
readdatamlx: no visible binding for global variable ‘iamt’
readdatamlx: no visible binding for global variable ‘iy’
readdatamlx: no visible binding for global variable ‘iytype’

Ok, I'm assuming all these are created within this loop:

  for (i in 1:length(headerList))
  {
    hi=headerList[[i]]
    ih <- which(header %in% hi)
    if (length(ih)==0)
      ih=NULL
    ii <- paste0( 'i', tolower(hi) )
    eval(parse(text= paste0(ii,"=ih")))
    if (!is.null(ih))
      newHeader[ih]=newList[i]      
  }

through the eval(parse()) trick. This is not going to fly.

MarcLavielle commented 9 years ago

is it ok if we initialize these variables before the loop

icov=NULL icat=NULL ...

?

2014-12-31 12:10 GMT+01:00 Romain François notifications@github.com:

R CMD check gives:

readdatamlx: no visible binding for global variable 'icov' readdatamlx: no visible binding for global variable 'icat' readdatamlx: no visible binding for global variable 'iid' readdatamlx: no visible binding for global variable 'iamt' readdatamlx: no visible binding for global variable 'iy' readdatamlx: no visible binding for global variable 'iytype'

Ok, I'm assuming all these are created within this loop:

for (i in 1:length(headerList)) { hi=headerList[[i]] ih <- which(header %in% hi) if (length(ih)==0) ih=NULL ii <- paste0( 'i', tolower(hi) ) eval(parse(text= paste0(ii,"=ih"))) if (!is.null(ih)) newHeader[ih]=newList[i] }

through the eval(parse()) trick. This is not going to fly.

Reply to this email directly or view it on GitHub https://github.com/romainfrancois/mlxR/issues/2.

Mixed Effects Models for the Population Approach http://www.math.u-psud.fr/~lavielle/book.html

Marc Lavielle Inria Saclay

Tel: (33) 6 37 31 93 05

Marc.Lavielle@inria.fr http://www.math.u-psud.fr/~lavielle

"""""""""""""""""""""""""""""""""""""""""""""""""""""""""

romainfrancois commented 9 years ago

Maybe. I'll try this after lunch :-)