bernardomig / atlas-augmented-perception

An augmented perception for AtlasCar@UA for detecting cars and pedestrians
0 stars 0 forks source link

Create makefile for compiling protos #1

Open miguelriemoliveira opened 6 years ago

miguelriemoliveira commented 6 years ago

We are using google protocol buffers 3.

Should compile in python and csharp.

bernardomig commented 6 years ago

It's done.

To compile:

Note: the generation of messages requires the protoc runtime and the grpc plugin for the protoc.

miguelriemoliveira commented 6 years ago

Hi @bernardomig ,

this

for python: pip install grpcio grpcio_utils for csharp: nuget Grpc.Tools and copy the binary grpc_csharp_plugin to /usr/bin or /usr/local/bin

should also be in the readme (probably installation sectioin). Can you add it?

miguelriemoliveira commented 6 years ago

Hi @bernardomig

  1. I was testing the Makefile. I think it should be in /interface rather than /interface/generated . It took me a while to find it.

  2. I could not find grpcio_utils ... I used grpcio-tools. Is that ok?

  3. Added a requirements.txt file. Now we can install all dependencies with

    sudo pip install -r requirements.txt

miguelriemoliveira commented 6 years ago

I cannot compile ...

e gen-python mkdir -p python touch python/init.py python -m grpc_tools.protoc \ -I=../protos \ --python_out=python \ --grpc_python_out=python \ ../protos/*.proto /usr/bin/python: No module named grpc_tools Makefile:16: recipe for target 'gen-python' failed

any ideas?