-
Notifications
You must be signed in to change notification settings - Fork 14
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
SCEasy - AnnData-->Seurat #225
Comments
anndata -> seurat is in the github version of sceasy, but not released yet. I was hoping to also get the dependency issues fixed in the new release to support anndata >= 0.7 (with the cost of dropping support for exchangeable loom but still supports standard loom). Full automatic interoperability at intermediate steps without user input is quite unrealistic and perhaps not of high usage. What do people think? |
That's 9 kinds of awesome! Horray! |
those use cases can be covered without the "exchangeable loom". It was for things in |
I have been trying the new SeuratDisk for Seurat v4, but at least with the examples I have tried it seems broken, and a lot of people are complaining for issues with the HDF5 capability of SeuratDisk (for Seurat v4). On Seurat v3 we have been able to read annData for sometime now (I haven't tested it with AnnData 0.7.x though). Is SeuratDisk working for your @nh3 ? or how are you doing AnnData -> Seurat? Or you are doing it on Seurat v3? What I was thinking was to add a Seurat v3 converter with all the formats that we support already and release that before moving Seurat scripts to v4. Then one could do AnnData -> Seurat v3 -> read it on a v4 tool (that should work I think). |
We are still working with Seurat v3. Never tried v4. Sorry that I didn't realise this is v4 specific. |
@pcm32 Did your fix for this work? AnnData --> Seurat would be great! |
# Description Based on the code from https://github.com/cellgeni/sceasy/blob/master/R/functions.R I tried to adapt it to SCEasy in order to allow Galaxy users to convert AnnData to Seurat files. I find it more useful to expand the SCEasy tool rather than create a new one. It might be also possible to incorporate AnnData to CDS object conversion within SCEasy, once my pull request fixing the bug in the source code is approved. Fixes # (issue) If the changed code works, it will fix the issue ebi-gene-expression-group#225 (SCEasy - AnnData-->Seurat) ## Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) - [x] This change requires a documentation update ## Checklist - [ ] I have made any required changes to upstream dependencies for a tool wrapper, and they are available in distribution channels (e.g. Pip, Conda). - [ ] If I have updated the underlying software for a tool wrapper (e.g. scanpy-scripts by changing the value of `@TOOL_VERSION@`), then I have reset all 'build' values to 0 (e.g. `@TOOL_VERSION@+galaxy0`) - [ ] If I have updated a tool wrapper without a software change, then I have bumped the associated 'build' values (e.g. `@TOOL_VERSION@+galaxy0` `@TOOL_VERSION@+galaxy1`)
AnnData --> Seurat, particularly now there is Seurat v4 trajectory stuff/automated annotation stuff/Monocl can import Seurat objects... this is important!
The text was updated successfully, but these errors were encountered: