### Summary
In today’s TVM TensorFlow frontend, there is only limited support for control 
flow, which resulting in difficult in covering TensorFlow object detection 
models. In this RFC, we will discuss how to improve current TVM TF frontend to 
fully support TensorFlow control flow.

### Solution
#### Visit TensorFlow Graph Def
Currently TVM parses TensorFlow Graph Def node by node in topological order. 
However, topological order might not stand any more if control flow is 
introduced into graph def. We do the following to alter the visiting order:

1. Visit all nodes in original graph def order and fetch all control flow nodes.
2. For all control flows nodes, modify the order so that for each while_loop, 
all Exit nodes sit at the end.
3. Visit all control flow nodes in modified order and generate corresponding 
Relay Functions. For each control flow node, we backtrack all its ancestor 
nodes until input nodes.
4. Visit and parse other unvisited nodes.

#### Parse Control Flow nodes
In TVM, while loop is represented as a recursive function. To convert a 
TensorFlow while_loop block into a Relay function, the following assumption 
must stand:

1. All `Exit` nodes under the same while_loop must have the same node name 
prefix as this while loop block, which is got with `node.name.rsplit('/', 
1)[0]`. In TensorFlow 1.x, user can specify the name of while_loop, but `Exit` 
node prefix is automated generated with ancestor nodes’ names. 
2. `Merge`, `Switch`, `NextIteration` and `Exit` nodes with the same postfix 
number are belong to the same iteration body. For example, `Merge_1`, 
`Switch_1`, `NextIteration_1` and `Exit_1` should refer to the same iteration 
body. Since these four nodes are generated when user creating while_loop, the 
postfix number can’t be arbitrarily set.

We convert a while loop block to a Relay function with the following method:

1. When we get a `Merge` node inside a while_loop block: if it is a direct 
child of a `while_loop` block, we create a Loop wrapper to start adding all 
sub-nodes under this loop block. Otherwise, we create a complete branch op with 
previously stored Switch condition.
2. When we get an `Enter` node: we simply convert its input node to Relay node.
3. When we get a `LoopCond` node: we first convert its input node to Relay 
node. Then we set the converted node as our Loop wrapper condition.
4. When we get a `Switch` node: we first convert both its input and condition 
to Relay nodes. If it is a direct child of a  while_loop block, we add 
converted input to loop variable list of the corresponding Loop wrapper. 
Otherwise, we create a Branch wrapper, set its condition to be converted 
condition node and add it to a global branch dictionary.
5. When we get an `Exit` node: we generate Relay function for the corresponding 
while loop body.

### Tested models
With above changes(together with pending PR of dynamic NMS), we can convert and 
compile TensorFlow object detection models. The following models from 
TensorFlow object detection model zoo are tested: 
ssd_mobilenet_v1_coco
ssd_mobilenet_v2_coco 
ssd_resnet_50_fpn_coco
mask_rcnn_inception_v2_coco
mask_rcnn_resnet50_atrous_coco
faster_rcnn_resnet50_coco

###TODO
Investigate TensorFlow 2.0 control flow and what changes are required.

@tqchen @jroesch @srkreddy1238 @zhiics @yongwww @wweic 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/apache/incubator-tvm/issues/4969

Reply via email to