Skip to content

updates for Metab4_Docker#1

Open
jwhowarth wants to merge 4 commits intoxia-lab:masterfrom
jwhowarth:master
Open

updates for Metab4_Docker#1
jwhowarth wants to merge 4 commits intoxia-lab:masterfrom
jwhowarth:master

Conversation

@jwhowarth
Copy link

The following changes are proposed.

  1. Fix the war file installation by prefixing the $DEPLOY_DIR path to $METABOANALYST_FILE_NAME on the wget call which eliminates the need for an explicit COPY command.
  2. Update docker image from Trusty (14.04) to Xenial (16.04) for better security and add an explicit installation of the required 'sudo' package not present in the Xenial image.
  3. Append missing 'xcms' R package to the Bioconductor package list installed by the Dockerfile.
  4. Remove the use of p_load() in metab4script.R to avoid needlessly reinstalling the R packages on every execution of the Rscript.
  5. Append the '--rm' option to the recommended 'docker run' command so that the container is always automatically removed on exit. Having to manually remove the container can be confusing to new users of docker.

@superleah
Copy link

Create a suppleme.t for pg13

@superleah
Copy link

Boy http and pg13 are now sense xshtml=

@superleah
Copy link

Wml under sign lib for plug and play bugs

@superleah
Copy link

Isbn lib

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.

3 participants