-
Notifications
You must be signed in to change notification settings - Fork 922
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
compatibility with spaceranger 2.0 #6208
compatibility with spaceranger 2.0 #6208
Conversation
Thanks for the PR, Stephen! |
You're welcome! |
@mojaveazure do you have an approximate timeline as to when this fix will be available on the CRAN version of Seurat? |
Hi Stephen, sorry for the delay. We just submitted this to CRAN as part of Seurat v4.2.0; hopefully it'll be accepted quickly |
Awesome!———————————————Stephen Williams, PhDStaff Scientist, Computational ***@***.*** 804 426 744610X Genomics, Inc.6230 Stoneridge Mall RoadPleasanton, CA 94588-326010xgenomics.comOn Sep 21, 2022, at 4:55 PM, Paul Hoffman ***@***.***> wrote:
Hi Stephen, sorry for the delay. We just submitted this to CRAN as part of Seurat v4.2.0; hopefully it'll be accepted quickly
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>
|
This is a fair way out of my comfort/skill range (apologies if not the right place to ask): I had a script working well for SpaceRanger1.0 data but since the update it's broken at the Load10X_Spatial step
It's throwing the same error due to the tissue_positions.csv naming issue. I've had a look at the Read10X_Image fix for spaceranger 2.0 and I think I understand what was done, but I cannot work out how to do the same fix for Load10X_Spatial or how to integrate them together. Is anyone able to help? |
@shieldartin what exactly is the error? |
Sorry, this is the error
I can manually change the file name to get it to load, but i think the headers then cause an issue with 'SpatialFeaturePlot'
|
@shieldartin what version of spaceranger are you running for this sample? Can you please run Also what version of Seurat are you running? |
Results: I didn't run the SpaceRanger process, but was told it was the new version that came out this year (and the file name changes are consistent). |
@shieldartin can you please run
restart R and run the following code?
|
Okay, so it seems that the restart of R has fixed the issue. Many thanks for the support. |
@shieldartin glad it's fixed! |
Step1: You can change the file name "tissue_positions.csv" to "tissue_positions_list.csv". Step2: Remove the first line of tissue_positions_list.csv (barcode,in_tissue,array_row,array_col,pxl_row_in_fullres,pxl_col_in_fullres). After these 2 steps, It works. |
Thanks @yuzhouf |
This PR updates seurat to accommodate the latest changes made in spaceranger 2.0 which will break the released version of seurat. Will provide backwards compatibility to pre 2.0 releases.
Note
Thanks for your interest in contributing! Please make your PR to the
develop
branch. You can check out our wiki for the current developers guide.