Handling of Assert Ops ? about tfjs HOT 5 CLOSED

From @pyu10055 on March 21, 2018 20:50

The Assert op is a control flow op which requires run-time eval of the tensor, which will be my goal to support with other control op (merge, switch, ...).
Can you tell what is your use case for Assert in inference mode, do you expect an error, or it is a by-product of the training setup?

from tfjs.

From @pyu10055 on March 21, 2018 23:46

I notice Assert is usually accompanied by Switch op, does your model only have the Assert op unsupported? If so, do you mind sharing the model, so I can investigate a bit more. Thanks.

from tfjs.

From @pyu10055 on March 31, 2018 18:2

We believe that the Assert op should not exist in the inference graph, we will remove the assert and other debug nodes from the graph during the conversion phase.

from tfjs.

+1 on this issue. It would be extremely painful for me (requiring hundreds of lines of code changes) to manually remove all the assertions from my graph because they are generated by library code (e.g. DeepMind's sonnet)

from tfjs.

Even core API calls such as tf.nn.dynamic_rnn contain asserts: https://github.com/tensorflow/tensorflow/blob/r1.9/tensorflow/python/ops/rnn.py#L597

from tfjs.