iennacca / aconcagua Goto Github PK

test
I noticed this has changed in your code

client = aconcagua_pb2_grpc.TimeseriesDataServiceStub(channel)

before it was

client = aconcagua_pb2_grpc.AconcaguaStub(channel)

and your original code is running against localhost so I have attached my code so you can
see the changes I made and catch any errors on my part.

I just set the server’s URL and the DMX file and series to ones that worked before.
The error is happening before I get to asking for the series.

• Compiling for grpc-web needs an additional executable pulled from grpc-web. The Windows executable from the release page is suffixed with version and OS information; (i.e., protoc-gen-grpc-web-1.0.3-windows-x86_64.exe); this needs to be renamed to protoc-gen-grpc-web.exe. In generate_proto.bat, I stuck it in the TOOLS_PATH for simplicity's sake [Ideally, this should be addressed by the package manager]

• The js compiler either compiled the imported proto files in some unknown folder or it didn't have precompiled versions in the first place. I'm still trying to figure this out. As a workaround, I had to force-compile (in generate_protos.bat) the ff. Google-created proto files that got written in the google/api and google/protobuf folders:

  • annotations.proto
  • http.proto
  • empty.proto

Note that empty.proto was referenced as well, but I think since it was in the google-protobuf npm package it compiled without errors. [Ideally we shouldn't have to recompile Google's reference proto files]