You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2020/09/16 16:53:41 UTC

[GitHub] [incubator-tvm] yongwww opened a new pull request #6493: [wip][TOPI][Relay] unbind op

yongwww opened a new pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493


   cc @zhiics @kevinthesun 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] yongwww commented on pull request #6493: [wip][TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
yongwww commented on pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493#issuecomment-693883967


   cc @were @icemelon9 @kevinthesun to add shapefunc for unbind, I need to return a tuple of shapes using hybrid script, and the size/length of the tuple depends on the input data shape,  seems hybrid doesn't support this. Any suggestion?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] yongwww edited a comment on pull request #6493: [wip][TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
yongwww edited a comment on pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493#issuecomment-693883967


   cc @were @icemelon9 @kevinthesun to add [shapefunc for unbind,](https://github.com/apache/incubator-tvm/blob/cc579eb3a21a7a9ca282566da01387dcae63dd48/python/tvm/relay/op/_transform.py#L717-L737) I need to return a tuple of shapes using hybrid script, and the size/length of the tuple depends on the input data shape,  seems hybrid doesn't support this. Any suggestion?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] tqchen closed pull request #6493: [TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
tqchen closed pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493


   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] yongwww commented on pull request #6493: [TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
yongwww commented on pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493#issuecomment-698452432


   Seems we haven't encountered dynamic cases of unbind in PyTorch od applications so far. Moved dynamism support in todo, hybrid and TupleWrapper might need to be changed to support dynamic tuple size.
   
   Also tried to compose with tensorarray [in this branch](https://github.com/yongwww/tvm/commit/7fac56430bc5adcae4e295f803aa233d344e209e), failed in type checking. 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] kevinthesun commented on pull request #6493: [wip][TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
kevinthesun commented on pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493#issuecomment-694513122


   @yongwww Probably we can use tensor array to compose unbind.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] yongwww commented on pull request #6493: [TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
yongwww commented on pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493#issuecomment-698452432


   Seems we haven't encountered dynamic cases of unbind in PyTorch od applications so far. Moved dynamism support in todo, hybrid and TupleWrapper might need to be changed to support dynamic tuple size.
   
   Also tried to compose with tensorarray [in this branch](https://github.com/yongwww/tvm/commit/7fac56430bc5adcae4e295f803aa233d344e209e), failed in type checking. 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [incubator-tvm] yongwww edited a comment on pull request #6493: [wip][TOPI][Relay] unbind op

Posted by GitBox <gi...@apache.org>.
yongwww edited a comment on pull request #6493:
URL: https://github.com/apache/incubator-tvm/pull/6493#issuecomment-693883967


   cc @were @icemelon9 @kevinthesun to add [shapefunc for unbind,](https://github.com/apache/incubator-tvm/blob/cc579eb3a21a7a9ca282566da01387dcae63dd48/python/tvm/relay/op/_transform.py#L717-L735) I need to return a tuple of shapes using hybrid script, and the size/length of the tuple depends on the input data shape,  seems hybrid doesn't support this. Any suggestion?


----------------------------------------------------------------
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:
users@infra.apache.org