Skip to content

Commit

Permalink
Fix workaround in documentation for #78
Browse files Browse the repository at this point in the history
JosseVanDelm committed Jul 25, 2019
1 parent 5ef62b2 commit eaae1bc
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions DOCKER.md
Original file line number Diff line number Diff line change
@@ -70,11 +70,15 @@ The `/home/appuser/best/out` folder (inside the container) is connected to your
Download the dataset:
```bash
cd /home/appuser/best/downloader
python downloader.py ../out
mkdir tmp
python downloader.py /tmp
cd /tmp
mv * ../../out
```
The `mv` command will throw an error, but you can ignore this.

## Issues

* The example above does not work, since there are apparently still some permission issues
* 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.
* 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.
* 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 comments on commit eaae1bc

Please sign in to comment.