Skip to content

Conversation

yarikoptic
Copy link
Member

to build a docker image following "the recipe" but using jessie as it was some time in 2015 with FSL 5.0.9 (of some Debian revision):

./Simple_Prep_docker jessie 20150306T060524Z

and then to execute actual analysis use

docker run -it --rm repronim:simple_prep_yoh_jessie_20150306T060524Z /bin/bash -c '. setup_environment; cd simple_workflow-master && python run_demo_workflow.py --key 11an55u9t2TAf0EV2pHN0vOd8Ww2Gie-tHp9xGULh_dA && python check_output.py'

to build a docker image following "the recipe" but using jessie as it was some
time in 2015 with FSL 5.0.9 (of some Debian revision):

./Simple_Prep_docker jessie 20150306T060524Z

and then to execute actual analysis use

docker run -it --rm repronim:simple_prep_yoh_jessie_20150306T060524Z /bin/bash -c '. setup_environment; cd simple_workflow-master && python run_demo_workflow.py --key 11an55u9t2TAf0EV2pHN0vOd8Ww2Gie-tHp9xGULh_dA && python check_output.py'
@satra
Copy link
Collaborator

satra commented Jan 26, 2017

yup source doesn't work directly, this is why we do this in many cases: https://github.com/BIDS-Apps/nipypelines/blob/master/Dockerfile#L12

@yarikoptic
Copy link
Member Author

so for now we could stay with direct sourcing in the call -- listing/hardcoding all ENVs would not be possible for varying versions of FSL etc. How do you like it otherwise? ;)

@satra
Copy link
Collaborator

satra commented Jan 26, 2017

it looks good for now.

next things are:

  • make submit_output.sh when output differs from expected output together with provenance
  • create dashboard for comparing outputs
  • use docker in circle to generate
  • create multiple dockers (like nibotmi to go across OSes and FSL versions at least)
  • represent the whole thing in NIDM-W

* origin/master:
  added preprint
  adds license - closes ReproNim#12
  DOC: improve README to point to needing curl and unzip to install environment etc
@yarikoptic
Copy link
Member Author

ok then it is ready to be merged for now -- README.md documentation was tuned up to fit it in ;)

@yarikoptic yarikoptic force-pushed the enh-docker branch 2 times, most recently from 7e6c673 to a174157 Compare January 26, 2017 18:02
@satra satra merged commit dd1afdb into ReproNim:master Jan 26, 2017
@yarikoptic yarikoptic deleted the enh-docker branch April 5, 2017 14:08
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