search

chanzuckerberg / cellxgene-census

CZ CELLxGENE Discover Census
https://chanzuckerberg.github.io/cellxgene-census/
MIT License
76 stars 20 forks source link

[feat] option to return csr tensors in datapipe #1062

Open martinkim0 opened 5 months ago

martinkim0 commented 5 months ago

BREAKING CHANGE: replaces return_sparse_X argument with X_format that allows to specify the sparse format. default is kept to dense.

closes #617

prathapsridharan commented 4 months ago

@atolopko-czi @martinkim0 @pablo-gar - There has been no activity on the PR. Was this forgotten?

atolopko-czi commented 3 months ago

@atolopko-czi @martinkim0 @pablo-gar - There has been no activity on the PR. Was this forgotten?

It still needs to be reviewed & completed.

ebezzi commented 3 months ago

I'll do some cleanup to make sure the CI/CD is green and then I'll merge.

github-actions[bot] commented 2 months ago

This PR has not seen any activity in the past 4 weeks; if no one comments or reviews it in the next 3 days, this PR will be closed.

prathapsridharan commented 2 months ago

@ebezzi - note above automated message to close this PR. This comment serves to tell the automated closer to not close it :)

github-actions[bot] commented 1 hour ago

This PR has not seen any activity in the past 4 weeks; if no one comments or reviews it in the next 3 days, this PR will be closed.