Skip to content

Commit eaae1bc

Browse files
committed
Fix workaround in documentation for #78
1 parent 5ef62b2 commit eaae1bc

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

DOCKER.md

+6-2
Original file line numberDiff line numberDiff line change
@@ -70,11 +70,15 @@ The `/home/appuser/best/out` folder (inside the container) is connected to your
7070
Download the dataset:
7171
```bash
7272
cd /home/appuser/best/downloader
73-
python downloader.py ../out
73+
mkdir tmp
74+
python downloader.py /tmp
75+
cd /tmp
76+
mv * ../../out
7477
```
78+
The `mv` command will throw an error, but you can ignore this.
7579

7680
## Issues
7781

7882
* The example above does not work, since there are apparently still some permission issues
7983
* The current Dockerfile uses alpine linux as its base image. Consequently the build time is very long. This can be optimised further by using a slightly bigger minimal ubuntu image for example for which precompiled libraries exist.
80-
* The current Dockerfile contains a lot of `RUN` statements, this does not allow for minimal image size. The reason why this is not implemented is because the dockerfile is still in development, and the multiple run statements allow for caching during build time.
84+
* The current Dockerfile contains a lot of `RUN` statements, this does not allow for minimal image size. The reason why this is not implemented is because the dockerfile is still in development, and the multiple run statements allow for caching during build time.

0 commit comments

Comments
 (0)