liangfu commented on a change in pull request #4703: [VTA] Support network
which have no unique operator as start/stop name for graph pack.
URL: https://github.com/apache/incubator-tvm/pull/4703#discussion_r367749718
##########
File path: vta/python/vta/top/graphpack.py
##########
@@ -246,32 +246,38 @@ def visit_call(self, call):
class BT(Exception):
pass
-def get_subgraph(expr, start_name, stop_name):
+def get_subgraph(expr, start_name, stop_name, start_name_idx=-1,
stop_name_idx=-1):
Review comment:
I suggest setting default `start_name_idx` being 0 (first occurrence of the
operator), while keeping `stop_name_idx` being -1 (last occurrence).
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
