-
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
From MuData to SpatialData #319
base: main
Are you sure you want to change the base?
Conversation
|
||
|
||
### dependencies | ||
- pinned "spatialdata==0.2.6", "spatialdata-io==0.1.6", "dask==2024.12.1" as temporary fix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why these versions?
import os | ||
from pathlib import Path | ||
""" | ||
this script copies the make_adata_from_csv.py that creates | ||
ONE MUDATA PER SAMPLE, with in each ONE LAYER per modality |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this comment up to date?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah no! i can update it
L.info("Reading in SpatialData from '%s'" % args.input_spatialdata) | ||
sdata = sd.read_zarr(args.input_spatialdata) | ||
#mdata = mu.read(args.input_spatialdata) | ||
#spatial = mdata.mod['spatial'] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is it expected that at this checkpoint you don't offer the option to read a mudata anymore?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
expected from what? the plot_qc_spatial.py
file gets .zarr files as input from the pipeline in qc and preprocessing
L.info("Reading in spatial MuData from '%s'" % args.input_spatial) | ||
mdata_spatial = mu.read(args.input_spatial) | ||
adata_st = mdata_spatial.mod['spatial'] | ||
L.info("Reading in spatial SpatialData from '%s'" % args.input_spatial) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cell2location only accepts spatialData now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. do you want cell2location to also run on MuData/AnnData?
Hi!
I adjusted both the Ingestion and the Preprocessing to SpatialData. I also added the ingestion for xenium
I can, as a next step, adjust the github actions and tutorials.