[GitHub] [tvm] zenyuhao opened a new issue, #12672: [Bug] its fuzziness that the num of outputs per node > 1

[GitBox <gi...@apache.org>]

zenyuhao opened a new issue, #12672:
URL: https://github.com/apache/tvm/issues/12672

   
   https://github.com/apache/tvm/blob/main/src/runtime/graph_executor/graph_executor.cc#L100
   
   my model has 7 outputs of last node  named "reshap_op"
   but  there is only one node-name in node map,
   i think outputs_map should store out_tensor_index , should be  "reshape_nop:0" as tensorflow does
   
   ~~~
   
   >>> c = m.get_function("default")(tvm.cpu(0))
   reshape_nop:0
   reshape_nop:1
   reshape_nop:2
   reshape_nop:3
   reshape_nop:4
   reshape_nop:5
   reshape_nop:6
   
   >>> c.get_function("get_output_index")("reshape_nop")
   6
   
   ~~~
   
   
   Please don’t hesitate to correct me if I’m wrong. Many thanks in advance.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] zenyuhao commented on issue #12672: [Bug] its fuzziness that the num of outputs per node > 1

[GitBox <gi...@apache.org>]

zenyuhao commented on issue #12672:
URL: https://github.com/apache/tvm/issues/12672#issuecomment-1240326084

   thanks for reply
   In tensorflow graph, output of graph node is naming by `op_name:output_index`, so at least  there alwasy be 0 after outout.
   im not sure that the same design in onnx, and i think the stable way to get output by output index is offical in tvm doc,
   even though get_output_by_name is not stable in some situation, here is simple fix  which works well
   
   
   ```cpp
     for (size_t i = 0; i < outputs_.size(); i++) {
       const uint32_t nid = outputs_[i].node_id;
       std::string& name = nodes_[nid].name;
       std::stringstream ss;
       ss << name << ":" << i;
       output_map_[ss.str()] = i;
     }
   ```
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] masahi closed issue #12672: [Bug] its fuzziness that the num of outputs per node > 1

[GitBox <gi...@apache.org>]

masahi closed issue #12672: [Bug] its fuzziness that  the num of outputs per node  > 1
URL: https://github.com/apache/tvm/issues/12672


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [tvm] cery999 commented on issue #12672: [Bug] its fuzziness that the num of outputs per node > 1

[GitBox <gi...@apache.org>]

cery999 commented on issue #12672:
URL: https://github.com/apache/tvm/issues/12672#issuecomment-1237594433

   I can support this feature. but I found some problem.
   if there is one result in onnx ,the name should be output:0 or output? 
   and can you support a case to reproduce it, so I can add it to ci?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org