Skip to content

Commit 44a4ceb

Browse files
committed
fix: updated docker image
1 parent ee01af5 commit 44a4ceb

File tree

4 files changed

+5
-13
lines changed

4 files changed

+5
-13
lines changed

Dockerfile

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -161,10 +161,6 @@ RUN git clone --depth 1 https://github.com/KChen-lab/SiFit.git && \
161161
mv SiFit/SiFit.jar /usr/local/lib/SiFit.jar && \
162162
rm -rf SiFit
163163

164-
# Install Monovar
165-
RUN git clone --depth 1 https://github.com/KChen-lab/MonoVar.git && \
166-
ln -s "${PWD}/MonoVar/src/monovar.py" /usr/local/bin/monovar
167-
168164
VOLUME /root/data
169165

170166
WORKDIR /root/data

README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,13 @@ We have configured a docker file containing everything needed to run the benchma
1111
To acqire the docker image, pull from Docker Hub with
1212

1313
```bash
14-
$ docker pull senbaikang/sieve_benchmark:0.1
14+
$ docker pull senbaikang/sieve_benchmark:0.2
1515
```
1616

1717
or build from Dockerfile in the root of this repository with
1818

1919
```bash
20-
$ docker build -t sieve_benchmark:0.1 .
20+
$ docker build -t sieve_benchmark:0.2 .
2121
```
2222

2323
### Manually (Alternative)
@@ -77,7 +77,7 @@ Since SiFit requires a large amount of memory even working on a small dataset, t
7777
The docker image only contains executables of all the benchmarked tools. To run the pipeline, you need to mount the local directory to this repository containing the snakemake rules and supporting scripts to the docker container under `/root/data`:
7878

7979
```bash
80-
$ docker run -n sieve_benchmark -v /local/path/to/SIEVE_benchmark_pipeline:/root/data senbaikang/sieve_benchmark:0.1
80+
$ docker run --name sieve_benchmark -v /local/path/to/SIEVE_benchmark_pipeline:/root/data senbaikang/sieve_benchmark:0.2
8181
```
8282

8383
The console output of snakemake will appear in the terminal. To run the pipeline in the background, add `-d` to the command above before the image name, and access the console outputs through:
@@ -108,7 +108,7 @@ $ snakemake --use-conda --cores {NUM} -kp
108108
If benchmarking of the efficiency is of the concern, the snakemake file containing the corresponding rules should be used. Hence, the default commands specified in the docker image must be overwritten. To do so, run the docker container with the following command:
109109

110110
```bash
111-
$ docker run -n sieve_benchmark -v /local/path/to/SIEVE_benchmark_pipeline:/root/data senbaikang/sieve_benchmark:0.1 snakemake --use-conda --cores all -s efficiency_benchmark.snake --rerun-triggers mtime -kp
111+
$ docker run --name sieve_benchmark -v /local/path/to/SIEVE_benchmark_pipeline:/root/data senbaikang/sieve_benchmark:0.2 snakemake --use-conda --cores all -s efficiency_benchmark.snake --rerun-triggers mtime -kp
112112
```
113113

114114
#### Manually

config.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,9 +17,6 @@ tools:
1717
# TO BE SET [MANDATORY]
1818
sifitRemote: /path/to/server/sifit
1919

20-
# TO BE SET [MANDATORY (not for docker)]
21-
monovar: monovar
22-
2320
# TO BE SET [MANDATORY (not for docker)]
2421
sciphi: sciphi
2522

monovar.snake

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,6 @@ rule runMonovarWithTrueParams:
3535
output:
3636
protected(MONOVARTRUEPARAMSDIR + "{dataType}/{datasetName}/monovar.vcf")
3737
params:
38-
monovar=config["tools"]["monovar"],
3938
fpPrior=config["benchmark"]["analysis"]["effSeqErrRate"],
4039
fnPrior=config["benchmark"]["analysis"]["adoRate"],
4140
prefix=MONOVARTRUEPARAMSDIR + "{dataType}/{datasetName}/"
@@ -47,7 +46,7 @@ rule runMonovarWithTrueParams:
4746
repeat(EFFICIENCYMONOVARDIR + "{dataType}/{datasetName}.tsv", config["benchmark"]["efficiency"]["repetition"])
4847
shell:
4948
"cat {input.mpileupFile} | "
50-
"{params.monovar} "
49+
"monovar "
5150
"-p {params.fpPrior} "
5251
"-a {params.fnPrior} "
5352
"-t 0.05 "

0 commit comments

Comments
 (0)