MarioniLab / scran

Clone of the Bioconductor repository for the scran package.
https://bioconductor.org/packages/devel/bioc/html/scran.html
40 stars 22 forks source link

Suggestion to `combineVar` function #68

Closed AndrMenezes closed 4 years ago

AndrMenezes commented 4 years ago

Hello,

I am using the combineVar to averaging the variance components across 11 batches. The DataFrames objects are in a list. The combineVar function requires to pass the DataFrames objects separately by comma and the first line code of the function performed: collected <- list(...)

My suggestion is to include an optional argument to pass the DataFrames objects in a list. This will be helpful when there are several objects.

LTLA commented 4 years ago

I can do that, but FYI, you can just do do.call(combineVar, your_list_of_objects) for the time being.

AndrMenezes commented 4 years ago

Thanks!

LTLA commented 4 years ago

I'm going to reopen this, just to remind myself to add a list mode to this function.