Skip to content
This repository has been archived by the owner on Aug 30, 2022. It is now read-only.

Fixes issue with missing dependencies in docker image and linter errors #52

Merged
merged 4 commits into from
Sep 11, 2019
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
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
4 changes: 4 additions & 0 deletions .dockerignore
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# Project
results
output

# Pantsbuild
.pants.d
.pants.workdir.file_lock
Expand Down
1 change: 1 addition & 0 deletions Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,6 @@ RUN python setup.py egg_info && \
pip install $IR

COPY xain xain
COPY protobuf protobuf

RUN pip install -e .
1 change: 1 addition & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ POLITE NOTE: We want to point out that running the benchmarks as described below
XAIN requires the following tools to be installed:

- [Python 3.6+](https://python.org/)
- clang-format (on macOS: `brew install clang-format`)

### Clone Repository & Install XAIN

Expand Down
3 changes: 1 addition & 2 deletions protobuf/xain/grpc/hellonumproto.proto
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,7 @@ import "numproto/protobuf/ndarray.proto";
package hellonumproto;

service NumProtoServer {
rpc SayHelloNumProto(NumProtoRequest) returns (NumProtoReply) {
}
rpc SayHelloNumProto(NumProtoRequest) returns (NumProtoReply) {}
}

message NumProtoRequest {
Expand Down
4 changes: 2 additions & 2 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,8 @@ def run(self):
"grpcio==1.23.0", # Apache License 2.0
"protobuf==3.9.1", # 3-Clause BSD License
"numproto==0.2.0", # Apache License 2.0
"grpcio-tools==1.23.0", # Apache License 2.0
"mypy-protobuf==1.15", # Apache License 2.0
]

cpu_require = ["tensorflow==1.14.0"] # Apache 2.0
Expand All @@ -68,8 +70,6 @@ def run(self):
"isort==4.3.20", # MIT
"rope==0.14.0", # GNU GPL
"pip-licenses==1.15.2", # MIT License
"grpcio-tools==1.23.0", # Apache License 2.0
"mypy-protobuf==1.15", # Apache License 2.0
]

tests_require = [
Expand Down