-
Notifications
You must be signed in to change notification settings - Fork 600
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
parquet support for visium HD #2992
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2992 +/- ##
==========================================
- Coverage 75.51% 75.49% -0.02%
==========================================
Files 117 117
Lines 12955 12959 +4
==========================================
+ Hits 9783 9784 +1
- Misses 3172 3175 +3
|
Also refer to #2982 |
We ran some data through spaceranger 3.0.1 locally, and in doing so found that 10X have reduced redundancy in the A hotfix is to copy the images back into the appropriate subdirectory, but that's not a feasible expectation on users. I added an optional argument The code is currently included in bin2cell in case anybody needs it or just wants to take it out for a spin, but I think it belongs in a more central location. |
hi @ktpolanski ,thank you for the heads up
@LucaMarconato do we have any datasets that test for spaceranger 3.0.1 ? |
Last week together with Wouter we identified some good candidate datasets (some of them recommended by 10x). I believe it's 3.0.0 not 3.0.1 though. The list is in the top part of my comment here scverse/spatialdata#700 (comment) |
Visium HD stores its coordinates in a
.parquet
file. This loads said file.