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

hippb500 workflow, output_spaces #38

Merged
merged 9 commits into from
Apr 20, 2021
Merged

hippb500 workflow, output_spaces #38

merged 9 commits into from
Apr 20, 2021

Conversation

akhanf
Copy link
Member

@akhanf akhanf commented Mar 2, 2021

workflow for pre-processed hippb500, and option for output_spaces

Instead of using T1w to register to corobl, this workflow assumes hipp dwi (b500) is already in corobl space, and just performs a predefined crop to a predefined bounding box (shape used for training the CNN) for each hippocampus. This has been tested and working well with UAlberta dMRI protocol data. Note that the required pre-processed _b500.nii.gz data is not BIDS-compliant, but is the suffix used in snakedwi.

Also adds an output-spaces cmdline param to specify what space output images are generated in (cropT1w is default, but corobl used for hippb500)

To do:

  • make sure workflow still running as intended after merging in upstream
  • verify that output-spaces command-line parameter works as intended for all modalities
  • adapt the workflow for corobl hippT2w perhaps? (maybe a different PR)

akhanf and others added 9 commits February 7, 2021 21:16
workflow for pre-processed hippb500, and option for output_spaces
Check if all space-unfold files are being added as target rules
if cropT1w is in output_spaces, then use T1w (which will also genereate corobl as a
dependency), but if not, then use corobl as the ref space
@akhanf akhanf changed the base branch from master to v0.4.0rc April 20, 2021 23:48
@akhanf akhanf merged commit fd4fe03 into v0.4.0rc Apr 20, 2021
@akhanf akhanf deleted the hippdwi branch April 20, 2021 23:50
@akhanf akhanf mentioned this pull request Apr 21, 2021
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant