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

Freesurfer atlas #186

Merged
merged 4 commits into from
Jun 1, 2022
Merged

Freesurfer atlas #186

merged 4 commits into from
Jun 1, 2022

Conversation

akhanf
Copy link
Member

@akhanf akhanf commented May 30, 2022

Adds freesurfer-based subfield unfolded atlas, using this workflow to create the new files: https://github.com/khanlab/hippunfold-create-atlas/tree/freesurfer (ie, mapping automated labels at the midsurface to the unfolded atlas, and doing maxprob over all subjects)

  • add to docs update docs in another PR since other docs branch is still open
  • consider removing labels (?) -- I kept all but amygdala (labels 7001+) from the freesurfer subfields LUT, but we could potentially remove all that we don't want to get mapped into the CA/Sub/DG ribbon..

@jordandekraker
Copy link
Collaborator

Since hippunfold already separates our SRLM, it might be best to remove all labels with _molecular and the hippocampal fissure label. HippUnfold also provides a separate DG label, so we could also remove this from the atlas as well. Doing these things would probably clean up the discontinuities in the unfolded space quite a bit.

One other thing to consider might be conjoining the equivalent labels from head and body. I think this is a pretty arbitrary distinction, though a lot of people find it useful as a rough way to look at anterior-posterior differences in the hippocampus so I don't mind leaving the separation either.

@akhanf
Copy link
Member Author

akhanf commented May 30, 2022

Actually it looks like FS spits out additional sets of segmentations:
https://surfer.nmr.mgh.harvard.edu/fswiki/HippocampalSubfieldsAndNucleiOfAmygdala
and the "CA" set is probably the most suitable here, as it already folds in the SRLM labels. I'll regenerate a new label nifti with those.

@jordandekraker
Copy link
Collaborator

Sounds good! I would still maybe remove the fissure label since in our protocol we include it as part of SRLM.

Also my test case seems to have worked perfectly! I'll try again with the new labels when its committed

@akhanf
Copy link
Member Author

akhanf commented May 30, 2022

Looks like using the "CA" set of labels dealt with things well, the only labels that show up are presub,sub,CA1,CA3,CA4. There is a tiny island of parasubiculum (and islands of CA1, Sub) in the most distal end, but I think it's fine to just leave that as is since easier to explain. I'll commit these in a sec, but here are screenshots fyi

image

image

@akhanf akhanf added the enhancement New feature or request label May 30, 2022
@akhanf akhanf merged commit eb1d1ce into master Jun 1, 2022
@akhanf akhanf deleted the freesurfer-atlas branch June 1, 2022 16:16
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

Successfully merging this pull request may close these issues.

2 participants