-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ModelProto::ParseFromIstream missing #6
Comments
Sounds like a ProtoBuf version mismatch. I'm not too familiar with it either - it is just a ONNX requirement. |
I wrote too hastily. Master branch was indeed broken on 0186da3 on Ubuntu 16.04 (when building clean - which I didn't, so I didn't notice it was broken), and I could repeat the problem you saw. |
Excellent, this fixes it indeed! Thanks a bunch! |
Just for the sake of completeness,
So whatever change made it work out of the box should have happened between Ubuntu 20.04 and 20.10... |
… issues when using protobug 3.6.1 (see kraiskil/onnx2c#6)
… issues when using protobug 3.6.1 (see kraiskil/onnx2c#6)
… issues when using protobug 3.6.1 (see kraiskil/onnx2c#6)
Hi! I'm trying to compile this repo, but get the following message when running
make
:I'm not really familiar with ProtoBuf stuff (or ONNX, for that matter), but I did everything like the readme suggests, dependencies installed, submodules at their respective commits. Any idea what I'm missing?
The text was updated successfully, but these errors were encountered: