[GitHub] [incubator-tvm] mbrookhart commented on pull request #5826: [DYNAMIC] Add Dynamic reshape to a dynamic namespace and add DynamicToStatic Pass

2020-06-26 Thread GitBox 


mbrookhart commented on pull request #5826:
URL: https://github.com/apache/incubator-tvm/pull/5826#issuecomment-650348127


   @icemelon9 @lixiaoquan Per Haichen's request, I removed the dynamic option 
from the standard reshape and changed the places that used it to use the new 
purely dynamic op. This touches many more files, could you take another look?



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

[GitHub] [incubator-tvm] mbrookhart commented on pull request #5826: [DYNAMIC] Add Dynamic reshape to a dynamic namespace and add DynamicToStatic Pass

2020-06-20 Thread GitBox 


mbrookhart commented on pull request #5826:
URL: https://github.com/apache/incubator-tvm/pull/5826#issuecomment-647008054


   :+1: Happy to shorten it.



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

[GitHub] [incubator-tvm] mbrookhart commented on pull request #5826: [DYNAMIC] Add Dynamic reshape to a dynamic namespace and add DynamicToStatic Pass

2020-06-18 Thread GitBox 


mbrookhart commented on pull request #5826:
URL: https://github.com/apache/incubator-tvm/pull/5826#issuecomment-646347835


   Hitting an issue with dynamic shapes on GPU with the VM.



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