Skip to content

Commit

Permalink
Merge pull request #478 from azavea/rde/fix/filesystem-classes
Browse files Browse the repository at this point in the history
Small fixes
  • Loading branch information
lossyrob committed Oct 10, 2018
2 parents ea0ed7b + 1635261 commit 8a1acb7
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 8 deletions.
6 changes: 3 additions & 3 deletions .travis/test
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
#!/bin/bash

docker run -v $(pwd):$(pwd) --rm -it raster-vision-cpu rm -f $(pwd)/.coverage $(pwd)/coverage.xml
docker run --rm -it raster-vision-cpu /opt/src/scripts/style_tests && \
docker run -w $(pwd) -v $(pwd):$(pwd) --rm -it raster-vision-cpu rm -f $(pwd)/.coverage $(pwd)/coverage.xml
docker run -w $(pwd) -v $(pwd):$(pwd) --rm -it raster-vision-cpu $(pwd)/scripts/style_tests && \
docker run -w $(pwd) -v $(pwd):$(pwd) --rm -it raster-vision-cpu $(pwd)/scripts/compile && \
docker run -w $(pwd) -v $(pwd):$(pwd) --rm -it raster-vision-cpu $(pwd)/scripts/unit_tests && \
docker run -v $(pwd):$(pwd) --rm -it raster-vision-cpu $(pwd)/scripts/integration_tests
docker run -w $(pwd) -v $(pwd):$(pwd) --rm -it raster-vision-cpu $(pwd)/scripts/integration_tests
5 changes: 3 additions & 2 deletions Dockerfile-cpu
Original file line number Diff line number Diff line change
Expand Up @@ -29,14 +29,15 @@ RUN pip install -r requirements-dev.txt
# Install extras_requirements.json
# Don't install tensorflow
COPY ./extras_requirements.json /opt/src/extras_requirements.json
RUN cat extras_requirements.json | jq '.[][]' | grep -v 'tensorflow' | sort -u | xargs pip install
RUN cat extras_requirements.json | jq '.[][]' | grep -v 'tensorflow' | sort -u | xargs pip install

# Install requirements.txt
COPY ./requirements.txt /opt/src/requirements.txt
RUN pip install -r requirements.txt

COPY scripts/rastervision /usr/local/bin/
COPY ./ /opt/src/
COPY rastervision /opt/src/rastervision
COPY scripts/compile /opt/src/scripts/compile
RUN /opt/src/scripts/compile

CMD ["bash"]
3 changes: 2 additions & 1 deletion Dockerfile-gpu
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,8 @@ COPY ./requirements.txt /opt/src/requirements.txt
RUN pip install -r requirements.txt

COPY scripts/rastervision /usr/local/bin/
COPY ./ /opt/src/
COPY rastervision /opt/src/rastervision
COPY scripts/compile /opt/src/scripts/compile
RUN /opt/src/scripts/compile

CMD ["bash"]
4 changes: 2 additions & 2 deletions rastervision/filesystem/__init__.py
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# flake8: noqa

from rastervision.filesystem.filesystem import (FileSystem, NotReadableError,
NotWritableError)
from rastervision.filesystem.filesystem import (
FileSystem, NotReadableError, NotWritableError, ProtobufParseException)
from rastervision.filesystem.local_filesystem import LocalFileSystem
from rastervision.filesystem.s3_filesystem import S3FileSystem
from rastervision.filesystem.http_filesystem import HttpFileSystem

0 comments on commit 8a1acb7

Please sign in to comment.