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

ValueError: Unsupported Ops in the model before optimization LogSoftmax #684

Closed
rumschuettel opened this issue Sep 6, 2018 · 3 comments · Fixed by tensorflow/tfjs-core#1342
Assignees

Comments

@rumschuettel
Copy link

Hi, I'm trying to convert a .pb file using

tensorflowjs_converter --input_format=tf_frozen_model --output_node_names='LogSoftmax,concat_40,concat_41' --saved_model_tags=serve test.pb testout/test

and get an error

Using TensorFlow backend.
Traceback (most recent call last):
  File "/home/foo/opt/anaconda5/envs/nlp-onnx/bin/tensorflowjs_converter", line 11, in <module>
    sys.exit(main())
  File "/home/foo/opt/anaconda5/envs/nlp-onnx/lib/python3.6/site-packages/tensorflowjs/converters/converter.py", line 272, in main
    strip_debug_ops=FLAGS.strip_debug_ops)
  File "/home/foo/opt/anaconda5/envs/nlp-onnx/lib/python3.6/site-packages/tensorflowjs/converters/tf_saved_model_conversion.py", line 325, in convert_tf_frozen_model
    skip_op_check=skip_op_check, strip_debug_ops=strip_debug_ops)
  File "/home/foo/opt/anaconda5/envs/nlp-onnx/lib/python3.6/site-packages/tensorflowjs/converters/tf_saved_model_conversion.py", line 113, in optimize_graph
    ', '.join(unsupported))
ValueError: Unsupported Ops in the model before optimization
LogSoftmax

tensorflow.js version

Using TensorFlow backend.

tensorflowjs 0.5.7

Dependency versions:
  keras 2.1.6
  tensorflow 1.9.0

Is that operation not satisfied? Any way I could add it?
Thanks a lot!
/ J

@pyu10055
Copy link
Collaborator

pyu10055 commented Sep 8, 2018

@rumschuettel It would be great f you would like to contribute, please refer to this issue for guidance.
#624

@prathameshnetake
Copy link

Worked after using --skip_op_check=SKIP_OP_CHECK flag

@yhwang
Copy link
Contributor

yhwang commented Oct 25, 2018

Hi @pyu10055, I open a PR for LogSoftmax op and you can see the link above. Hope you can provide review/comment. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants