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

Multi-class tile import? #6

Open
andreped opened this issue Jan 26, 2022 · 4 comments
Open

Multi-class tile import? #6

andreped opened this issue Jan 26, 2022 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@andreped
Copy link
Owner

The current tileImport script does not support multiclass tiles, that is produced prediction tiles generated from MIB by a trained multi-class model.

It is possible to import predictions from MIB to QuPath by predicting on the full WSI in MIB which produces a stitched prediction image, which can be imported in QuPath using the importStitchedTIFfromMIBscript, similarly as done for FastPathology. And therefore, for the full pipeline a tile importer is not critical for using our pipeline.

However, there are scenarios where having a tile importer is beneficial over the other alternative, for instance if one only wishes to run prediction on a smaller region of the WSI (based on segmentations in QuPath) - which can be especially helpful if the image is extremely large. This can make inference a lot faster.

It is possible to do such a workflow in FastPathology, given that you provide the region of interest to run inference on. However, no such method has been made easily accessible. It would also require annotations or a model that can produce these ROI segmentation to use as a preprocessing step for the next patch-wise model.

Therefore, having multi-class support for the tile importer could be a valuable feature.

@andreped andreped added the enhancement New feature or request label Jan 26, 2022
@SahPet
Copy link
Collaborator

SahPet commented May 4, 2022

I've just created a tutorial on how to do multiclass import on the WSI-level here: #10

In some rare cases one would ideally also like to be able to import multiclass predictions from DeepMIB from exported patches from QuPath, but the current WSI-prediction approach will suffice in most cases I think.

@andreped
Copy link
Owner Author

andreped commented Sep 9, 2024

@SahPet Do you have a working groovy script that we could use as a drop-in replacement for the current script:
https://github.com/andreped/NoCodeSeg/blob/main/source/importTiles.groovy

Alternatively, we could offer it as an alternative method (for the multi-class case).

@SahPet
Copy link
Collaborator

SahPet commented Sep 9, 2024

Import of downsampled full WSI tifs is working well with multiclass, but I've struggled with adapting the tile script to multiclass (I think I managed once, but I can't find the version of the script). For now, I recommend using the multiclass import for full downsampled versions of the WSI:
https://github.com/andreped/NoCodeSeg/blob/main/source/importStitchedTIFfromMIB.groovy

@andreped
Copy link
Owner Author

andreped commented Sep 9, 2024

For now, I recommend using the multiclass import for full downsampled versions of the WSI:

@SahPet That you for the info!

OK, but that means that we still do not have a working multi-class tile import script, right?
If you manage to create on down the line, feel free to mention it here, and I can add it for you.
I will keep this issue open till then.

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
None yet
Development

No branches or pull requests

2 participants