gustaveroussy / sopa

Technology-invariant pipeline for spatial omics analysis (Xenium / MERSCOPE / CosMx / PhenoCycler / MACSima / Hyperion) that scales to millions of cells
https://gustaveroussy.github.io/sopa/
BSD 3-Clause "New" or "Revised" License
115 stars 11 forks source link

[Feature] Support WSI analysis #19

Closed quentinblampey closed 4 months ago

quentinblampey commented 7 months ago

High priority

Low priority

Will not be added for now

stergioc commented 7 months ago

That could be a great feature add, thanks @quentinblampey for creating this issue!

So here are some additional features that could also be considered:

I can take a look on how to implement the IO for WSI and come back with a suggestion. :)

quentinblampey commented 7 months ago

What segmentation method should we use? CellViT? Hover-net? @HakimBenkirane is also using something else, called HistomicsTK

The method needs to have the following characteristics:

stergioc commented 6 months ago

While trying to make the whole patch embedding pipeline faster I have added the openslide backend at PR #26 . After some tests it is clear that openslide is much faster as a backend (x3). After a bit of digging I found this is issue that it might be related: https://github.com/Bayer-Group/tiffslide/issues/72, I am adding it here just for reference atm.

quentinblampey commented 6 months ago

Ok, thanks for investigating @stergioc! Since openslide is a heavy dependency, I think I will not add it inside the openslide extra: therefore, users that use openslide backend will need to install openslide themselves. What do you think? We can show a nice log if a user tries to use the openslide backend when it's not installed (as we already do for cellpose and baysor)

quentinblampey commented 5 months ago

Hello @stergioc, do you have any updates on the openslide backend?

stergioc commented 5 months ago

Hi @quentinblampey, I don't have any updates yet but I have planned some time for this next week. I will post it as soon as I have something. ;)

quentinblampey commented 5 months ago

Ok thanks! I'm merging the dev branch into the wsi branch, since a lot of things changed since our last push on the wsi branch

quentinblampey commented 4 months ago

Hi @stergioc, thanks again for your contributions, I'm closing this issue since we finished all tasks! Feel free to open again if you want to see a new feature