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 2019/11/14 04:48:01 UTC

[GitHub] [incubator-tvm] anijain2305 opened a new pull request #4335: [Relay][WIP] ConvertLayout pass.

anijain2305 opened a new pull request #4335: [Relay][WIP] ConvertLayout pass.
URL: https://github.com/apache/incubator-tvm/pull/4335
 
 
   RFC - https://discuss.tvm.ai/t/layout-conversion-pass/4009
   
   WIP because of 2 reasons
   * Add tests
   * ConvertLayout can't be called twice, because init function has a registered function. Calling it again leads to following error
   ~~~
   TVMError: Check failed: p.second != plevel: Attribute FTVMConvertOpLayout of operator nn.conv2d is already registered with same plevel=10
   ~~~
   
   @tqchen @yzhliu Would appreciate early feedback if possible. For the above error, does it make sense to deprecate plevel?

----------------------------------------------------------------
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


With regards,
Apache Git Services