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.

Sign up for GitHub

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

Jump to bottom

Optional pre-processing #29

Open

3 tasks

kaitj opened this issue Nov 22, 2022 · 2 comments

Milestone

Collaborator

kaitj commented Nov 22, 2022 •

edited

Loading

Optionally perform pre-processing for workflow if not already run.

brain extraction (Synthseg)
Freesurfer / Fastsurfer
~~DWI Preprocessing (Snakedwi)~~

kaitj added this to the v0.2.x milestone

kaitj modified the milestone: v0.2.x

Collaborator Author

kaitj commented May 1, 2023

Some notes from the SCATTR meetings:

Probably want to set up some kind of severely downsampled source dataset.
Maybe look into Snakemake unit testing facilities
Potential issue for GH Actions: It would need the container.
Can build the container on the GH actions runner and overwrite the config value.

kaitj modified the milestones: v0.2.x, v0.3.x

Collaborator Author

kaitj commented Feb 8, 2024

Additional thoughts / comments after last meeting:

Preprocessing (of DWI and maybe all things considered here) may fall out-of-scope of SCATTR. The docker container would either need to include the software necessary to perform preprocessing or call snakedwi, the processing workflow of choice, which would download another larger container.
Freesurfer / fastsurfer may be possible with an updated version of freesurfer used in the container.
Brain extraction may also be possible (synthseg via Freesurfer or via mrtrix's brain extraction tool).

The gist of this is that anatomical processing is possible with existing container while diffusion pre-processing is likely out-of-scope.

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