-
Notifications
You must be signed in to change notification settings - Fork 10
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
convert Seurat to SFE and back #3
Comments
I'm not currently doing it. Yes, help is needed. Thank you so much for the PR! |
sounds good! happy to contribute. |
Yes, readXenium is really easy to implement. |
Hey Lambda, |
You can store it in rowData, but I'm not sure what to do with the huge size of these files at present. As a result, readVizgen doesn't read it by default. |
ok, for Seurat I made it optional -> to have molecules or not |
Thank you so much! I need to look into on disk representations of geometry to better deal with the transcript spots. Now there's SpatialData in development, mainly raster, and I'm considering sedona for on disk vector geometry operations. I don't know whether raster or vector is more efficient on disk. Anyway, for now the large files read into memory can be dealt with by running the code on a server. BTW, I added you to the Acknowledgement section of the Voyager paper. |
Always happy to contribute, Many thanks for the Acknowledgement! :-) |
Almost ready with the optimized
I'm splitting by gene and organizing it into
The question is:
Let me know. |
Thank you so much! Yes, I actually meant rowGeometries. |
👍 great! |
Almost completed on this converter function
|
Can you put your function in Don't worry about |
Right now, it just a single function that can convert from Seurat (v4) to SFE and back to Seurat (eg, when arg I can definitely try with
Yeah, no problem, I managed to |
a small update, TODO (from top to low priority):
|
Hi there )
as we discussed a bit on voyager #2 to add support to convert from Seurat to SFE and back. Is that something you are currently doing? If yes, and help is needed, I could jump in.
Else, I could try to implement that conversion, but could do PR only around September, if that works, let me know.
Thanks
A.
The text was updated successfully, but these errors were encountered: