Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

deprecate/remove read spatial and plotting spatial from scanpy. #2331

Open
Tracked by #2702
giovp opened this issue Sep 16, 2022 · 3 comments
Open
Tracked by #2702

deprecate/remove read spatial and plotting spatial from scanpy. #2331

giovp opened this issue Sep 16, 2022 · 3 comments

Comments

@giovp
Copy link
Member

giovp commented Sep 16, 2022

I think it makes sense to deprecate and then remove sc.read_visium. and sc.pl.spatial since I don't plan to maintain it here anymore and updated versions are present in Squidpy. What do you think @ivirshup ?

@Zethson
Copy link
Member

Zethson commented Sep 25, 2022

I would like to second this. I think that it makes sense that we don't duplicate functionality and responsibility of our core tools. People shouldn't be looking for spatial stuff in scanpy nor be confused if they suddenly see spatial tooling in the scanpy docs. We should always point them to the awesome squidpy.

@linhuawang
Copy link

I just had issues using sc.pl.spatial after reading spaceranger2.0.0 outputs using sc.read_visium. Although the issue is resolved. I am wondering if these functions are not maintained anymore. I am not familiar with Squidpy. But it seems to use these functions in its package. I would very much appreciate any clarification here.

@giovp
Copy link
Member Author

giovp commented Oct 10, 2022

hi @linhuawang ,

yes I'd suggest you to use squidpy for plotting, unfortunately the spaceranger 2.0 reading is not in main yet (will be by end of the week).

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

No branches or pull requests

4 participants