Skip to content

Updated compatibility for Seurat - #138

Open
aliibarry wants to merge 9 commits into
kharchenkolab:devfrom
aliibarry:main
Open

Updated compatibility for Seurat#138
aliibarry wants to merge 9 commits into
kharchenkolab:devfrom
aliibarry:main

Conversation

@aliibarry

Copy link
Copy Markdown

Minor updates to work with Seurat V5, addressing depreciation of slot for layer as discussed in #133

Includes additional switch from tsne to umap default when running basicSeuratProc(); although I never debugged why tsne fails with some Seurat V5 objects.

@evanbiederstedt

Copy link
Copy Markdown
Contributor

Thanks @aliibarry

So, basically everything works except the Seurat developers deprecated 'slot' for 'layer' ? This is for Seurat V3 as well?

It's been a while since I have looked at the v5 Seurat tutorials.

I suspect you are just running this guide, right? https://satijalab.org/seurat/articles/seurat5_integration

Question:
Is there anything else you are doing with Seurat and Conos? I'll try to debug + to check if this works.

Question:

Includes additional switch from tsne to umap default when running basicSeuratProc(); although I never debugged why tsne fails with some Seurat V5 objects.

You could provide more details/outputs? That would be helpful for me to debug.

Best, Evan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants