-
Notifications
You must be signed in to change notification settings - Fork 120
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
Some zeros in aseg.stats #492
Comments
Can you please say which file you are referring to and which FastSurfer version you are using? I assume it is Generally, some zero-values in the stats file are expected, if these specific regions are "requested", but no voxels in the image are labeled as this region. In particular, with both your options ( We should also note, that the regions that are zero in this file are regions that might be reasonably zero. Vessels are often not segmented and WM-hypointenisities are not lateralized. |
Hello, yes it is from aseg.stats, and the version is the Dev branch from Feb 7th (I had to use the dev branch due to an OOM issue from the release branch). Not sure how to check version. Looking into it further, it seems the 5th ventricle is rare and expected to be zero. The vessels and optic chiasm, however, I do not think should be zeros. I ran FastSurfer on 76 subjects and have FreeSurfer volumes for these same subjects, and they are non-zero (typically volume of 50 mm^3 for left vessel, and 187 mm^3 for optic chiasm). For all 76 subjects both traits were exactly 0 from FastSurfer. I am looking into whether the aseg.mgz has any voxels labeled as these now. |
OK, I do not believe that there are any voxels labeled as optic chiasm or 5th vessel (left or right) in aseg.mgz, assuming this is the correct look up table: https://github.com/Deep-MI/FastSurfer/blob/dev/FastSurferCNN/config/FreeSurferColorLUT.txt . The unique values in aseg.mgz are array([ 0, 2, 3, 4, 5, 7, 8, 10, 11, 12, 13, 14, 15, Is this expected behavior that none will be labeled as these two regions? I understand both are small regions, and so maybe the values from freesurfer are just essentially noise (i.e., misclassification) and FastSurfer treats this noise slightly differently? Just want to make sure I am not making a mistake somewhere. |
This all seems correct to me. I think your examples all fall into this category. |
You can also add Please note that the dev branch is not a validated setup. It may produce unexpected results. We are currently working to implement CI to reduce this, but only the stable branch and released versions are rigorously tested.
I'll have a look at the numbers of these regions in our rigorous tests later. But we only do those tests for release versions. |
Also, I cannot really explain why your OOM errors were fixed by dev over the release version. That makes little sense to me. |
Hello, I was suggested to try this by: #444 It seems to have largely solved the issue (still happens but only on about 1/1000 cases, whereas before it was happening on most subjects). |
Hi, will close this now. It is correct that some labels are not generated in our segmentation network (such as 5th ventricle or optic chiasm). These labels were pretty unreliable and some are non existent in many subjects, which is why we dropped them. Other labels that don't get segmented directly in the neural net get either:
And some, like the ones above, get never created. |
Hello, a few values are zero in aseg.stats when I run this command:
docker run -v ${SETUP_DIR}:/data my_fastsurfer:cpu --t1 /data/T1_unbiased_brain.nii.gz --sd /data/ --sid Tutorial --py python3 --allow_root --3T --fs_license /data/FreeSurferLicense.txt --parallel --fsaparc --threads 2
Namely, things like the left and right vessel. Is this expected behavior, or am I missing something? Similarly, I have zeros in the fifth ventricle and optic chiasm.
Thanks very much in advance for your help!!
The text was updated successfully, but these errors were encountered: