Skip to content

Commit

Permalink
Merge pull request #31 from triagemd/no-update-pip
Browse files Browse the repository at this point in the history
Use python -m pip instead
  • Loading branch information
stephensolis authored May 10, 2018
2 parents 5dde0a0 + 07ddaea commit 571d923
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -4,13 +4,13 @@ FROM debian:latest
RUN apt-get -y update && \
apt-get -y install python-pip python3-pip && \
apt-get -y clean && \
pip2 install --upgrade --upgrade-strategy=eager --no-cache-dir pip stored && \
pip3 install --upgrade --upgrade-strategy=eager --no-cache-dir pip stored
python2 -m pip install --upgrade --upgrade-strategy=eager --no-cache-dir pip stored && \
python3 -m pip install --upgrade --upgrade-strategy=eager --no-cache-dir pip stored

# we install these packages at runtime in the entrypoint, but might as well fetch the latest version into the container
RUN mkdir /pip && \
pip2 download -d /pip tensorflow keras h5py model-converters && \
pip3 download -d /pip tensorflow keras h5py model-converters
python2 -m pip download -d /pip tensorflow keras h5py model-converters && \
python3 -m pip download -d /pip tensorflow keras h5py model-converters
# note: don't install model-converters here, since it declares a dependency on tensorflow and keras (so those will be installed too)
# which is a waste of space since they get reinstalled in the entrypoint

Expand Down

0 comments on commit 571d923

Please sign in to comment.