createMOFAobject() function - error with view names
QinqinHuang opened this issue · 5 comments
Hi,
I tried to create a MOFA object from an existing list. The names of the list is not NULL, but createMOFAobject() assigns new view names for it. I had a look at it and I think here:
if(!is.null(names(data))) { ## it should be if(is.null(names(data))), right?
warning(paste0("View names are not specified in the data, renaming them to: ", paste0("view_",1:length(data), collapse=", "), "\n"))
names(data) <- paste("view",1:length(data), sep="_")
}
Thanks for this tool! I'm really interested to apply it on my dataset.
Hi Qinqin,
you are right, i just introduced this bug in my last commit. It is fixed now, thanks!
Thanks, Ricard!
Hi Ricard,
I feel something is still going on there.. If I run:
data("CLL_data"); MOFAobject <- createMOFAobject(CLL_data)
MOFAobject
The view names are still missing:
Untrained MOFA model with the following characteristics:
Number of views: 4
View names:
Number of features per view: 310 4248 5000 69
Number of samples: 200
And apparently when trying to plot using plotTilesData(), the colour values can not be assigned either.
I did not have this issue yesterday.. I think this happened since your last commit & I reinstalled the package :)
Thanks,
Qinqin
Apologies, didn't quite fix it :)
I just made another commit. Can you try now?
Yeah it's working now. Thank you!