Skip to content
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

Seems we need add unit test to make sure onnx work properly. #22

Closed
pkuyym opened this issue Apr 13, 2018 · 2 comments
Closed

Seems we need add unit test to make sure onnx work properly. #22

pkuyym opened this issue Apr 13, 2018 · 2 comments
Labels
Enhancement New feature or request

Comments

@pkuyym
Copy link
Collaborator

pkuyym commented Apr 13, 2018

I execute sh setup.sh and all python dependencies are installed successfully. However, I can't run convert.py and the error message is:

Traceback (most recent call last):
  File "convert.py", line 18, in <module>
    from onnx import helper, checker
  File "/usr/local/lib/python2.7/dist-packages/onnx/__init__.py", line 10, in <module>
    import onnx.helper  # noqa
  File "/usr/local/lib/python2.7/dist-packages/onnx/helper.py", line 15, in <module>
    import onnx.defs as defs
  File "/usr/local/lib/python2.7/dist-packages/onnx/defs/__init__.py", line 6, in <module>
    import onnx.onnx_cpp2py_export.defs as C
ImportError: /usr/local/lib/python2.7/dist-packages/onnx/onnx_cpp2py_export.so: undefined symbol: _ZNK6google8protobuf7Message13SpaceUsedLongEv
@pkuyym pkuyym added the Enhancement New feature or request label Apr 13, 2018
@pkuyym pkuyym changed the title Seems we have to add unit test to make sure onnx work properly. Seems we need add unit test to make sure onnx work properly. Apr 13, 2018
@kuke
Copy link

kuke commented Apr 14, 2018

It should be a known problem in onnx, see:

onnx/onnx#489
onnx/onnx-caffe2#19

@pkuyym
Copy link
Collaborator Author

pkuyym commented Apr 15, 2018

@kuke Thanks, I suspect that incorrect version of protoc would results in the failure. So we should make the installation for onnx more robust.

Zeref996 pushed a commit to Zeref996/Paddle2ONNX that referenced this issue Aug 24, 2021
* add optimizer

* add base run script

* git optimizer run script
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants