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/15 07:09:53 UTC

[GitHub] [incubator-tvm] liangfu commented on issue #4293: [VTA] Bug fix for padded load with large inputs

liangfu commented on issue #4293: [VTA] Bug fix for padded load with large inputs
URL: https://github.com/apache/incubator-tvm/pull/4293#issuecomment-554241928
 
 
   @vegaluisjose Yes, current Chisel implement passes all the unit-tests.
   
   @tmoreau89 I did not test this with HLS version yet. If you want, I think I can test this with PYNQ hardware next week.
   
   On the other hand, I think some recent changes (either in the runtime or the compiler) has made this broken. I mean even without this PR, unit test with `fsim` could even fail in some cases (not always). I can reproduce the failure case by fixing random seed. All my successful tests are based on eariler version of runtime & compiler. This PR only focus on the hardware improvements.

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