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/06/23 09:02:57 UTC

[GitHub] [incubator-tvm] t-vi edited a comment on pull request #5887: keep parameter names from PyTorch

t-vi edited a comment on pull request #5887:
URL: https://github.com/apache/incubator-tvm/pull/5887#issuecomment-648010250


   I found the problem and will send a fix.
   The old code included the embedding weight twice (which arguably is a bug in itself and needs fixing) and the new code deduplicated the param but not the var (which is even worse).
   We underappreciate the structure of the PyTorch input...


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