Skip to content
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

Add support for non-Visium data and/or SpatialFeatureExperiment #61

Open
lcolladotor opened this issue Nov 30, 2023 · 3 comments
Open
Labels
enhancement New feature or request
Milestone

Comments

@lcolladotor
Copy link
Member

lcolladotor commented Nov 30, 2023

Right now spatialLIBD is built upon the SpatialExperiment container class. There is also https://bioconductor.org/packages/SpatialFeatureExperiment/ that can support non-Visium spatially-resolved data.

As new technologies come online, we will see which container class(es) adapt the most and become widely adopted for these data types. We will likely need to refactor and/or update spatialLIBD to support these other classes. This might involve rewriting several functions to use S4 classes.

Once these internal updates are complete, we should make a new guide (vignette) describing how to use spatialLIBD for each of the new data types that we add support for.

@lcolladotor lcolladotor added the enhancement New feature or request label Nov 30, 2023
@lcolladotor lcolladotor added this to the bioc v3.22 milestone Nov 30, 2023
@lcolladotor
Copy link
Member Author

Some examples of non Visium data include Xenium, a new platform by 10x Genomics that is currently being rolled out.

@cathalgking
Copy link

FYI, SpatialExperimentIO::readXeniumSXE() will form an SPE from 10x Xenium data. However, I am not sure if that would work with this package.

If I construct my SPE manually (with count matrix, image and spatial co-ords) can I use spatialLIBD::run_app()? If not, what is the bare minimum I need to add to the SPE in order for it to run?
https://www.bioconductor.org/packages/release/bioc/vignettes/SpatialExperiment/inst/doc/SpatialExperiment.html#21_Manually

@lcolladotor

@lcolladotor
Copy link
Member Author

Hi @cathalgking,

We currently haven't worked on adding support for Xenium data to spatialLIBD. This issue is part of our proposed future plans that we submitted for a grant application, that was unfortunately unsuccessful.

We likely will circle back to adding Xenium once my team starts analyzing Xenium data, but that's currently not the case. So you might figure out things before us.

Best,
Leo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

No branches or pull requests

2 participants