search

Bioconductor / HDF5Array

HDF5 backend for DelayedArray objects
https://bioconductor.org/packages/HDF5Array
9 stars 13 forks source link

Add support for seurat hdf5 format. #59

Open yduanBioinfo opened 8 months ago

yduanBioinfo commented 8 months ago

Add support for h5seurat format to TENxMatrix object. In this version, only functions in TENxMatrix-class and TENxMatrixSeed-class has been changed. At the same time, I have added a toy data and written some example codes in the help document.

As a concern, the change in .find_rownames_dataset of TENxMatrixSeed-class is a little dangerous. I'm afraid that the 'features' prior than 'features/id' will cause a bug. However, if set 'feature/id' prior, it will cause an error: "Error in H5Lexists(fid, name) : HDF5. Links. Can't get value". If the bug actually exist, it's maybe a solution to write a new function .read_h5seurat_component to read h5seurat data separately as in my last pull request.

hpages commented 8 months ago

Thanks @yduanBioinfo. Will look at this shortly.