Skip to content

Conversation

@jcupitt
Copy link

@jcupitt jcupitt commented Jan 25, 2018

It seems to work for me ... test with:

docker build \
	--build-arg VCS_REF=`git rev-parse --short HEAD` \
	-t <user>/structural-pipeline:latest .

Fetching the FSL key seems a little unreliable, I'm not sure why. If you
get an error, try just running again.

see #2

It seems to work for me ... test with:

	docker build \
		--build-arg VCS_REF=`git rev-parse --short HEAD` \
		-t <user>/structural-pipeline:latest .

Fetching the FSL key seems a little unreliable, I'm not sure why. If you
get an error, try just running again.

see #2
@jcupitt
Copy link
Author

jcupitt commented Jan 25, 2018

(Dockerfile shamelessly copied from MIRTK)

@schuhschuh
Copy link

Sweet! BTW I myself have no permissions to merge. Either wait for @amakropoulos's reply or get permissions from in-house if your part of BioMedIA/dHCP.

I made it urge people to do a build in docker, perhaps that's not wise
@jcupitt
Copy link
Author

jcupitt commented Jan 25, 2018

I added some notes to the README. It should probably use -v to attach the user's data files to the container for processing, I'll experiment.

fsl5 has "fsl5.0-" in front of all exe names

the dockerfile needs to get root shell to run the fsl init script
@jcupitt
Copy link
Author

jcupitt commented Feb 2, 2018

It seems to be working well now. I'm going to merge to master on my fork, I'll leave this here.

The README has some notes -- tldr: build the pipeline to a docker image with:

# docker build -t <user>/structural-pipeline:latest .

Run the pipeline on the file sub-CC00183XX11_ses-60300_T2w.nii.gz in the subdirectory data of the current directory:

# docker run --rm -t -v $PWD/data:/data \
    <user>/structural-pipeline:latest \
    bash -c ". /etc/fsl/fsl.sh; \
        cd /usr/src/structural-pipeline; \
        ./dhcp-pipeline.sh subject1 session1 44 \
            -d /data -T2 /data/sub-CC00183XX11_ses-60300_T2w.nii.gz -t 8"

I'll try and get OpenMOLE using this next.

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.

2 participants