-
Notifications
You must be signed in to change notification settings - Fork 3
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
Incorrect segmentation on a thoracic healthy patient #122
Comments
This is strange, the cord is well contrasted. @NathanMolinier can you give it a try with TotalSpineSeg? |
hey @po09i, how urgent is this? I believe the newer model (unreleased yet) will work better on the 2nd image for sure. I was planning to make a pre-release of the newer model by the end of this week. I would say v2.4 model is old now |
It is not urgent at all, the segmentation on the second image covers more than the FOV of the image I need it registered to. So this is more of a feedback than something I need help with :) |
hey @po09i, I just made a release of the latest model I was referring to, please do try it out if you have time! Note that you might to update the model URL in SCT's codebase in order to access the model via SCT. |
no, this is bad practice-- we should not encourage ppl to manually edit hard-coded path in SCT, resulting in code that is out-of-sync with git history. @naga-karthik please update the URL with your model and submit a PR on SCT, thanks |
Maybe Naga meant to download the pre-release as now possible using sct_deepseg -install seg_sc_contrast_agnostic -custom-url https://github.com/sct-pipeline/contrast-agnostic-softseg-spinalcord/releases/download/v2.5/model_contrast-agnostic_20240930-1002.zip
sct_deepseg -i <IMAGE> -task seg_sc_contrast_agnostic (i.e., no need to touch the SCT code) |
I tried @naga-karthik / @valosekj 's suggestion and got this error: Terminal dump
*I shortened some paths |
@po09i Oopsi, I realized that the model architecture was also slightly updated with v2.5. Hence the size mismatch errors you got. I created a PR on the SCT repo and tested it on an image myself. If you could please try it again (usage instructions in the PR), it should work fine now! |
@naga-karthik The PR did resolve the crash. I used the newer version (2024_09_30) and similarly did not get any segmentation of the spinal cord :/ |
Information
Contrast: T1w
Region: Thoracic
Pathology: Healthy
SCT model: https://github.com/sct-pipeline/contrast-agnostic-softseg-spinalcord/releases/download/v2.4/model_soft_bin_20240425-170840.zip
Issue
In my pipeline, I register the T1w image (0.9x0.9x5mm) to another acquisition (0.5x0.5x5mm) then performed the segmentation
Segmentation completely fails
I tried to segment the original image as well. This went better, but it stopped in lower slices
Incomplete segmentation in lower slices in the original acquisition
The text was updated successfully, but these errors were encountered: