grimbough / rhdf5

Package providing an interface between HDF5 and R
http://bioconductor.org/packages/rhdf5
61 stars 22 forks source link

Tenx speed #2

Closed grimbough closed 6 years ago

grimbough commented 6 years ago

There is a do.call in h5readDataset that takes a subset of an object. This appears to result in a memory copy even if the subset is the entirety of the original object. This patch adds logical to check whether we really need to subset, or can just use the original as is, in which case the mem-copy doesn't occur and things work more quickly.