search

davismcc / archive-scater

An archived version of the scater repository, see https://github.com/davismcc/scater for the active version.
64 stars 18 forks source link

Maximum number of cells? #114

Closed wikiselev closed 7 years ago

wikiselev commented 7 years ago

What is the maximum number of cells supported by newSCESet? I've tried 10^5 cells and got this error:

Error in .checkedCall(cxx_missing_exprs, exprs(object)) : 
  long vectors not supported yet: ../../../../R-devel/src/main/memory.c:3424

I then tried 99,999 cells and got the same error.

wikiselev commented 7 years ago

Forgot to say that it worked ok with ~89,000 cells.

LTLA commented 7 years ago

I daresay it's because I used the length of the matrix (product of number of rows and columns) in one of my C++ functions. Presumably this exceeds the maximum vector size allowed in R, hence the error. I've since wound back the use of C++ functions (see #112) so this should no longer be a problem in the next release.

wikiselev commented 7 years ago

Thanks, Aaron! So, it's not in bioc-devel yet?

LTLA commented 7 years ago

No, it's not even in the Github master branch yet. Nudge @davismcc.

davismcc commented 7 years ago

This issue was moved to davismcc/scater#15