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

Add surface reconstruction #101

Closed
chrisgorgo opened this issue Aug 12, 2016 · 6 comments
Closed

Add surface reconstruction #101

chrisgorgo opened this issue Aug 12, 2016 · 6 comments

Comments

@chrisgorgo
Copy link
Contributor

chrisgorgo commented Aug 12, 2016

  • Use FreeSurfer 6 beta
  • Run FS by default but allow turning it off on the command line
  • Make sure to use all available cores for FreeSurfer
  • Use -highres if the input T1 resolution is <1mm
  • Use WM and ventricle parcels produced by FS in aCompCor
  • Feed multiple T1s to recon-all if available (but only from one session and only if they have the same dimensions/resolution - for longitudinal datasets each session should be analyzed separately)
  • Feed T2s to recon-all if available (but only if they have resolution along the coarsest dimension < 1.2mm)
  • Replace internal Freesurfer skullstripping with something better (for example ANTs) - see https://github.com/nipy/nipype/blob/master/nipype/workflows/smri/freesurfer/recon.py#L12
  • If T2s are available use sqrt(T1*T2) bias field correction method (see HCP pipelines for details)
  • Use FreeSurfer implementation of BBR for EPI to T1 coregistration
  • Output:
    • SUBJECTS_DIR (with all subjects)
    • fMRI timeseries in fsaverage space

Implement and use reportlets outlined in nipreps/niworkflows#102 and nipreps/niworkflows#103

@oesteban
Copy link
Member

Duplicate of #63. But here we have a clear description. I close the old one.

@oesteban oesteban modified the milestone: Version 1.1.2 Aug 14, 2016
@oesteban oesteban self-assigned this Aug 15, 2016
@oesteban oesteban modified the milestones: Version 1.1.2, Version 1.1.3 Aug 15, 2016
@oesteban oesteban modified the milestones: Version 0.1.3, Version 0.2.0 Aug 30, 2016
@oesteban oesteban modified the milestones: Version 0.2.0, Version 0.2.1 Jan 11, 2017
@chrisgorgo chrisgorgo assigned effigies and unassigned oesteban Jan 12, 2017
@effigies
Copy link
Member

Focus group findings

  • Verifying white/pial surface alignment with structural was 👍
    • As a summary stat, noting spikes in cortical thickness (distance from white to pial that is different from surrounding vertices) would be a good addition. (Greg F)
  • Interest shown in gifti/cifti outputs (Greg F)
  • Mixed response on FreeSurfer 5.3.0 vs 6.0.0
    • Positive response to v6
    • For v5.3, concern was established labs would be more interested in using the same tools, although if they're doing longitudinal, they probably have existing pipelines and aren't going to go back to use FMRIPREP.
    • So our focus should mainly be people setting up new pipelines, so v6.0 is the smarter move

@effigies
Copy link
Member

Will note here that there weren't any particularly strong opinions about surface sampling techniques (vertex, voxel or pixel based) among the focus group participants I talked to. Not sure if this should be its own issue, given that there's no recommendation to be made.

@chrisgorgo
Copy link
Contributor Author

chrisgorgo commented Jan 13, 2017 via email

@effigies
Copy link
Member

Proper FreeSurfer v6 support is going to have to wait on resolution of nipy/nipype#1782.

I have a hacky branch we can pin against just to get it working.

@chrisgorgo
Copy link
Contributor Author

Could you work on resolving nipy/nipype#1782? We will need it in a release version of nipype sooner or later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants