Severin Dicks

Results 122 comments of Severin Dicks

Yes the todo will be a seperate PR once this one is merged. I have the kernel ready

@flying-sheep I don't know why the benchmarks are failing. But I think we should already be seeing improvement. If not we have to adjust the big dataset to have a...

As far as I know this affects cuvs nn_descent and all_neighbors. It also affects cuml umap. There might be more.

I see the same behavior on 2 different Linux based systems.

There will be a seperate PR for the update of the docstrings and a tutorial.

I renamed the functions for QC and renamed some of the variables so its a bit clearer whats happening.

> https://github.com/scverse/rapids_singlecell/pull/179/files#r1838498091 is not done and from what I can tell [#179 (review)](https://github.com/scverse/rapids_singlecell/pull/179#pullrequestreview-2430197776) has not been addressed. What happens if you pass a csc dask array to pca? That will...

Median out of core is a bad choice. Uses way more memory and is slower. Loose Loose

@kaushalprasadhial We internal discussed adding `scikit-learn-intelex` as a dependency. We came to the conclusion that we dont want it as such. Since this a patch that the user can do...