scverse / scverse-tutorials

Tutorials for learning scverse
https://scverse-tutorials.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
12 stars 12 forks source link

Update scRNA tutorial #159

Closed flying-sheep closed 1 month ago

flying-sheep commented 5 months ago

Fixes #153

basic clustering tutorial notebook

Other notebooks

Conda setup

Some of the conda setup does no longer work, namely use-only-tar-bz2, so I needed to disable it. Let’s fully fix the setup in another PR, this PR makes it work again which was hard enough.

We should probably go away from conda in the first place. Installing pip packages into a conda environment is an undefined operation, all version bounds for the conda packages become meaningless when doing that. uv could be enough, otherwise we could go pixi or so.

review-notebook-app[bot] commented 5 months ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

grst commented 1 month ago

Seems like the manual annotation is already broken, so I removed it. If you have an idea how we can keep it stable, we could do that instead, but I couldn’t figure out which cluster assignments actually work.

In https://github.com/scverse/scanpy/issues/2014, we figured out that setting

export NUMBA_CPU_NAME=generic

makes the clustering results more stable (at least in the conditions I tested there) by only relying on basic CPU features. Would still need to redo it once but then hopefully it wouldn't change anymore in the future.

grst commented 1 month ago

And not sure why we went with conda in the first place, maybe because of compatibility with mybinder?

flying-sheep commented 1 month ago

let’s restore the manual clustering in another PR, I don’t feel like going through all clusters and assigning labels manually.