Skip to content
This repository was archived by the owner on Sep 24, 2025. It is now read-only.

add docker file instructions #37

Open
OeslleLucena wants to merge 3 commits into
base: master
Choose a base branch
from
Open

add docker file instructions #37

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
97a884b
add docker file instructions
OeslleLucena Jul 15, 2020
b59120c
Update README.md
OeslleLucena Jul 21, 2020
32b2f02
Create Dockerfile
OeslleLucena Jul 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 19 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,25 @@ pip install tox
tox
```

### Docker Image
You can also directly use TAPAS via Docker.

For first time use of nvidia-docker, it is important to run the following commands:

```
distribution=$(. /etc/os-release;echo $ID$VERSION_ID)
curl -s -L https://nvidia.github.io/nvidia-docker/gpgkey | sudo apt-key add -
curl -s -L https://nvidia.github.io/nvidia-docker/$distribution/nvidia-docker.list | sudo tee /etc/apt/sources.list.d/nvidia-docker.list

@eisenjulian eisenjulian Jul 20, 2020

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there and official documentation for this steps, it would be better to link directly there and keep the comment in the readme short.

sudo apt-get update && sudo apt-get install -y nvidia-container-toolkit
sudo systemctl restart docker
```

To pull and run the docker image (contains all prerequisites) you can call the following command:

```
sudo docker run -it --gpus all -v /absolute/path/to/my/data/directory:/home/data oeslle/tapas:1.0

@eisenjulian eisenjulian Jul 20, 2020

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we would prefer to include the Dockerfile directly in the repository instad of linking to an external source

@OeslleLucena OeslleLucena Jul 21, 2020

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @eisenjulian I added the Dockerfile. However, due to issue #33 tensorflow-probability==0.10.1 must be in the requirements.txt file and tox.ini should have envlist = py36 only.

```

## Data

See the section below for the pre-training data.
Expand Down