Dockerize #7
Dockerize #7
Conversation
make sure to change the repo namespace from `bitstrider` to `bobbens` when merging
|
I can't comment on the docker pull request much as I don't use docker, but it is likely that you will have to make the docker image use pytorch 0.4.1 and not latest (see issue #8 ) |
|
you're right thanks for the tip |
|
unfortunately there's no 0.4.1 image available https://hub.docker.com/r/pytorch/pytorch/tags do you know if 0.4.0 works? |
|
I assume it would, but someone would have to test it to make sure. |
| ./figs.sh | ||
| python simplify.py --img test_line.png --out out_rough.png --model model_pencil2.t7 | ||
| ``` | ||
|
|
There was a problem hiding this comment.
After building this image and running it interactively, inside the shell, running figs.sh gives:
root@da71b5a52837:/workspace/sketch_simplification# ./figs.sh
Processing fig01_eisaku.png...Traceback (most recent call last):
File "simplify.py", line 4, in
from torch.utils.serialization import load_lua
ModuleNotFoundError: No module named 'torch.utils.serialization'
root@da71b5a52837:/workspace/sketch_simplification#
Added support for deploying with Docker, including usage notes.
Make sure to change the repo namespace from
json2dtobobbenswhen merging, inDockerfileandREADME.md