abcd-hcp-pipeline icon indicating copy to clipboard operation
abcd-hcp-pipeline copied to clipboard

Include Dockerfile generator script for neurodocker in repo

Open DavidEWarrenPhD opened this issue 6 years ago • 1 comments

The included Dockerfile appears to be generated using neurodocker, but a script containing the neurodocker call is not included in this repo. Including the neurodocker generator script would be convenient and would enhance reproducibility. Example at https://github.com/kaczmarj/neurodocker/blob/master/examples/conda_python/generate.sh.

DavidEWarrenPhD avatar Sep 29 '19 21:09 DavidEWarrenPhD

@DavidEWarrenPhD Thanks for pointing this out! We should remove most of that from the Dockerfile and credit kaczmarj for what came from their neurodocker. What happened was we used kaczmarj's neurodocker from DockerHub for the original first base Dockerfile build and just never deleted all the comments after we had basically rewiritten the whole Dockerfile. Regardless, I will leave this issue open until we make those corrections.

ericearl avatar Oct 10 '19 21:10 ericearl