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 script for brain processing #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

atefbadji
Copy link

Processing script for WM atlas registration

% Participants are first aligned into a common space. The JHU-ICBM-FA template is used as a reference for all subjects.

% Affine transfo on FA
sct_unix('isct_antsRegistration --dimensionality 3 --transform affine[1] --metric MI[FA.nii.gz,JHU-ICBM-FA-1mm.nii.gz,1,32] --convergence 30x10x5 --shrink-factors 5x3x1 --smoothing-sigmas 0x0x0mm --output [step0,src_regStep0.nii] --verbose 1');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sct_unix does not seem to be defined anywhere in this repos

@charleygros
Copy link
Contributor

I am closing this PR as we have not considered the diffusion processing at this stage. But the branch remains active in case it is the case one day.

@jcohenadad
Copy link
Member

jcohenadad commented Apr 27, 2020

I would reopen because I prefer to have branches associated with PRs (even if we keep PRs for a while), rather than accumulating stale branches. At some point, repository maintainers will not know the purpose of each branch and it will be more work to reach out to the branch creator to ask why this branch was created. With PRs, we can describe the purpose of a specific branch (and decide if we need to keep or not in the long term).

I've added the label "wontfix" so we know this PR is not a priority.

@jcohenadad jcohenadad reopened this Apr 27, 2020
@jcohenadad jcohenadad added the wontfix This will not be worked on label Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants