Skip to content
This repository has been archived by the owner on Oct 13, 2021. It is now read-only.

Add both tf.nn.X and tf.compat.v1.nn.X to activation_map #513

Merged
merged 9 commits into from
Jun 4, 2020

Conversation

jiafatom
Copy link
Collaborator

@jiafatom jiafatom commented Jun 3, 2020

The issue cannot convert softmax activation because operator.raw_operator.activation is a function that is not the same as tf.nn.softmax or keras.activations.get('softmax'). We need import tf from keras2onnx.proto.tfcompat

@jiafatom jiafatom requested a review from wenbingl June 3, 2020 23:36
@jiafatom jiafatom changed the title Update activation_map by adding name as key Add both tf.nn.X and tf.compat.v1.nnx.X to activation_map Jun 4, 2020
@jiafatom jiafatom changed the title Add both tf.nn.X and tf.compat.v1.nnx.X to activation_map Add both tf.nn.X and tf.compat.v1.nn.X to activation_map Jun 4, 2020
@jiafatom jiafatom merged commit 85e8132 into onnx:master Jun 4, 2020
@jiafatom jiafatom deleted the softmax_non_act branch June 4, 2020 02:25
@jiafatom jiafatom mentioned this pull request Jun 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants