-
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
feat: cxg conversion script updates for uns #6909
Conversation
Deployment Summary
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6909 +/- ##
=======================================
Coverage 91.65% 91.65%
=======================================
Files 182 183 +1
Lines 15354 15390 +36
=======================================
+ Hits 14072 14106 +34
- Misses 1282 1284 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Closing this due to this change: https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/gh/chanzuckerberg/single-cell-explorer/839 |
Reason for Change
cxg
conversion script to includeuns
data] #6659Changes
uns
dictionary from AnnData and store object underuns
directory in thecxg
in_tissue
column values to boolean to make it categorical, and fix the issue of it appearing as "continuous" in explorerTesting steps
https://cellxgene.dev.single-cell.czi.technology/e/super-cool-spatial.cxg/?spatial=true
Local testing:
run
def make_cxg()
in a notebook or from the command line against a valid AnnData/h5ad dataset.run converted
cxg
in explorer, verifyspatial
modeNotes for Reviewer